From e62e0b167581e7881e93b91127f6349b6b4e2143 Mon Sep 17 00:00:00 2001 From: Vitaly Buka Date: Wed, 4 Nov 2020 22:31:36 -0800 Subject: [PATCH] Revert "[LSAN] Fix preprocessor condition for MSVC" Missread the error message. It was not the reason. This reverts commit 2d041554d1f94abed3ddd396d8af944acf63fe4e. --- compiler-rt/lib/lsan/lsan_common.h | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/compiler-rt/lib/lsan/lsan_common.h b/compiler-rt/lib/lsan/lsan_common.h index da5d231..a26a8b3 100644 --- a/compiler-rt/lib/lsan/lsan_common.h +++ b/compiler-rt/lib/lsan/lsan_common.h @@ -29,14 +29,10 @@ // To enable LeakSanitizer on a new architecture, one needs to implement the // internal_clone function as well as (probably) adjust the TLS machinery for // the new architecture inside the sanitizer library. -#if SANITIZER_ANDROID // Exclude leak-detection on arm32 for Android because `__aeabi_read_tp` // is missing. This caused a link error. -#if (__ANDROID_API__ < 28 || defined(__arm__)) +#if SANITIZER_ANDROID && (__ANDROID_API__ < 28 || defined(__arm__)) #define CAN_SANITIZE_LEAKS 0 -#else -#define CAN_SANITIZE_LEAKS 1 -#endif #elif (SANITIZER_LINUX || SANITIZER_MAC) && (SANITIZER_WORDSIZE == 64) && \ (defined(__x86_64__) || defined(__mips64) || defined(__aarch64__) || \ defined(__powerpc64__) || defined(__s390x__)) -- 2.7.4