<p dir="ltr">Yes. It only matters with tot gold, when the plugin has the ability to drop inputs. I guess I'll have to figure out a way to check for the linker version in tests.</p>
<div class="gmail_quote">On Mar 7, 2016 6:58 AM, "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The test passes with the patch reverted.<br>
<br>
<br>
On 4 March 2016 at 19:32, Evgeniy Stepanov <<a href="mailto:eugenis@google.com">eugenis@google.com</a>> wrote:<br>
> eugenis created this revision.<br>
> eugenis added a reviewer: rafael.<br>
> eugenis added a subscriber: llvm-commits.<br>
> eugenis set the repository for this revision to rL LLVM.<br>
><br>
> LLVM Gold plugin decides which instance of a common symbol it wants<br>
> based on the symbol size in claim_file_hook. If the file that<br>
> contains the chosen instance is later dropped from the link, we end<br>
> up with an undefined reference.<br>
><br>
> This change delays this decision until the set of the included files<br>
> is known.<br>
><br>
> Repository:<br>
>   rL LLVM<br>
><br>
> <a href="http://reviews.llvm.org/D17900" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17900</a><br>
><br>
> Files:<br>
>   test/tools/gold/X86/Inputs/common.ll<br>
>   test/tools/gold/X86/Inputs/common2.ll<br>
>   test/tools/gold/X86/common.ll<br>
>   tools/gold/gold-plugin.cpp<br>
><br>
</blockquote></div>