[PATCH] D61612: [NewPassManager] Add tuning option: ForgetAllSCEVInLoopUnroll [NFC].

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 21:14:26 PDT 2019


asbirlea added a comment.

It is currently turned on by default for Halide, which is still using the old pass manager. It makes a big difference for them in compile times, so switching them to the new pass manager will cause regressions without this flag (see https://github.com/halide/Halide/blob/master/src/CodeGen_LLVM.cpp#L1218).

Alternatively, we could enable it by default, since it benefits clang as well (see timings in D60144 <https://reviews.llvm.org/D60144>).

I don't have a preference either way.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61612





More information about the llvm-commits mailing list