<div dir="ltr">No worries at all. If you need anything let me know!<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 22, 2013 at 4:32 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Eric,<br>
<br>
Thanks!  I'll make the changes and I'll send you another patch.  I'm not yet familiar with llvm-dwarfdump, so it may take a few days.<span class="HOEnZb"><font color="#888888"><br>
<br>
-Krzysztof</font></span><div class="im"><br>
<br>
<br>
On 1/22/2013 6:19 PM, Eric Christopher wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi Krzysztof,<br>
<br>
Sorry for the delay, here's the review you wanted :)<br>
<br>
+static cl::opt<bool><br>
GenerateDwarfPubNamesSection("<u></u>generate-dwarf-pubnames",<br>
+     cl::Hidden, cl::ZeroOrMore, cl::init(false),<br>
+     cl::desc("Generate DWARF pubnames section"));<br>
<br>
I'd prefer this be a debug info option and not a TLOF option, also you<br>
can just check the file static then as well.<br>
<br>
Not sure if there's a point in emitting the section in emitDebugPubNames<br>
(can we make it emitDebugPubnames since we're adding it back?) if<br>
Globals.size() == 0 either.<br>
<br>
+  if (const MCSection *PNS = TLOF.getDwarfPubNamesSection()<u></u>)<br>
+    emitSectionSym(Asm, PNS);<br>
<br>
You'll probably want to just emit the sym if we're emitting the section,<br>
that said, since we pretty much don't check for any of the other<br>
sections no need for the check here. (and SwitchSection has an assert in<br>
it).<br>
<br>
If you wouldn't mind fleshing out the pubnames testcase to make sure a<br>
wider variety of items are emitted into the pubnames section, also<br>
llvm-dwarfdump needs to parse the section for the testcase.<br>
<br>
Thanks!<br>
<br>
-eric<br>
<br>
<br>
On Mon, Jan 14, 2013 at 11:20 AM, Krzysztof Parzyszek<br></div><div class="im">
<<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.org</a> <mailto:<a href="mailto:kparzysz@codeaurora.org" target="_blank">kparzysz@codeaurora.<u></u>org</a>>> wrote:<br>
<br>
    This is a patch to restore the generation of the pubnames section in<br>
    the DWARF information.  The code is disabled by default and can be<br>
    enabled with -generate-dwarf-pubnames.<br>
<br>
    Please review.<br>
<br>
    Thank you,<br>
    -Krzysztof<br>
<br>
    --<br>
    Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,<br>
    hosted by The Linux Foundation<br>
<br>
<br>
</div></blockquote><div class="HOEnZb"><div class="h5">
<br>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation<br>
</div></div></blockquote></div><br></div>