<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 11, 2013 at 10:52 AM, Krzysztof Parzyszek <span dir="ltr"><<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On 2/5/2013 5:44 PM, Krzysztof Parzyszek wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On 2/1/2013 11:27 AM, Krzysztof Parzyszek wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Here's a patch that includes the changes you asked for.  I hope I didn't<br>
miss anything.  Let me know what you think.<br>
<br>
</blockquote>
<br>
Hello... hello... hello...<br>
Is there anybody in there?  :)<br>
</blockquote>
<br></div>
Hi Eric,<br>
I haven't heard from you about this updated patch yet.  Please take a look.  It's attached to my email from Feb 1.</blockquote><div><br></div><div style>Sorry, was hitting my review limit pretty fast last week. Here you go:</div>
<div style><br></div><div style> +++ b/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp</div><div>@@ -27,6 +27,7 @@</div><div> #include "llvm/Target/Mangler.h"</div><div> #include "llvm/Target/TargetFrameLowering.h"</div>
<div> #include "llvm/Target/TargetMachine.h"</div><div>+#include "llvm/Target/TargetLoweringObjectFile.h"</div><div><br></div><div style>Unnecessary include.</div><div style><br></div><div style><div>+  /// Globals - A map of globally visible named entities for this unit.</div>
<div>+  ///</div><div>+  StringMap<DIE*> Globals;</div><div><br></div><div style>GlobalNames for consistency?</div><div style><br></div><div style><div>+  if (GenerateDwarfPubNamesSection) {</div><div>+    // Check if any of the CUs has any globals.</div>
<div>+    bool HaveGlobals = false;</div><div>+    typedef DenseMap<const MDNode*, CompileUnit*> CUMapType;</div><div>+    for (CUMapType::iterator I = CUMap.begin(), E = CUMap.end(); I != E; ++I) {</div><div>+      CompileUnit *TheCU = I->second;</div>
<div>+      if (!TheCU->getGlobals().empty()) {</div><div>+        HaveGlobals = true;</div><div>+        break;</div><div>+      }</div><div>+    }</div><div>+    if (HaveGlobals)</div><div>+      emitSectionSym(Asm, TLOF.getDwarfPubNamesSection());</div>
<div>+  }</div><div><br></div><div style>I appreciate the attempt to minimize whether or not we emit the section at all, but feel free to just check if we're generating and emit the section sym.</div><div style><br></div>
<div style>Missing a testcase it appears as well? Once you've got a testcase and have fixed the above feel free to commit if you have commit privs, otherwise send me something. It'd be nice to split out the section patches from the dwarfdump and emission patch. Try to make sure in the testcase that you cover a good set of what should be in the pubnames section and include the .c file in the comments of the testcase so we can duplicate it via clang's IR emission.</div>
<div style><br></div><div style>Thanks!</div><div style><br></div><div style>-eric</div><div style><br></div></div></div></div></div></div>