[PATCH] D137707: Move "auto-init" instructions to the dominator of their users

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 15:15:46 PST 2022


fhahn added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:888
   EarlyFPM.addPass(EarlyCSEPass());
+  EarlyFPM.addPass(MoveAutoInitPass());
   if (Level == OptimizationLevel::O3)
----------------
fhahn wrote:
> What's the rational behind placing it here and not closer to other memory optimizations like DSE/MemCpyOpt?
(moving it closer to those will also allow re-using existing memorySSA)


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

https://reviews.llvm.org/D137707



More information about the llvm-commits mailing list