<div dir="ltr">Given the lack of response (they may be using different addresses with the changes to MIPS since that time) I intend to get the current patch reviewed as is. Keeping the default triple behaviour the same. Then do a follow up change to remove the default triple behaviour, which can be reverted more easily if needed.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 15 May 2020 at 06:58, Fangrui Song via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2020-05-14, Joerg Sonnenberger via cfe-dev wrote:<br>
>On Thu, May 14, 2020 at 03:11:23PM +0100, David Spickett wrote:<br>
>> > ...but I don't think we should ever be looking for<br>
>> > ${DEFAULT_TRIPLE}-gcc. That doesn't make sense to me at all.<br>
>><br>
>> IIt was added in <a href="https://reviews.llvm.org/D13340" rel="noreferrer" target="_blank">https://reviews.llvm.org/D13340</a>. Quoting a comment there:<br>
>> ""LLVMDefaultTargetTriple is the value specified with the CMake<br>
>> variable -DLLVM_DEFAULT_TARGET_TRIPLE= during configuration time. For<br>
>> example, using --target=mips64el-mti-linux will search for files prefixed<br>
>> with either mips64el-mti-linux-{as,ld} and mips-mti-linux-{as,ld} in our<br>
>> toolchain where we specify -DLLVM_DEFAULT_TARGET_TRIPLE=mips-mti-linux."<br>
>><br>
>> I guess that allows you to only package one copy of tools that can handle<br>
>> both targets.<br>
><br>
>That seems ..strangely specific. We certainly have cases where multiple<br>
>architecture variants are targetting a single binutils environment in<br>
>NetBSD, but those are all controlled by the appropiate flags to the<br>
>tools. The default target doesn't leak through and I don't think it ever<br>
>should. That completely defeats the goal of a universal toolchain.<br>
><br>
>Joerg<br>
<br>
I vote for not checking ${LLVM_DEFAULT_TARGET_TRIPLE}-gcc if --target is<br>
specified.<br>
<br>
CCed people from <a href="https://reviews.llvm.org/D13340" rel="noreferrer" target="_blank">https://reviews.llvm.org/D13340</a><br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>