<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Feb 3, 2015 at 6:22 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.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 2/3/2015 7:33 PM, Michael Spencer wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sun, Feb 1, 2015 at 10:31 PM, Shankar Easwaran<br>
<<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 2/2/2015 12:24 AM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sun, Feb 1, 2015 at 10:18 PM, Shankar Easwaran<br>
<<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The reason was more that the Context should own what is the default<br>
search<br>
directory. The ELFLinkingContext is using the baseTriple here not the<br>
real<br>
triple.<br>
<br>
</blockquote>
Could you please elaborate why you think it should?<br>
<br>
My point was that this patch scattered the logic that used to be in one<br>
function and also added a new virtual public function that we used to not<br>
need. It doesn't seem like a good tradeoff.<br>
</blockquote>
Are you thinking that the default search path for all the ELF<br>
targets(aarch64, armv7, etc) be housed in the Driver ?  I would disagree.<br>
</blockquote>
They should. It's the drivers job to interact with the environment to<br>
figure out how to drive the linker. This is also how it works in<br>
clang.<br>
</blockquote></span>
I am looking at Toolchains.cpp inside clang, I think they do it since there is not a context thats used to drive the process.<br>
<br>
If this is what we want to follow in lld, I will revert the patch.</blockquote><div><br></div><div>Yes, please. </div></div><br></div></div>