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

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 25 11:30:17 PDT 2019


hsaito added a comment.

In D61030#1478957 <https://reviews.llvm.org/D61030#1478957>, @asbirlea wrote:

> Update test.


Have you already had an RFC regarding this change? If so, please add a link here. I bet there are a reasonable number of people who are dependent on vectorizer's interleave behavior is hooked up to the unroll flag. I support the direction as being the right thing to do, but we better give a good warning signal so that the affected people will scope the work. Do we already have a clang level interleave control flag? Else, those who are using unroll flag to control interleaving have to use -mllvm -interleave-loops= to get back to the original behavior......


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61030





More information about the llvm-commits mailing list