<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 5, 2016 at 1:49 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 5 April 2016 at 16:46, Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> wrote:<br>
> tejohnson added inline comments.<br>
><br>
> ================<br>
> Comment at: tools/gold/gold-plugin.cpp:779<br>
> @@ +778,3 @@<br>
> +      // FIXME: For can preempted Comdat members be dropped from their<br>
> +      // Comdat so they can use available_externally linkage?<br>
> +      if (options::thinlto && !GV->hasComdat() &&<br>
> ----------------<br>
> rafael wrote:<br>
>> It is OK to have available_externally to a comdat member, no?<br>
>><br>
>> I mean, it is equivalent to a declaration, and we can have a declaration to a comdat member.<br>
> You can't have a declaration or available_externally in a comdat. >From Verifier.cpp:<br>
><br>
>   if (GV.isDeclarationForLinker())<br>
>     Assert(!GV.hasComdat(), "Declaration may not be in a Comdat!", &GV);<br>
<br>
</span>Yes, what I mean is that is valid to have a declaration that points to<br>
something in a  comdat. That is what you get in c++ with an explicit<br>
template instantiation declaration.<br></blockquote><div><br></div><div>Oh sure, that's what I'm referring to in the FIXME. I could do that in this patch pretty easily I suppose, let me do that.</div><div><br></div><div>Thanks,</div><div>Teresa</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>