From 9a88edeb49b3edefbb9933fa5b71d7d421fc99bd Mon Sep 17 00:00:00 2001 From: Petr Hosek Date: Mon, 30 Dec 2024 11:51:44 -0800 Subject: [PATCH] [TYSan][CMake] CMake build fixes (#121224) TYSan CMake build follows patterns used by other sanitizers, but there's also a number of issues, like referring to undefined variables, which breaks the build in some cases (such as cross-compiling). This change addresses the issues. --- compiler-rt/lib/tysan/CMakeLists.txt | 29 +++++++++++++++++++++------- 1 file changed, 22 insertions(+), 7 deletions(-) diff --git a/compiler-rt/lib/tysan/CMakeLists.txt b/compiler-rt/lib/tysan/CMakeLists.txt index 859b67928f00..7d13ae396391 100644 --- a/compiler-rt/lib/tysan/CMakeLists.txt +++ b/compiler-rt/lib/tysan/CMakeLists.txt @@ -3,11 +3,25 @@ include_directories(..) # Runtime library sources and build flags. set(TYSAN_SOURCES tysan.cpp - tysan_interceptors.cpp) + tysan_interceptors.cpp + ) + +SET(TYSAN_HEADERS + tysan.h + tysan_flags.inc + tysan_platform.h + ) + set(TYSAN_COMMON_CFLAGS ${SANITIZER_COMMON_CFLAGS}) append_rtti_flag(OFF TYSAN_COMMON_CFLAGS) # Prevent clang from generating libc calls. append_list_if(COMPILER_RT_HAS_FFREESTANDING_FLAG -ffreestanding TYSAN_COMMON_CFLAGS) +set(TYSAN_DYNAMIC_CFLAGS ${TYSAN_COMMON_CFLAGS}) + +set(TYSAN_COMMON_DEFINITIONS "") +set(TYSAN_DYNAMIC_DEFINITIONS ${TYSAN_COMMON_DEFINITIONS} TYSAN_DYNAMIC=1) + +# Compile TYSan sources into an object library. add_compiler_rt_object_libraries(RTTysan_dynamic OS ${SANITIZER_COMMON_SUPPORTED_OS} @@ -47,17 +61,18 @@ if(APPLE) DEFS ${TYSAN_COMMON_DEFINITIONS} PARENT_TARGET tysan) else() + set(TYSAN_CFLAGS ${TYSAN_COMMON_CFLAGS}) + append_list_if(COMPILER_RT_HAS_FPIE_FLAG -fPIE TYSAN_CFLAGS) + foreach(arch ${TYSAN_SUPPORTED_ARCH}) - set(TYSAN_CFLAGS ${TYSAN_COMMON_CFLAGS}) - append_list_if(COMPILER_RT_HAS_FPIE_FLAG -fPIE TYSAN_CFLAGS) add_compiler_rt_runtime(clang_rt.tysan STATIC ARCHS ${arch} SOURCES ${TYSAN_SOURCES} - $ - $ - $ - $ + OBJECT_LIBS RTInterception + RTSanitizerCommon + RTSanitizerCommonLibc + RTSanitizerCommonSymbolizer CFLAGS ${TYSAN_CFLAGS} PARENT_TARGET tysan) endforeach()