[compiler-rt] [compiler-rt] [CMake] Add option to disable sanitizer-common builds by default (PR #160962)
Dan Blackwell via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 30 05:09:12 PDT 2025
================
@@ -8,7 +8,7 @@ include(SanitizerUtils)
# sanitizers or xray (or both).
#
#TODO: Refactor sanitizer_common into smaller pieces (e.g. flag parsing, utils).
-if (COMPILER_RT_HAS_SANITIZER_COMMON AND
+if (COMPILER_RT_HAS_SANITIZER_COMMON AND COMPILER_RT_BUILD_SANITIZER_COMMON AND
(COMPILER_RT_BUILD_SANITIZERS OR COMPILER_RT_BUILD_XRAY OR COMPILER_RT_BUILD_MEMPROF OR COMPILER_RT_BUILD_CTX_PROFILE))
----------------
DanBlackwell wrote:
I presume that these options depend on having sanitizer_common built; should we instead be adding the new option to the 'OR'? i.e. is there a use case for building sanitizers/xray/memprof without sanitizer_common?
https://github.com/llvm/llvm-project/pull/160962
More information about the llvm-commits
mailing list