[PATCH] D61030: [PassManagerBuilder] Add option for interleaved loops, for loop vectorize.

[Hideki Saito via Phabricator via llvm-commits]

hsaito added inline comments.


================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:674
 
-  MPM.add(createLoopVectorizePass(DisableUnrollLoops, !LoopVectorize));
+  MPM.add(createLoopVectorizePass(!LoopsInterleaved, !LoopVectorize));
 
----------------
rengolin wrote:
> I may not have understood the problem, but to keep the previous behaviour, souldn't this be:
>     MPM.add(createLoopVectorizePass(DisableUnrollLoops || !LoopsInterleaved, !LoopVectorize));
I think, the intent of the patch is to change that (but not for clang until next time around).

http://lists.llvm.org/pipermail/llvm-dev/2019-April/131968.html



Repository:
  rL LLVM

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

https://reviews.llvm.org/D61030