[PATCH] D94266: [polly][NewPM][test] Fix polly tests under -enable-new-pm

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 12:36:02 PST 2021


aeubanks marked 2 inline comments as done.
aeubanks added a comment.

In D94266#2507718 <https://reviews.llvm.org/D94266#2507718>, @Meinersbur wrote:

> Note that for same passes, there Polly has NPM equivalents. E.g. `-polly-prepare` has `-passes=polly-prepare`, `-polly-scops` has `-passes='scop(print<polly-scops>)'`.
>
> Assuming this is the accepted workaround for the NPM transition (I personally think it is ugly), this looks fine for the Polly side of things.

I agree it's ugly, but it allows us to update the polly tests to use the new PM in parallel with the new PM switch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94266



More information about the llvm-commits mailing list