Skip to content

Commit ac3398a

Browse files
committed
Revert "[CMake] Include runtimes test suites in check-all"
This reverts commit e54a7bf. There was a bug reported with this patch, see: https://reviews.llvm.org/D121276#3375464
1 parent df2fcea commit ac3398a

File tree

4 files changed

+4
-27
lines changed

4 files changed

+4
-27
lines changed

llvm/CMakeLists.txt

+3
Original file line numberDiff line numberDiff line change
@@ -1138,6 +1138,9 @@ if( LLVM_INCLUDE_TESTS )
11381138
DEPENDS ${LLVM_LIT_DEPENDS} ${LLVM_ADDITIONAL_TEST_TARGETS}
11391139
ARGS ${LLVM_LIT_EXTRA_ARGS}
11401140
)
1141+
if(TARGET check-runtimes)
1142+
add_dependencies(check-all check-runtimes)
1143+
endif()
11411144
add_custom_target(test-depends
11421145
DEPENDS ${LLVM_LIT_DEPENDS} ${LLVM_ADDITIONAL_TEST_DEPENDS})
11431146
set_target_properties(test-depends PROPERTIES FOLDER "Tests")

llvm/runtimes/CMakeLists.txt

+1-14
Original file line numberDiff line numberDiff line change
@@ -220,13 +220,6 @@ function(runtime_default_target)
220220
endforeach()
221221

222222
if(LLVM_INCLUDE_TESTS)
223-
include(${LLVM_BINARY_DIR}/runtimes/Tests.cmake OPTIONAL RESULT_VARIABLE have_tests)
224-
set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${LLVM_BINARY_DIR}/runtimes/Tests.cmake)
225-
if(have_tests)
226-
set_property(GLOBAL APPEND PROPERTY LLVM_LIT_TESTSUITES ${RUNTIMES_LIT_TESTSUITES})
227-
set_property(GLOBAL APPEND PROPERTY LLVM_LIT_PARAMS ${RUNTIMES_LIT_PARAMS})
228-
set_property(GLOBAL APPEND PROPERTY LLVM_LIT_EXTRA_ARGS ${RUNTIMES_LIT_EXTRA_ARGS})
229-
endif()
230223
list(APPEND test_targets runtimes-test-depends check-runtimes)
231224
endif()
232225

@@ -303,13 +296,6 @@ function(runtime_register_target name target)
303296
endforeach()
304297

305298
if(LLVM_INCLUDE_TESTS)
306-
include(${LLVM_BINARY_DIR}/runtimes/${name}/Tests.cmake OPTIONAL RESULT_VARIABLE have_tests)
307-
set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${LLVM_BINARY_DIR}/runtimes/${name}/Tests.cmake)
308-
if(have_tests)
309-
set_property(GLOBAL APPEND PROPERTY LLVM_LIT_TESTSUITES ${RUNTIMES_LIT_TESTSUITES})
310-
set_property(GLOBAL APPEND PROPERTY LLVM_LIT_PARAMS ${RUNTIMES_LIT_PARAMS})
311-
set_property(GLOBAL APPEND PROPERTY LLVM_LIT_EXTRA_ARGS ${RUNTIMES_LIT_EXTRA_ARGS})
312-
endif()
313299
set(runtimes-test-depends-${name} runtimes-test-depends)
314300
set(check-runtimes-${name} check-runtimes)
315301
list(APPEND ${name}_test_targets runtimes-test-depends-${name} check-runtimes-${name})
@@ -482,6 +468,7 @@ if(runtimes)
482468

483469
if(LLVM_INCLUDE_TESTS)
484470
set_property(GLOBAL APPEND PROPERTY LLVM_ADDITIONAL_TEST_DEPENDS runtimes-test-depends)
471+
set_property(GLOBAL APPEND PROPERTY LLVM_ADDITIONAL_TEST_TARGETS check-runtimes)
485472

486473
set(RUNTIMES_TEST_DEPENDS
487474
FileCheck

runtimes/CMakeLists.txt

-10
Original file line numberDiff line numberDiff line change
@@ -221,16 +221,6 @@ if(LLVM_INCLUDE_TESTS)
221221
add_subdirectory(${LLVM_MAIN_SRC_DIR}/utils/llvm-lit
222222
${CMAKE_CURRENT_BINARY_DIR}/llvm-lit)
223223
endif()
224-
225-
if(LLVM_RUNTIMES_TARGET)
226-
configure_file(
227-
${CMAKE_CURRENT_SOURCE_DIR}/Tests.cmake.in
228-
${LLVM_BINARY_DIR}/runtimes/${LLVM_RUNTIMES_TARGET}/Tests.cmake)
229-
else()
230-
configure_file(
231-
${CMAKE_CURRENT_SOURCE_DIR}/Tests.cmake.in
232-
${LLVM_BINARY_DIR}/runtimes/Tests.cmake)
233-
endif()
234224
endif()
235225

236226
get_property(SUB_COMPONENTS GLOBAL PROPERTY SUB_COMPONENTS)

runtimes/Tests.cmake.in

-3
This file was deleted.

0 commit comments

Comments
 (0)