<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Dec 12, 2017 at 3:01 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">Don Hinton <<a href="mailto:hintonda@gmail.com">hintonda@gmail.com</a>> writes:<br>
> On Tue, Dec 12, 2017 at 2:04 PM, Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>><br>
> wrote:<br>
><br>
>> Alex Bradbury <<a href="mailto:asb@lowrisc.org">asb@lowrisc.org</a>> writes:<br>
>> > On 12 December 2017 at 13:32, Don Hinton via llvm-commits<br>
>> > <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> >> Yes, for this to work, we'd need to pass LLVM_EXPERIMENTAL_TARGETS_TO_<br>
>> BUILD<br>
>> >> as well.<br>
>> >><br>
>> >> However, since Native is the only target actually used, I'd lean toward<br>
>> >> reverting this change and perhaps adding a comment.<br>
>><br>
>> This isn't true, Native is not used (in fact, most of my builds don't<br>
>> have it or the equivalent, which is part of why I made this change). It<br>
>> would make just as much sense to pass in an empty target list.<br>
>><br>
><br>
> "Native" is never used literally, it's just a wildcard placeholder.<br>
><br>
> string(REPLACE "Native" ${LLVM_NATIVE_ARCH}<br>
>   LLVM_TARGETS_TO_BUILD "${LLVM_TARGETS_TO_BUILD}")<br>
> list(REMOVE_DUPLICATES LLVM_TARGETS_TO_BUILD)<br>
><br>
> With LLVM_NATIVE_ARCH set in config-ix.cmake.<br>
<br>
</span>Yes, but the host tools do not need the native arch in the targets in<br>
order to work.<br>
<br>
If I build on X86 and set LLVM_TARGETS_TO_BUILD=AArch64, then the host<br>
tools should have LLVM_TARGETS_TO_BUILD="<wbr>AArch64" (which they do<br>
currently), not "X86", and not "AArch64;X86". Native expands to "X86" in<br>
this case, and that doesn't actually make sense for the build.<br></blockquote><div><br></div><div>Since we're only talking about the host tools built in the NATIVE subdirectory -- currently just (clang|llvm)-tblgen -- and invoked on the host, I'm not really sure it matters.  But perhaps it will someday.<br></div><div><br></div><div>So, passing <span style="font-size:12.8px">LLVM_EXPERIMENTAL_TARGETS_TO_</span><wbr style="font-size:12.8px"><span style="font-size:12.8px">BUILD seems like the best idea at this point.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">I can make this change later today if you want.</span></div></div><br></div></div>