[PATCH] D74185: Revert the revert of vectorization commits

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 7 09:49:27 PST 2020


ABataev added a comment.

In D74185#1864233 <https://reviews.llvm.org/D74185#1864233>, @george.karpenkov wrote:

> > Runnable reproducer requires some prerequisites. It would be good if you could provide a simpler one.
>
> I understand, but actually we have already spent a very large amount of time on reducing this.
>
> To make sure we are on the same page: what do you mean by prerequisites? In order to run the reproducer, you only need to wget a single C++ file with no dependencies and link it to the IR dump/


Still, in its current form, it won't help. I already said, I don't see a real problem in the LLVM IR files. Maybe, this optimization just triggers some other dangerous transformations. Need to investigate this thoroughly to understand what is the real cause of the problem.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74185/new/

https://reviews.llvm.org/D74185





More information about the llvm-commits mailing list