sgraenitz abandoned this revision.
sgraenitz added a comment.
I will split off the "dead code elimination" part and close both reviews in
favor of a more comprehensive fix.
================
Comment at: CMakeLists.txt:49
set(LLDB_DEFAULT_TEST_FILECHECK
"${LLVM_BINARY_DIR}/${CMAKE_CFG_INTDIR}/bin/FileCheck${CMAKE_EXECUTABLE_SUFFIX}")
-
- if (NOT LLDB_TEST_USE_CUSTOM_C_COMPILER AND TARGET clang)
- set(LLDB_DEFAULT_TEST_C_COMPILER
"${LLVM_BINARY_DIR}/${CMAKE_CFG_INTDIR}/bin/clang${CMAKE_EXECUTABLE_SUFFIX}")
- else()
- set(LLDB_DEFAULT_TEST_C_COMPILER "")
- endif()
-
- if (NOT LLDB_TEST_USE_CUSTOM_CXX_COMPILER AND TARGET clang)
- set(LLDB_DEFAULT_TEST_CXX_COMPILER
"${LLVM_BINARY_DIR}/${CMAKE_CFG_INTDIR}/bin/clang++${CMAKE_EXECUTABLE_SUFFIX}")
- else()
- set(LLDB_DEFAULT_TEST_CXX_COMPILER "")
+ if(TARGET clang)
+ set(LLDB_DEFAULT_TEST_COMPILER
"${LLVM_BINARY_DIR}/${CMAKE_CFG_INTDIR}/bin/clang${CMAKE_EXECUTABLE_SUFFIX}")
----------------
stella.stamenova wrote:
> Nit: it looks like your if statements lack a space before the parenthesis,
> while all the existing ones have a space. Please be consistent with the
> existing convention.
That doesn't seem to be the most popular convention ;-)
BTW do we have documentation for CMake coding style?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56400/new/
https://reviews.llvm.org/D56400
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
