<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 17, 2013 at 3:10 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@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><div>On 5/17/2013 3:42 PM, Sean Silva 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">
Unrelated to the patch, but can these be `static`?:<br>
<br>
+const uint8_t X86_64PLTAtom::_<u></u>defaultContent[16] = {<br>
+  0xff, 0x25, 0x00, 0x00, 0x00, 0x00, // jmpq *gotatom(%rip)<br>
+  0x68, 0x00, 0x00, 0x00, 0x00,       // pushq reloc-index<br>
+  0xe9, 0x00, 0x00, 0x00, 0x00        // jmpq plt[-1]<br>
+};<br>
<br>
-- Sean Silva<br>
<br>
</blockquote></div></div>
Thanks for your reply, Sean.<br>
<br>
These cannot be static, as these are entries for each PLT entry. Each PLT entry may have a different relocation index, thats pointing to the .rela.dyn entry.<br></blockquote><div><br></div><div>Sorry, I think I miscommunicated. What I meant is whether these small _defaultContent arrays can be put in an anonymous namespace or otherwise internalized so that they do not appear as symbols in the final lld binary.</div>

<div><br></div><div>-- Sean Silva</div></div></div></div>