<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 11, 2013 at 2:15 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"><div class="im">On 2/11/2013 1:21 PM, Eric Christopher wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Missing a testcase it appears as well? Once you've got a testcase and<br>
have fixed the above feel free to commit if you have commit privs,<br>
otherwise send me something. It'd be nice to split out the section<br>
patches from the dwarfdump and emission patch. Try to make sure in the<br>
testcase that you cover a good set of what should be in the pubnames<br>
section and include the .c file in the comments of the testcase so we<br>
can duplicate it via clang's IR emission.<br>
</blockquote>
<br></div>
Interestingly, I had a testcase like that, and it did include the original source in the comments.  It just somehow didn't make it to my local commiy.  One thing I wasn't sure about it is that I put it in the X86-specific test directory.  Here's a snippet explaining why:<br>

<br>
; Test this on x86 because it is known to use ELF.  This way we can use<br>
; the actual section name to make sure that the symbol names are defined<br>
; inside of it.<br>
;<br>
; CHECK: pubnames_begin0<br>
; This test will fail if these names are emitted in a different order.<br>
; CHECK: global_namespace_variable<br>
; CHECK: global_namespace_function<br>
; CHECK: static_member_function<br>
; CHECK: global_variable<br>
; CHECK: global_function<br>
; CHECK: member_function<br>
; CHECK: pubnames_end0<br>
<br>
<br>
There is also the complication related to the order of the names.  Are you ok with this, or do you have another suggestion?<br></blockquote><div><br></div><div style>Mmm.. I'd prefer you use llvm-dwarfdump after making an object file with it. Also, I'd hope that whatever way you're iterating over the list is stable. I didn't think to check that, so if it isn't please make sure it is. I don't think there's a reason to make this x86 only since all of the strings for the pubnames section have to be actual strings and so won't be subject to the unimplemented relocation problem.</div>
<div style><br></div><div style>-eric</div></div></div></div>