[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.

Yea.  Right now there's FOUR different variables to specify the compiler, and they can all conflict with each other.  You can theoretically set

  LLDB_TEST_COMPILER=<path-to-out-of-tree-clang>
  LLDB_TEST_CLANG=On
  LLDB_TEST_C_COMPILER=/usr/bin/cc
  LLDB_TEST_CXX_COMPILER=/usr/bin/clang++

This is a completely nonsensical configuration, and it shouldn't be allowed.  One variable to rule them, `LLDB_TEST_COMPILER=<path>`.  If it's present, use it.  Otherwise, default to in-tree clang.  Deduce cxx and c compilers from the given path.


https://reviews.llvm.org/D39215