If that works, let’s just do that.  I would much rather make definite forward progress towards the desired end state than support something nobody even uses<br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 23, 2017 at 8:24 PM Pavel Labath 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">labath added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D39215#904677" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39215#904677</a>, @zturner wrote:<br>
<br>
> Ok the issue is that you cant use CMake generator expressions in this way.  This should work though:<br>
><br>
>   if (TARGET clang)<br>
>     set(LLDB_DEFAULT_TEST_COMPILER "${LLVM_BINARY_DIR}/clang${CMAKE_EXECUTABLE_SUFFIX}")<br>
>   else()<br>
>     set(LLDB_DEFAULT_TEST_COMPILER "")<br>
>   endif()<br>
<br>
<br>
That will work, if you don't mind that it will break on multi-configuration builds such as visual studio (though I am not sure how/if regular lit tests work in that scenario). We don't run those tests anyway, and I do not think they are ready to be used, so I definitely won't mind.<br>
<br>
> I think you might be able to do something like:<br>
><br>
>   if (TARGET clang)<br>
>     get_property(LLDB_DEFAULT_TEST_COMPILER TARGET clang PROPERTY LOCATION)<br>
>   endif()<br>
<br>
This (although recommended by stack overflow), does not work on cmake 3.0+.<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>