aha, DAE does have to think about PHI nodes. Many thanks to "Dr Evil" (aka Duncan)

for pointing this out :)

llvm-svn: 67212

1 files changed, 3 insertions, 3 deletions

diff --git a/llvm/lib/Transforms/IPO/DeadArgumentElimination.cpp b/llvm/lib/Transforms/IPO/DeadArgumentElimination.cpp
index ab8df698457..666db7e8d74 100644
--- a/llvm/lib/Transforms/IPO/DeadArgumentElimination.cpp
+++ b/llvm/lib/Transforms/IPO/DeadArgumentElimination.cpp
@@ -803,9 +803,9 @@ bool DAE::RemoveDeadStuffFromFunction(Function *F) {
                " must have been a struct!");
         Instruction *InsertPt = Call;
         if (InvokeInst *II = dyn_cast<InvokeInst>(Call)) {
-          InsertPt = II->getNormalDest()->begin();
-          assert(!isa<PHINode>(InsertPt) &&
-                "Can't have a use of the invoke value if the edge is critical");
+          BasicBlock::iterator IP = II->getNormalDest()->begin();
+          while (isa<PHINode>(IP)) ++IP;
+          InsertPt = IP;
         }
           
         // We used to return a struct. Instead of doing smart stuff with all the