mtrofin added a comment.
Another example where there is a discrepancy with the old pass manager: in the
old pass manager (PassManagerBuilder::addFunctionSimplificationPasses):
if (OptLevel > 1) {
if (EnableGVNHoist)
MPM.add(createGVNHoistPass());
(before this change, new pass manager):
if (Level > O1) {
if (EnableGVNHoist)
FPM.addPass(GVNHoistPass());
Which really means "O2 <https://reviews.llvm.org/owners/package/2/>-3, and Os
and Oz". I currently left it backwards compatible - since I'm not sure the
added support for gvn hoisting for Os/z was intentional.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72547/new/
https://reviews.llvm.org/D72547
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
