<div dir="ltr">Ok, that seems reasonable to me. Maybe add a FIXME comment unless it will be fixed imminently.<div>Teresa</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 23, 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="">>> > ================<br>
>> > Comment at: lib/Linker/LinkModules.cpp:567<br>
>> > @@ -567,2 +566,3 @@<br>
>> >      for (GlobalValue *GV2 : ComdatMembers[SC])<br>
>> > -      ValuesToLink.insert(GV2);<br>
>> > +      if (GV2->hasInternalLinkage())<br>
>> > +        ValuesToLink.insert(GV2);<br>
>> > ----------------<br>
>> > Why only internal members? If we have a comdat member in ValuesToLink (added in linkIfNeeded), then presumably we had selected that copy of the comdat and wouldn't we want all members?<br>
>><br>
>> The correct thing is to drop this entire loop. I indent to do that in<br>
>> a followup patch.<br>
><br>
><br>
> I see now why we need to add internal members to the ValuesToLink set here - when we selected the dest copy of that particular symbol but need to keep a local (will be renamed) copy of the source symbol for references, right?<br>
<br>
</span>I actually thing it is a bug, but would like to address that in a<br>
followup patch if possible.<br>
<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>