<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 15, 2016 at 11:02 PM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">grimar added inline comments.<br>
<span class=""><br>
================<br>
Comment at: ELF/OutputSections.cpp:1319<br>
@@ -1318,2 +1318,3 @@<br>
</span><span class="">   Symbol *S = Body->symbol();<br>
   uint8_t Visibility = S->Visibility;<br>
</span><span class="">+  if (!Config->Relocatable)<br>
----------------<br>
</span><span class="">ruiu wrote:<br>
> I'm not sure if this would fix the problem entirely. My gut is that we want to keep the original symbol type if -r is specified. If that's the case, we want to add<br>
><br>
>   if (Config->Relocatable)<br>
>     return S->Binding;<br>
><br>
> here. But I don't know if that's correct.<br>
</span>gold allows GNU_UNIQUE->STB_GLOBAL transformation for relocatable output,<br>
so I tried to be consistent with it here.<br>
That is the only difference with your suggestion I think. And looks gnu ld does not support<br>
--no-gnu-unique at all, so probably we can go with any solution here.<br></blockquote><div><br></div><div>Do you think what GNU gold does makes sense? </div></div></div></div>