diff options
author | Eric Christopher <echristo@gmail.com> | 2013-07-24 01:51:12 +0000 |
---|---|---|
committer | Eric Christopher <echristo@gmail.com> | 2013-07-24 01:51:12 +0000 |
commit | e6f774e7ef1a11af39a6565465355db905f59d81 (patch) | |
tree | 99a57c047fa5ddca0b24ae3e990c02d5a0358809 /lib | |
parent | 167153080bb192dc7ecb58ea41093c85a7d6a84f (diff) | |
download | llvm-e6f774e7ef1a11af39a6565465355db905f59d81.tar.gz llvm-e6f774e7ef1a11af39a6565465355db905f59d81.tar.bz2 llvm-e6f774e7ef1a11af39a6565465355db905f59d81.tar.xz |
Replace with a "null" RAUW with an assert since we'd actually like
to assume we're replacing. Clarify comments.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@187018 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/IR/DebugInfo.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/lib/IR/DebugInfo.cpp b/lib/IR/DebugInfo.cpp index f0593450ee..c7cdb80a6a 100644 --- a/lib/IR/DebugInfo.cpp +++ b/lib/IR/DebugInfo.cpp @@ -327,11 +327,11 @@ unsigned DIArray::getNumElements() const { return DbgNode->getNumOperands(); } -/// replaceAllUsesWith - Replace all uses of debug info referenced by -/// this descriptor. +/// replaceAllUsesWith - Replace all uses of the MDNode used by this +/// type with the one in the passed descriptor. void DIType::replaceAllUsesWith(DIDescriptor &D) { - if (!DbgNode) - return; + + assert(DbgNode && "Trying to replace an unverified type!"); // Since we use a TrackingVH for the node, its easy for clients to manufacture // legitimate situations where they want to replaceAllUsesWith() on something @@ -347,11 +347,11 @@ void DIType::replaceAllUsesWith(DIDescriptor &D) { } } -/// replaceAllUsesWith - Replace all uses of debug info referenced by -/// this descriptor. +/// replaceAllUsesWith - Replace all uses of the MDNode used by this +/// type with the one in D. void DIType::replaceAllUsesWith(MDNode *D) { - if (!DbgNode) - return; + + assert(DbgNode && "Trying to replace an unverified type!"); // Since we use a TrackingVH for the node, its easy for clients to manufacture // legitimate situations where they want to replaceAllUsesWith() on something |