<p dir="ltr"><br>
On Mar 7, 2016 1:43 PM, "Evgenii Stepanov" <<a href="mailto:eugenis@google.com">eugenis@google.com</a>> wrote:<br>
><br>
> 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>
<p dir="ltr">If you want, for now just create a new test with</p>
<p dir="ltr">REQUIRES: NEWGOLD</p>
<p dir="ltr">It will show up as unsupported everywhere, but should let me review it locally. <br></p>
<p dir="ltr">><br>
> On Mar 7, 2016 6:58 AM, "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> 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">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>
</p>