[Lldb-commits] [PATCH] D39215: Default to using in-tree clang for building test executables

[Zachary Turner via Phabricator via lldb-commits]

zturner added a comment.

There is already a CMake variable called `LLDB_TEST_COMPILER`.  This `LLDB_TEST_CLANG` was introduced later, I guess unaware of the presence of `LLDB_TEST_COMPILER`.  Would it be possible to standardize on one variable?  This would also mean deleting the `TEST_C_COMPILER` and TEST_CXX_COMPILER` variables.  I don't see why we should ever have both.  Run the test suite a second time if you need to test with different C and C++ compilers.


https://reviews.llvm.org/D39215