[compiler-rt] [rtsan] Re-enable rtsan tests (PR #98219)

Chris Apple via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 14:26:03 PDT 2024


================
@@ -751,7 +751,8 @@ else()
   set(COMPILER_RT_HAS_ASAN FALSE)
 endif()
 
-if (COMPILER_RT_HAS_SANITIZER_COMMON AND RTSAN_SUPPORTED_ARCH)
+if (COMPILER_RT_HAS_SANITIZER_COMMON AND RTSAN_SUPPORTED_ARCH AND
+    OS_NAME MATCHES "Android|Darwin|Linux")
----------------
cjappl wrote:

Sounds good to me.

I believe we need to add the conditional check for COMPILER_RT_HAS_RTSAN in compiler-rt/lib/rtsan/CMakeLists.txt. We missed that in the first PR.

With that change, this should be good! I'm going to approve, assuming that you add that conditional in there.



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


More information about the llvm-commits mailing list