PR4317: Handle splits where the new block is unreachable correctly in
authorEli Friedman <eli.friedman@gmail.com>
Wed, 3 Jun 2009 21:42:06 +0000 (21:42 +0000)
committerEli Friedman <eli.friedman@gmail.com>
Wed, 3 Jun 2009 21:42:06 +0000 (21:42 +0000)
DominatorTreeBase::Split.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72810 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Analysis/Dominators.h
test/CodeGen/Generic/2009-06-03-UnreachableSplitPad.ll [new file with mode: 0644]

index b405f5b71ed7c23c570a7cce8bdea73b730f882d..347e239d8ea7fc943102e3414e5c35be5d72b9f9 100644 (file)
@@ -270,12 +270,17 @@ protected:
         NewBBIDom = PredBlocks[i];
         break;
       }
-    assert(i != PredBlocks.size() && "No reachable preds?");
+
+    // It's possible that none of the predecessors of NewBB are reachable;
+    // in that case, NewBB itself is unreachable, so nothing needs to be
+    // changed.
+    if (!NewBBIDom)
+      return;
+
     for (i = i + 1; i < PredBlocks.size(); ++i) {
       if (DT.isReachableFromEntry(PredBlocks[i]))
         NewBBIDom = DT.findNearestCommonDominator(NewBBIDom, PredBlocks[i]);
     }
-    assert(NewBBIDom && "No immediate dominator found??");
 
     // Create the new dominator tree node... and set the idom of NewBB.
     DomTreeNodeBase<NodeT> *NewBBNode = DT.addNewBlock(NewBB, NewBBIDom);
diff --git a/test/CodeGen/Generic/2009-06-03-UnreachableSplitPad.ll b/test/CodeGen/Generic/2009-06-03-UnreachableSplitPad.ll
new file mode 100644 (file)
index 0000000..59e7d0c
--- /dev/null
@@ -0,0 +1,15 @@
+; RUN: llvm-as < %s | llc
+; PR4317
+
+declare i32 @b()
+
+define void @a() {
+entry:
+  ret void
+
+dummy:
+  invoke i32 @b() to label %reg unwind label %reg
+
+reg:
+  ret void
+}