[PATCH] D101468: [Passes] Run sinking/hoisting in SimplifyCFG earlier.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 29 04:36:21 PDT 2021


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

I'm actually in favor of doing this pre-inliner, because that will simplify an upcoming patch, should i post it :)
If everyone is okay with extra inlining, i think this is fine.
I checked, and since this runs post-looprotation, we could do this.
So LG.



================
Comment at: llvm/lib/Passes/PassBuilder.cpp:1326-1327
   // before SLP vectorization.
   // FIXME: study whether hoisting and/or sinking of common instructions should
   //        be delayed until after SLP vectorizer.
   OptimizePM.addPass(SimplifyCFGPass(SimplifyCFGOptions()
----------------
Drop comment


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101468



More information about the llvm-commits mailing list