Skip to content

Commit 3cef099

Browse files
[TySan][CMake] Depend on tysan for check-tysan in runtimes build (#143597)
The runtimes build expects libclang_rt.tysan.a to be available, but the check-tysan target does not actually depend on it when built using a runtimes build with LLVM_ENABLE_RUNTIMES pointing at ./llvm. This means we get test failures when running check-compiler-rt due to the missing static archive. This patch also makes check-tysan depend on tysan when we are using the runtimes build. This is causing premerge failures currently since we recently migrated to the runtimes build.
1 parent 6f62979 commit 3cef099

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

compiler-rt/test/tysan/CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,7 @@ foreach(arch ${TYSAN_TEST_ARCH})
2121
endforeach()
2222

2323
set(TYSAN_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS})
24-
if(NOT COMPILER_RT_STANDALONE_BUILD)
25-
list(APPEND TYSAN_TEST_DEPS tysan)
26-
endif()
24+
list(APPEND TYSAN_TEST_DEPS tysan)
2725

2826
add_lit_testsuite(check-tysan "Running the TypeSanitizer tests"
2927
${TYSAN_TESTSUITES}

0 commit comments

Comments
 (0)