labath added a comment.

In https://reviews.llvm.org/D39215#904677, @zturner wrote:

> Ok the issue is that you cant use CMake generator expressions in this way.  
> This should work though:
>
>   if (TARGET clang)
>     set(LLDB_DEFAULT_TEST_COMPILER 
> "${LLVM_BINARY_DIR}/clang${CMAKE_EXECUTABLE_SUFFIX}")
>   else()
>     set(LLDB_DEFAULT_TEST_COMPILER "")
>   endif()


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.

> I think you might be able to do something like:
> 
>   if (TARGET clang)
>     get_property(LLDB_DEFAULT_TEST_COMPILER TARGET clang PROPERTY LOCATION)
>   endif()

This (although recommended by stack overflow), does not work on cmake 3.0+.


https://reviews.llvm.org/D39215



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to