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

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 12:40:07 PST 2020


anna added a comment.

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. 
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...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92743



More information about the llvm-commits mailing list