[PATCH] D138034: [LegacyPM] remove unset variables in PassManagerBuilder
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 15 10:04:41 PST 2022
aeubanks added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/PassManagerBuilder.cpp:237
MPM.add(createMergedLoadStoreMotionPass()); // Merge ld/st in diamonds
- MPM.add(NewGVN ? createNewGVNPass()
- : createGVNPass(DisableGVNLoadPRE)); // Remove redundancies
+ MPM.add(createGVNPass(DisableGVNLoadPRE)); // Remove redundancies
}
----------------
mehdi_amini wrote:
> There is no way to enable NewGVNPass anymore? Is this intended?
PassManagerBuilder is deprecated (legacy PM for optimization pipeline is deprecated) and I'm in the process of removing it
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138034/new/
https://reviews.llvm.org/D138034
More information about the llvm-commits
mailing list