JDevlieghere marked 2 inline comments as done.
JDevlieghere added inline comments.


================
Comment at: CMakeLists.txt:160
     list(APPEND LLDB_TEST_DEPS clang)
+    if(APPLE)
+      list(APPEND LLDB_TEST_DEPS cxx)
----------------
clayborg wrote:
> What if the user wants to use a different compiler for tests? cmake has 
> LLDB_TEST_USE_CUSTOM_C_COMPILER and LLDB_TEST_C_COMPILER and 
> LLDB_TEST_CXX_COMPILER which can supposedly be set.
That still works. The change says that *if* you are going to build the in-tree 
clang, it has to build libcxx(abi) too. That should always make sense, because 
clang (on Darwin) won't be able to find the standard library otherwise. 


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55837/new/

https://reviews.llvm.org/D55837



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

Reply via email to