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

[Hideki Saito via Phabricator via llvm-commits]

hsaito added a comment.

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

> Thanks a lot for the suggestion! Sent: http://lists.llvm.org/pipermail/llvm-dev/2019-April/131968.html
>
> My intention was to not make any change visible to clang.
>  If clang currently sets the `DisableUnrollLoops`, and llvm will not use that for LoopVectorization, then have clang set `LoopsInterleaved` to the same value as the one used for unroll loops.


You mean, clang setting EnableLoopInterleaving. If needed, we can make that change in clang first, i.e., before D61030 <https://reviews.llvm.org/D61030> goes in, right? If so, there are no immediate surprises to clang users, I think.

> Adding a separate clang flag for interleaving can be a follow-up change.
> 
> Please let me know if this sound reasonable and if I misunderstood how/where clang is setting the flags.

Sounds reasonable to me.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61030