<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 6, 2016 at 5:39 PM, Michael Spencer via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</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 Fri, May 6, 2016 at 2:20 PM, Joerg Sonnenberger via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> On Fri, May 06, 2016 at 03:36:10PM -0400, Rafael Espíndola via llvm-commits wrote:<br>
>> On 6 May 2016 at 14:27, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> > On 6 May 2016 at 14:23, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
>> >> Can we simply drop it? I'm wondering if other libc's or dynamic linkers use<br>
>> >> it.<br>
>> ><br>
>> > From what is written in the abi it looks like it would only be used by<br>
>> > self relocating programs (i.e.: ld.so).<br>
>> ><br>
>> > But those could do just what musl does.<br>
>> ><br>
>> > Let me give it a try.<br>
>><br>
>> <a href="http://reviews.llvm.org/D20028" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20028</a><br>
><br>
> Let's not produce nightmares for poor porters that have to hunt for the<br>
> obscure bugs it can create please.<br>
<br>
</span>I would like to reiterate this. We need to fully understand the<br>
implications of changes like this, not just look at a few examples and<br>
decide to ditch it.<br></blockquote><div><br></div><div>Yes, it is important to understand what we are doing. At the same time, it is important to not blindly replicate behaviors of the other linkers. Creating a general-purpose Unix linker from scratch gives us a historic chance to flush out bureaucrats and out-of-date behaviors that have been done over decades. I'd like to carefully examine one by one.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- Michael Spencer<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
>> As for the rest of this patch: I think I agree with Rui. The added<br>
>> value is really small. If going for safety you should really disable<br>
>> lazy loading or even just compile with -fno-plt.<br>
><br>
> I agree on this part though, it seems pointless complexity for an edge<br>
> case, that doesn't give much.<br>
><br>
> Joerg<br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>