[PATCH] D92743: [ScalarizeMaskedMemIntrin] Add new PM support

Tue Dec 8 12:43:28 PST 2020

aeubanks added a comment.

In D92743#2440868 <https://reviews.llvm.org/D92743#2440868>, @anna wrote:

> In D92743#2438777 <https://reviews.llvm.org/D92743#2438777>, @aeubanks wrote:
>
>> can you remove this from the list in https://github.com/llvm/llvm-project/blob/6e614b0c7ed3a9a66428f342bf2a4b3700525395/llvm/tools/opt/opt.cpp#L463 and make sure that no check-llvm tests regress when -enable-new-pm is turned on by default? (I should rename that function)
>
> Yes, pls rename the function since it actually represents which passes are not yet ported to NewPM.

https://reviews.llvm.org/D92818

> Also, I have run the tests with enable-new-pm=true. However, there are 36 failures which are already failing with new-PM without my change (had to build both with and without my change to confirm this). Is there some buildbot that tracks the NewPM option? It will make it easier to catch if any testcases regress...

I'm working on reducing it to 0 so that we can have a bot that checks if it regresses.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92743