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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
