<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">I guess I can fix it by restoring "_files[0]->kind() == File::kindSharedLibrary" to GnuLdInputGraph.h. I'll try to see if it works.</span><br style="font-family:arial,sans-serif;font-size:13px">

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, May 11, 2014 at 10:36 PM, Simon Atanasyan <span dir="ltr"><<a href="mailto:simon@atanasyan.com" target="_blank">simon@atanasyan.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">CC to the mail list and attach the patch directly to the message.<br>
<br>
On Mon, May 12, 2014 at 9:28 AM, Simon Atanasyan <<a href="mailto:simon@atanasyan.com">simon@atanasyan.com</a>> wrote:<br>
> It looks like this commit introduced the following regression. Suppose we have:<br>
><br>
>   - Object file `main.o` which refers symbol `RT1`<br>
>   - Object file `obj.o` defines symbol `RT1` and refers symbol `T1`<br>
>   - Shared library libso.so defines symbol `T1`<br>
><br>
> If we link these files using the following command we incorrectly get an error:<br>
><br>
>   % lld -flavor gnu -target mipsel -e T -o a.out main.o --start-group libso.so obj.o --end-group<br>
>   Undefined symbol: obj.o: T1<br>
><br>
> That is because we do not scan the `libso.so` once again to find the `T1` definition. The attached patch {F57334} has a reproduction test case.<br>
><br>
> <a href="http://reviews.llvm.org/rL205566" target="_blank">http://reviews.llvm.org/rL205566</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Simon Atanasyan<br>
</font></span></blockquote></div><br></div>