<div dir="ltr"><div style>Hi Krzysztof,</div><div><br></div><div style>Sorry for the delay, here's the review you wanted :)</div><div><br></div><div>+static cl::opt<bool> GenerateDwarfPubNamesSection("generate-dwarf-pubnames",</div>
<div>+     cl::Hidden, cl::ZeroOrMore, cl::init(false),</div><div>+     cl::desc("Generate DWARF pubnames section"));</div><div><br></div><div style>I'd prefer this be a debug info option and not a TLOF option, also you can just check the file static then as well.</div>
<div style><br></div><div style>Not sure if there's a point in emitting the section in emitDebugPubNames (can we make it emitDebugPubnames since we're adding it back?) if Globals.size() == 0 either.</div><div style>
<br></div><div style><div>+  if (const MCSection *PNS = TLOF.getDwarfPubNamesSection())</div><div>+    emitSectionSym(Asm, PNS);</div></div><div style><br></div><div style>You'll probably want to just emit the sym if we're emitting the section, that said, since we pretty much don't check for any of the other sections no need for the check here. (and SwitchSection has an assert in it).</div>
<div style><br></div><div style><div>If you wouldn't mind fleshing out the pubnames testcase to make sure a wider variety of items are emitted into the pubnames section, also llvm-dwarfdump needs to parse the section for the testcase.</div>
<div><br></div><div style>Thanks!</div><div style><br></div><div style>-eric</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 14, 2013 at 11:20 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is a patch to restore the generation of the pubnames section in the DWARF information.  The code is disabled by default and can be enabled with -generate-dwarf-pubnames.<br>

<br>
Please review.<br>
<br>
Thank you,<br>
-Krzysztof<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation<br>
</font></span></blockquote></div><br></div>