[compiler-rt] [TySan][CMake] Depend on tysan for check-tysan in runtimes build (PR #143597)

[via llvm-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

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.

---
Full diff: https://github.com/llvm/llvm-project/pull/143597.diff


1 Files Affected:

- (modified) compiler-rt/test/tysan/CMakeLists.txt (+1-1) 


``````````diff
diff --git a/compiler-rt/test/tysan/CMakeLists.txt b/compiler-rt/test/tysan/CMakeLists.txt
index 76f57501e854e..6f1beef65ee46 100644
--- a/compiler-rt/test/tysan/CMakeLists.txt
+++ b/compiler-rt/test/tysan/CMakeLists.txt
@@ -21,7 +21,7 @@ foreach(arch ${TYSAN_TEST_ARCH})
 endforeach()
 
 set(TYSAN_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS})
-if(NOT COMPILER_RT_STANDALONE_BUILD)
+if(NOT COMPILER_RT_STANDALONE_BUILD OR LLVM_RUNTIMES_BUILD)
   list(APPEND TYSAN_TEST_DEPS tysan)
 endif()
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/143597