[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