diff options
| author | Zachary Turner <zturner@google.com> | 2017-09-20 16:01:50 +0000 |
|---|---|---|
| committer | Zachary Turner <zturner@google.com> | 2017-09-20 16:01:50 +0000 |
| commit | fd4d2852f0093fce3f0b0b6b8cd5d9a09aaf2701 (patch) | |
| tree | 58696d10af59fbf1c04fb83c053faee23d516cbf /cmake/modules | |
| parent | d469ac59ee110e64d7de571af165befe1e32ce19 (diff) | |
Make libcxx tests work when llvm sources are not present.
Despite a strong CMake warning that this is an unsupported
libcxx build configuration, some bots still rely on being
able to check out lit and libcxx independently with no
LLVM sources, and then run lit against libcxx.
A previous patch broke that workflow, so this is making it work
again. Unfortunately, it breaks generation of the llvm-lit
script for libcxx, but we will just have to live with that until
a solution is found that allows libcxx to make more use of
llvm build pieces. libcxx can still run tests by using the
ninja check target, or by running lit.py directly against the
build tree or source tree.
Differential Revision: https://reviews.llvm.org/D38057
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@313763 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'cmake/modules')
| -rw-r--r-- | cmake/modules/AddLLVM.cmake | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/cmake/modules/AddLLVM.cmake b/cmake/modules/AddLLVM.cmake index 06840fb9d74..7c26e396caf 100644 --- a/cmake/modules/AddLLVM.cmake +++ b/cmake/modules/AddLLVM.cmake @@ -1198,7 +1198,12 @@ function(get_llvm_lit_path base_dir file_name) cmake_parse_arguments(ARG "ALLOW_EXTERNAL" "" "" ${ARGN}) if (ARG_ALLOW_EXTERNAL) + set(LLVM_DEFAULT_EXTERNAL_LIT "${LLVM_EXTERNAL_LIT}") set (LLVM_EXTERNAL_LIT "" CACHE STRING "Command used to spawn lit") + if ("${LLVM_EXTERNAL_LIT}" STREQUAL "") + set(LLVM_EXTERNAL_LIT "${LLVM_DEFAULT_EXTERNAL_LIT}") + endif() + if (NOT "${LLVM_EXTERNAL_LIT}" STREQUAL "") if (EXISTS ${LLVM_EXTERNAL_LIT}) get_filename_component(LIT_FILE_NAME ${LLVM_EXTERNAL_LIT} NAME) @@ -1230,9 +1235,7 @@ function(get_llvm_lit_path base_dir file_name) elseif(NOT "${LLVM_RUNTIME_OUTPUT_INTDIR}" STREQUAL "") set(LLVM_LIT_BASE_DIR ${LLVM_RUNTIME_OUTPUT_INTDIR}) else() - message(WARNING "Could not find suitable output location for llvm-lit." - "Using default ${CMAKE_CURRENT_BINARY_DIR}/llvm-lit") - set(LLVM_LIT_BASE_DIR ${CMAKE_CURRENT_BINARY_DIR}/llvm-lit) + set(LLVM_LIT_BASE_DIR "") endif() # Cache this so we don't have to do it again and have subsequent calls |