From: Chris Lattner <sabre@nondot.org>
Date: Wed, 14 Jan 2004 04:25:59 +0000 (+0000)
Subject: Tighten up verifier checks.  The result of an invoke instruction only
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=30768ac3c2395d5b2047ec84e1b8f4330ba2658f;p=oota-llvm.git

Tighten up verifier checks.  The result of an invoke instruction only
dominates the normal destination, not the exceptional dest (ie, the result
of a call is undefined on an exception)


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

diff --git a/lib/VMCore/Verifier.cpp b/lib/VMCore/Verifier.cpp
index c2e3d9a6fee..a9bcbd5f8f2 100644
--- a/lib/VMCore/Verifier.cpp
+++ b/lib/VMCore/Verifier.cpp
@@ -506,18 +506,24 @@ void Verifier::visitInstruction(Instruction &I) {
               "Cannot take the address of an intrinsic!", &I);
 
     else if (Instruction *Op = dyn_cast<Instruction>(I.getOperand(i))) {
+      BasicBlock *OpBlock = Op->getParent();
+      // Invoke results are only usable in the normal destination, not in the
+      // exceptional destination.
+      if (InvokeInst *II = dyn_cast<InvokeInst>(Op))
+        OpBlock = II->getNormalDest();
+
       // Check that a definition dominates all of its uses.
       //
       if (!isa<PHINode>(I)) {
         // Definition must dominate use unless use is unreachable!
-        Assert2(DS->dominates(Op->getParent(), BB) ||
+        Assert2(DS->dominates(OpBlock, BB) ||
                 !DS->dominates(&BB->getParent()->getEntryBlock(), BB),
                 "Instruction does not dominate all uses!", Op, &I);
       } else {
         // PHI nodes are more difficult than other nodes because they actually
         // "use" the value in the predecessor basic blocks they correspond to.
         BasicBlock *PredBB = cast<BasicBlock>(I.getOperand(i+1));
-        Assert2(DS->dominates(Op->getParent(), PredBB) ||
+        Assert2(DS->dominates(OpBlock, PredBB) ||
                 !DS->dominates(&BB->getParent()->getEntryBlock(), PredBB),
                 "Instruction does not dominate all uses!", Op, &I);
       }