[PATCH] D15449: [PassManagerBuilder] Add a few more scalar optimization passes

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 11 06:31:28 PST 2015


jmolloy added inline comments.

================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:568
@@ +567,3 @@
+  PM.add(createInstructionCombiningPass()); // Clean up again
+  PM.add(createAggressiveDCEPass());
+
----------------
hfinkel wrote:
> I'm somewhat biased here, but I recommend using createBitTrackingDCEPass() instead. It should be a pure superset of ADCE (and a bit more powerful).
> 
SGTM, I'm perfectly happy to do that!


Repository:
  rL LLVM

http://reviews.llvm.org/D15449





More information about the llvm-commits mailing list