IR: Move call to dropAllReferences() to MDNode subclasses
authorDuncan P. N. Exon Smith <dexonsmith@apple.com>
Wed, 10 Dec 2014 01:45:04 +0000 (01:45 +0000)
committerDuncan P. N. Exon Smith <dexonsmith@apple.com>
Wed, 10 Dec 2014 01:45:04 +0000 (01:45 +0000)
Don't call `dropAllReferences()` from `MDNode::~MDNode()`, call it
directly from `~MDNodeFwdDecl()` and `~GenericMDNode()`.

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

include/llvm/IR/Metadata.h
lib/IR/Metadata.cpp

index 573e49fbb06954cff94d35f0cd0be83129008153..5e1b294fa1ffc3b8f7d3138f70ff62eee479fe26 100644 (file)
@@ -598,7 +598,7 @@ protected:
   }
 
   MDNode(LLVMContext &Context, unsigned ID, ArrayRef<Metadata *> MDs);
-  ~MDNode() { dropAllReferences(); }
+  ~MDNode() {}
 
   void dropAllReferences();
   void storeDistinctInContext();
@@ -766,7 +766,7 @@ class MDNodeFwdDecl : public MDNode, ReplaceableMetadataImpl {
 
   MDNodeFwdDecl(LLVMContext &C, ArrayRef<Metadata *> Vals)
       : MDNode(C, MDNodeFwdDeclKind, Vals) {}
-  ~MDNodeFwdDecl() {}
+  ~MDNodeFwdDecl() { dropAllReferences(); }
 
 public:
   static bool classof(const Metadata *MD) {
index 0f748495261bfd0434b8934149289d1e679d1842..506284fa87aef42305098faff2a314f11112025b 100644 (file)
@@ -428,6 +428,7 @@ GenericMDNode::~GenericMDNode() {
     pImpl->NonUniquedMDNodes.erase(this);
   else
     pImpl->MDNodeSet.erase(this);
+  dropAllReferences();
 }
 
 void GenericMDNode::resolve() {