[PATCH] D20606: [PM] Schedule InstCombine after late LICM run, to clean up LCSSA nodes.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Tue May 24 16:49:34 PDT 2016


joker.eph added inline comments.

================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:547
@@ -545,1 +546,3 @@
+    // Get rid of LCSSA nodes.
+    addInstructionCombiningPass(MPM);
   }
----------------
InstCombine is expensive, do we have an alternative? What is the impact of not doing this cleanup here?


http://reviews.llvm.org/D20606





More information about the llvm-commits mailing list