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

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 23 17:00:05 PDT 2019


chandlerc added a comment.

How does this impact use of `clang -fno-unroll-loops` and `clang -fno-loop-vectorize`? I'm betting today, `-fno-unroll-loops` controls the `DisableUnrollLoops` value, and so people who disable both unrolling and vectorization will be surprised when interleaving now happens.

I think we want to set this to true in clang only when both unrolling and vectorization are enabled as a coarse approximation if the separate flag here, mostly because I think those most impacted will be those who disable one or the other and don't want this behavior in that case.


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