[PATCH] D96780: [NPM][LTO] Update buildLTODefaultPipeline to be more in-line with the old pass manager

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 16 08:45:17 PST 2021


fhahn added a comment.

Thanks for putting up the patch! I am a bit surprised that all those optimizations were not run during LTO and that this did not pop up before the switch.



================
Comment at: llvm/lib/Passes/PassBuilder.cpp:1740
   // Nuke dead stores.
   MainFPM.addPass(DSEPass());
+  // FIXME: once we provide support for enabling MLSM, add it here.
----------------
It looks like there are a few other passes missing here, like `LoopDeletion` and `MergedLoadStoreMotionPass`, and conditionally `LoopInterchange`/`ConstraintElimination`. Should they be added as well? 


================
Comment at: llvm/lib/Passes/PassBuilder.cpp:1767
 
   MainFPM.addPass(InstCombinePass());
   MainFPM.addPass(SimplifyCFGPass(SimplifyCFGOptions().hoistCommonInsts(true)));
----------------
Should we run `WarnMissedTransformationsPass` here, like in the legacy PM?


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

https://reviews.llvm.org/D96780



More information about the llvm-commits mailing list