[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