[PATCH] D145265: [Pipeline] Remove GlobalCleanupPM

[Nikita Popov via Phabricator via cfe-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:476
 
+  FPM.addPass(EarlyCSEPass());
+
----------------
Why the extra EarlyCSE pass in the `O1` pipeline?


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:547
+      SimplifyCFGPass(SimplifyCFGOptions().convertSwitchRangeToICmp(true)));
+  FPM.addPass(AggressiveInstCombinePass());
 
----------------
Any particular reason why InstCombine and AggressiveInstCombine are no longer directly next to each other?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145265