diff options
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/IR/LLVMContextImpl.cpp | 18 | ||||
-rw-r--r-- | llvm/lib/IR/Metadata.cpp | 12 |
2 files changed, 8 insertions, 22 deletions
diff --git a/llvm/lib/IR/LLVMContextImpl.cpp b/llvm/lib/IR/LLVMContextImpl.cpp index a9268b46488..c78b76021da 100644 --- a/llvm/lib/IR/LLVMContextImpl.cpp +++ b/llvm/lib/IR/LLVMContextImpl.cpp @@ -135,18 +135,16 @@ LLVMContextImpl::~LLVMContextImpl() { for (auto &Pair : ValuesAsMetadata) delete Pair.second; - // Destroy MDNodes. ~MDNode can move and remove nodes between the MDTuples - // and the DistinctMDNodes sets, so copy the values out first. - SmallVector<UniquableMDNode *, 8> Uniquables; - Uniquables.reserve(MDTuples.size() + DistinctMDNodes.size()); - Uniquables.append(MDTuples.begin(), MDTuples.end()); - Uniquables.append(DistinctMDNodes.begin(), DistinctMDNodes.end()); - for (UniquableMDNode *I : Uniquables) + // Destroy MDNodes. + for (auto *I : DistinctMDNodes) I->dropAllReferences(); - for (UniquableMDNode *I : Uniquables) + for (auto *I : MDTuples) + I->dropAllReferences(); + + for (UniquableMDNode *I : DistinctMDNodes) delete cast<MDTuple>(I); - assert(MDTuples.empty() && DistinctMDNodes.empty() && - "Destroying all MDNodes didn't empty the Context's sets."); + for (MDTuple *I : MDTuples) + delete I; // Destroy MDStrings. MDStringCache.clear(); diff --git a/llvm/lib/IR/Metadata.cpp b/llvm/lib/IR/Metadata.cpp index 9088a75156e..fa1f302568f 100644 --- a/llvm/lib/IR/Metadata.cpp +++ b/llvm/lib/IR/Metadata.cpp @@ -427,13 +427,6 @@ UniquableMDNode::UniquableMDNode(LLVMContext &C, unsigned ID, SubclassData32 = NumUnresolved; } -UniquableMDNode::~UniquableMDNode() { - if (isStoredDistinctInContext()) - getContext().pImpl->DistinctMDNodes.erase(this); - - dropAllReferences(); -} - void UniquableMDNode::resolve() { assert(!isResolved() && "Expected this to be unresolved"); @@ -481,11 +474,6 @@ void UniquableMDNode::resolveCycles() { } } -MDTuple::~MDTuple() { - if (!isStoredDistinctInContext()) - getContext().pImpl->MDTuples.erase(this); -} - void MDTuple::recalculateHash() { setHash(hash_combine_range(op_begin(), op_end())); #ifndef NDEBUG |