[PATCH] D23225: [ADCE] Modify data structures to support removing control flow

David Callahan via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 16:31:30 PDT 2016


david2050 added a comment.

All comments have been addressed


================
Comment at: lib/Transforms/Scalar/ADCE.cpp:77
@@ +76,3 @@
+  /// Mapping of blocks to associated information, an element in BlockInfoVec.
+  DenseMap<BasicBlock *, BlockInfoType> BlockInfo;
+  bool isLive(BasicBlock *BB) { return BlockInfo[BB].Live; }
----------------
I was made some simplifications in code-to-be added and eliminated BlockInfoVec to just store the information directly in BlockInfo. 


https://reviews.llvm.org/D23225





More information about the llvm-commits mailing list