Skip to content

Commit

Permalink
[lsan] Remove unnecessary elf-tls condition
Browse files Browse the repository at this point in the history
Differential Revision: https://reviews.llvm.org/D90808
  • Loading branch information
oontvoo committed Nov 5, 2020
1 parent ac73b73 commit 796650d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions compiler-rt/lib/lsan/lsan_common.h
Expand Up @@ -29,14 +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_LINUX && !SANITIZER_ANDROID) || SANITIZER_MAC) && \
(SANITIZER_WORDSIZE == 64) && \
#if (SANITIZER_LINUX || SANITIZER_MAC) && (SANITIZER_WORDSIZE == 64) && \
(defined(__x86_64__) || defined(__mips64) || defined(__aarch64__) || \
defined(__powerpc64__) || defined(__s390x__))
#define CAN_SANITIZE_LEAKS 1
#elif defined(__i386__) && \
((SANITIZER_LINUX && !SANITIZER_ANDROID) || SANITIZER_MAC)
#elif defined(__i386__) && (SANITIZER_LINUX || SANITIZER_MAC)
#define CAN_SANITIZE_LEAKS 1
// Exclude leak-detection on arm32 for Android because `__aeabi_read_tp`
// is missing. This caused a link error.
#elif defined(__arm__) && SANITIZER_LINUX && !SANITIZER_ANDROID
#define CAN_SANITIZE_LEAKS 1
#elif SANITIZER_NETBSD || SANITIZER_FUCHSIA
Expand Down

0 comments on commit 796650d

Please sign in to comment.