Also worth noting is that the lit site config files only matter for 3-4 tests, which I don’t believe are run anywhere.  I plan to overhaul this whole system in the coming weeks/months, so personally I don’t mind much if it breaks <br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 23, 2017 at 6:17 PM Zachary Turner via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zturner added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D39215#904578" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39215#904578</a>, @labath wrote:<br>
<br>
> I've played around with this, but I couldn't get the `lit/<a href="http://lit.site.cfg.in" rel="noreferrer" target="_blank">lit.site.cfg.in</a>` file to see the expanded values of the `$<TARGET_FILE:clang>` generator expression (the reason it works now is because the file is special-casing LLDB_TEST_CLANG=True). Currently, I do not see how to work around that.<br>
><br>
> So, I propose to separate the functional change from the cleanup, and leave the cleanup part to someone with more cmake-fu. :P<br>
<br>
<br>
I think we just need to do:<br>
<br>
  set(DEFAULT_LLDB_TEST_COMPILER "$<TARGET_FILE:clang>")<br>
<br>
  set(LLDB_TEST_COMPILER "${DEFAULT_LLDB_TEST_COMPILER}" CACHE PATH "Path to clang executable used to run LLDB tests")<br>
<br>
Does this not work?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D39215" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39215</a><br>
<br>
<br>
<br>
</blockquote></div>