[compiler-rt] 2d04155 - [LSAN] Fix preprocessor condition for MSVC

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 22:29:19 PST 2020


Author: Vitaly Buka
Date: 2020-11-04T22:29:09-08:00
New Revision: 2d041554d1f94abed3ddd396d8af944acf63fe4e

URL: https://github.com/llvm/llvm-project/commit/2d041554d1f94abed3ddd396d8af944acf63fe4e
DIFF: https://github.com/llvm/llvm-project/commit/2d041554d1f94abed3ddd396d8af944acf63fe4e.diff

LOG: [LSAN] Fix preprocessor condition for MSVC

Added: 
    

Modified: 
    compiler-rt/lib/lsan/lsan_common.h

Removed: 
    


################################################################################
diff  --git a/compiler-rt/lib/lsan/lsan_common.h b/compiler-rt/lib/lsan/lsan_common.h
index a26a8b37aa0b..da5d2318144f 100644
--- a/compiler-rt/lib/lsan/lsan_common.h
+++ b/compiler-rt/lib/lsan/lsan_common.h
@@ -29,10 +29,14 @@
 // 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 SANITIZER_ANDROID && (__ANDROID_API__ < 28 || defined(__arm__))
+#if (__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__))


        


More information about the llvm-commits mailing list