[PATCH] D111032: Testing instcombine/licm pass order...

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:561
+      LICMPass(PTO.LicmMssaOptCap, PTO.LicmMssaNoAccForPromotionCap),
+      /*UseMemorySSA=*/true, /*UseBlockFrequencyInfo=*/true));
   invokePeepholeEPCallbacks(FPM, Level);
----------------
This LICM run is currently placed to reuse the MSSA analysis computed for MemCpyOpt and DSE. While the LTO change looks reasonable to me, the compile-time impact of this move is unlikely to be worthwhile under global consideration.


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

https://reviews.llvm.org/D111032