[PATCH] D18706: [DWARF] Force a linkage_name on an inlined subprogram's abstract origin

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 15:03:22 PDT 2016


dblaikie added inline comments.

================
Comment at: lib/CodeGen/AsmPrinter/DwarfUnit.cpp:1173-1174
@@ -1172,2 +1172,4 @@
   if (DeclLinkageName.empty())
-    addLinkageName(SPDie, LinkageName);
+    // Always emit it for abstract subprograms.
+    if (DD->useAllLinkageNames() || DU->getAbstractSPDies().lookup(SP))
+      addLinkageName(SPDie, LinkageName);
----------------
Just roll these conditions into the previous 'if', perhaps?


http://reviews.llvm.org/D18706





More information about the llvm-commits mailing list