[PATCH] D14308: [SimplifyCFG] Extend SimplifyResume to handle phi of trivial landing pad.

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 3 15:45:24 PST 2015


reames added inline comments.

================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:2959
@@ +2958,3 @@
+    // can not delete it because it has other dependents.
+    if (!PhiLPInst || TrivialBB->getUniqueSuccessor() == BB) {
+      // Blocks that will be deleted should also be removed from the phi node.
----------------
This is at the wrong point.  We should be checking that the terminator is a unconditional branch before considering the basic block trivial.


http://reviews.llvm.org/D14308





More information about the llvm-commits mailing list