[PATCH] D23559: [ADCE] Add control dependence computation

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 12:07:21 PDT 2016


mehdi_amini added inline comments.

================
Comment at: lib/Transforms/Scalar/ADCE.cpp:459
@@ -395,2 +458,3 @@
   void getAnalysisUsage(AnalysisUsage &AU) const override {
-    AU.setPreservesCFG();
+    AU.addRequired<PostDominatorTreeWrapperPass>();
+    AU.setPreservesCFG(); // TODO -- will remove when we start removing branches
----------------
How expensive is this?


https://reviews.llvm.org/D23559





More information about the llvm-commits mailing list