Indeed, that was my point.<br><br>As an aside, clang has the —driver-mode option that lets you specify either gcc or g++, which is in turn equivalent to running clang.exe vs clang++.exe.  Does gcc have this?  If so we can require it to be the non ++ version, and then build the driver mode argument as necessary.<br><br>If not, we can compute the other executable name based on the one variable <br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 23, 2017 at 4:52 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#904536" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39215#904536</a>, @labath wrote:<br>
<br>
> In <a href="https://reviews.llvm.org/D39215#904510" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39215#904510</a>, @zturner wrote:<br>
><br>
> > 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.<br>
><br>
><br>
> Unlike msvc, where you just use CL to compile everything, on ***nix you have different compiler drivers for C (gcc, clang) and C++ (g++, clang++). The differences are minute, but we still need both (clang has --driver-mode=g++ argument to emulate g++, but I don't know of any gcc option like that).<br>
><br>
> However, I do see a bit of space for simplification. I can remove LLDB_TEST_CLANG altogether, and just have LLDB_TEST_???_COMPILER default to clang.<br>
<br>
<br>
Scratch that, dotest.py already has some magic to compute the c++ compiler, given the c version, so I guess one variable is really enough.<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>