<div dir="ltr">On Mon, Apr 6, 2015 at 6:22 PM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 6 April 2015 at 17:23, Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br>
> On Mon, Apr 06, 2015 at 02:15:58PM -0700, Rui Ueyama wrote:<br>
>> Anyways, my point is that the change to the linker for this feature might<br>
>> be trivial, so it'd be easy to understand what is done here. But it would<br>
>> not help readers of the code understand why we want to do that. So I wanted<br>
>> to add a comment on that (a brief comment should suffice).<br>
><br>
> I fully agree on that. First time I saw the option was like "WTF do they<br>
> need this?"<br>
<br>
</span>Good point. Do you think we should go further and either just use<br>
DT_RUNPATH when we know the system's rtld supports it, or set both<br>
DT_RUNPATH and DT_RPATH in all cases?<br></blockquote><div><br></div><div>No, setting both would mean that you still have DT_RPATH, which means that LD_LIBRARY_PATH is no longer honored.  I think that would be detrimental to the use of the flag in the first place.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The "broken" DT_RPATH behaviour is as specified in gabi41.pdf, but I'm<br>
not sure that anyone really wants that. It seems a bit unfortunate to<br>
have the sensible behaviour hidden behind an option.<br></blockquote><div><br></div><div>I suppose we could add yet another flag to change to the older behavior if needed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>