[PATCH] D147604: [AggressiveInstCombine] Enable also for -O2

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 5 05:39:38 PDT 2023


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

LGTM



================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:509
   FPM.addPass(InstCombinePass());
-  if (Level == OptimizationLevel::O3)
+  if (Level.getSpeedupLevel() > 1)
     FPM.addPass(AggressiveInstCombinePass());
----------------
This check is not needed, O1 is handled separately above.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147604



More information about the llvm-commits mailing list