Skip to content

Commit b33b122

Browse files
Wael Yehiadaltenty
andcommitted
[compiler-rt] follow-up to 166837, rename COMPILER_RT_TEST_BUILTINS_DIR to COMPILER_RT_FORCE_TEST_BUILTINS_DIR
--------- Co-authored-by: David Tenty <daltenty@ibm.com>
1 parent 069d3aa commit b33b122

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

compiler-rt/test/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,10 @@ endfunction()
7373
# Run sanitizer tests only if we're sure that clang would produce
7474
# working binaries.
7575
if(COMPILER_RT_CAN_EXECUTE_TESTS)
76-
# COMPILER_RT_TEST_BUILTINS_DIR allows running tests against builtins built
76+
# COMPILER_RT_FORCE_TEST_BUILTINS_DIR allows running tests against builtins built
7777
# in an independent build. This option is only indended to be used by
7878
# LLVM_ENABLE_RUNTIMES-based builds.
79-
if(COMPILER_RT_BUILD_BUILTINS OR COMPILER_RT_TEST_BUILTINS_DIR)
79+
if(COMPILER_RT_BUILD_BUILTINS OR COMPILER_RT_FORCE_TEST_BUILTINS_DIR)
8080
add_subdirectory(builtins)
8181
endif()
8282
if(COMPILER_RT_BUILD_SANITIZERS)

compiler-rt/test/builtins/CMakeLists.txt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
set(BUILTINS_LIT_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR})
22

3-
# If COMPILER_RT_TEST_BUILTINS_DIR is set, the builtins
3+
# If COMPILER_RT_FORCE_TEST_BUILTINS_DIR is set, the builtins
44
# were already built and we are just going to test them.
55
# NOTE: This is currently an LLVM-internal option which should
66
# only be used by the LLVM_ENABLE_RUNTIMES build configured
77
# in llvm/runtimes/CMakeLists.txt
8-
if(COMPILER_RT_TEST_BUILTINS_DIR)
8+
if(COMPILER_RT_FORCE_TEST_BUILTINS_DIR)
99
set(BUILTINS_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS})
1010
else()
1111
set(BUILTINS_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS} builtins)
@@ -96,16 +96,16 @@ foreach(arch ${BUILTIN_TEST_ARCH})
9696
endif()
9797
# Normally, we can just inspect the target directly to get the sources, but if
9898
# we are testing an externally-built builtins library, we expect
99-
# COMPILER_RT_TEST_BUILTINS_DIR to be set and contain a file named
99+
# COMPILER_RT_FORCE_TEST_BUILTINS_DIR to be set and contain a file named
100100
# ${BUILTIN_LIB_TARGET_NAME}.sources.txt from the builtins build. This file
101101
# is created by compiler-rt/lib/builtins/CMakeLists.txt
102-
if(NOT COMPILER_RT_TEST_BUILTINS_DIR)
102+
if(NOT COMPILER_RT_FORCE_TEST_BUILTINS_DIR)
103103
if (NOT TARGET "${BUILTIN_LIB_TARGET_NAME}")
104104
message(FATAL_ERROR "Target ${BUILTIN_LIB_TARGET_NAME} does not exist")
105105
endif()
106106
get_target_property(BUILTIN_LIB_SOURCES "${BUILTIN_LIB_TARGET_NAME}" SOURCES)
107107
else()
108-
file(READ "${COMPILER_RT_TEST_BUILTINS_DIR}/${BUILTIN_LIB_TARGET_NAME}.sources.txt" BUILTIN_LIB_SOURCES)
108+
file(READ "${COMPILER_RT_FORCE_TEST_BUILTINS_DIR}/${BUILTIN_LIB_TARGET_NAME}.sources.txt" BUILTIN_LIB_SOURCES)
109109
endif()
110110
list(LENGTH BUILTIN_LIB_SOURCES BUILTIN_LIB_SOURCES_LENGTH)
111111
if (BUILTIN_LIB_SOURCES_LENGTH EQUAL 0)

0 commit comments

Comments
 (0)