diff options
| author | Victor Hernandez <vhernandez@apple.com> | 2010-01-20 06:57:02 +0000 |
|---|---|---|
| committer | Victor Hernandez <vhernandez@apple.com> | 2010-01-20 06:57:02 +0000 |
| commit | 6c50c06e70c111d380974ef6ef5064d68b70aac1 (patch) | |
| tree | a57804ec1cd3c7a083bb654c470d56ed5609ab22 /llvm/lib/VMCore/Verifier.cpp | |
| parent | f2462407ee02b2429eeeba72063a16aac2e3ed9d (diff) | |
| download | bcm5719-llvm-6c50c06e70c111d380974ef6ef5064d68b70aac1.tar.gz bcm5719-llvm-6c50c06e70c111d380974ef6ef5064d68b70aac1.zip | |
Fix/strengthen verification of llvm.dbg.declare
llvm-svn: 93990
Diffstat (limited to 'llvm/lib/VMCore/Verifier.cpp')
| -rw-r--r-- | llvm/lib/VMCore/Verifier.cpp | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/llvm/lib/VMCore/Verifier.cpp b/llvm/lib/VMCore/Verifier.cpp index 91517ea7f6a..6e473489956 100644 --- a/llvm/lib/VMCore/Verifier.cpp +++ b/llvm/lib/VMCore/Verifier.cpp @@ -1581,7 +1581,7 @@ void Verifier::visitIntrinsicFunctionCall(Intrinsic::ID ID, CallInst &CI) { // If the intrinsic takes MDNode arguments, verify that they are either global // or are local to *this* function. - for (unsigned i = 0, e = CI.getNumOperands(); i != e; ++i) + for (unsigned i = 1, e = CI.getNumOperands(); i != e; ++i) if (MDNode *MD = dyn_cast<MDNode>(CI.getOperand(i))) { if (!MD->isFunctionLocal()) continue; SmallPtrSet<MDNode *, 32> Visited; @@ -1591,12 +1591,17 @@ void Verifier::visitIntrinsicFunctionCall(Intrinsic::ID ID, CallInst &CI) { switch (ID) { default: break; - case Intrinsic::dbg_declare: // llvm.dbg.declare - if (MDNode *MD = dyn_cast<MDNode>(CI.getOperand(1))) + case Intrinsic::dbg_declare: { // llvm.dbg.declare + Assert1(CI.getOperand(1) && isa<MDNode>(CI.getOperand(1)), + "invalid llvm.dbg.declare intrinsic call", &CI); + MDNode *MD = cast<MDNode>(CI.getOperand(1)); + Assert1(MD->getNumOperands() == 1, + "invalid llvm.dbg.declare intrinsic call", &CI); + if (MD->getOperand(0)) if (Constant *C = dyn_cast<Constant>(MD->getOperand(0))) Assert1(C && !isa<ConstantPointerNull>(C), "invalid llvm.dbg.declare intrinsic call", &CI); - break; + } break; case Intrinsic::memcpy: case Intrinsic::memmove: case Intrinsic::memset: |
