[PATCH] D92796: Pin -loop-reduce to legacy PM
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 8 13:23:35 PST 2020
asbirlea added a comment.
In D92796#2440988 <https://reviews.llvm.org/D92796#2440988>, @aeubanks wrote:
> Currently LSR is not tested with the NPM in any circumstance.
> There are no tests with `-passes=loop-reduce`, and LSR is only used in the codegen pass manager, so even with `-DEXPERIMENTAL_NEW_PASS_MANAGER=ON`, it's never in a NPM context.
Exactly! But it's ported. So someone could add it to a pipeline anytime and chaos would ensue, that's what I meant :-). There may be out-of-tree usages or tests that are broken.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92796/new/
https://reviews.llvm.org/D92796
More information about the llvm-commits
mailing list