r222373 - Fix a temporary MDNode leak.

Frederic Riss friss at apple.com
Wed Nov 19 10:53:46 PST 2014

Author: friss
Date: Wed Nov 19 12:53:46 2014
New Revision: 222373

URL: http://llvm.org/viewvc/llvm-project?rev=222373&view=rev
Fix a temporary MDNode leak.

While emitting debug information for function forward decalrations, we
create DISubprogram objects that aran't stored in the AllSubprograms
list, and thus won't get finalized by the DIBuilder. During the DIBuilder
finalize(), the temporary MDNode allocated for the DISubprogram
Variables field gets RAUWd with a non temporary DIArray. For the forward
declarations, simply delete that temporary node before we delete the
parent node, so that it doesn't leak.


Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDebugInfo.cpp?rev=222373&r1=222372&r2=222373&view=diff
--- cfe/trunk/lib/CodeGen/CGDebugInfo.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGDebugInfo.cpp Wed Nov 19 12:53:46 2014
@@ -3398,6 +3398,13 @@ void CGDebugInfo::finalize() {
       VH = p.second;
       VH = it->second;
+    // Functions have a fake temporary MDNode operand that is supposed
+    // to get RAUWed upon DIBuilder finalization. Do not leak these
+    // nodes for the temporary functions we are about to delete.
+    if (FwdDecl.isSubprogram())
+      llvm::MDNode::deleteTemporary(llvm::DISubprogram(FwdDecl).getVariablesNodes());

More information about the cfe-commits mailing list