[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