This is an archive of the discontinued LLVM Phabricator instance.

Differential D115584

[lsan] Format CAN_SANITIZE_LEAKS defines
ClosedPublic

Authored by clemenswasser on Dec 11 2021, 12:55 PM.

Details

Reviewers
vitalybuka
Commits
rG66d1fdb29b7a: [lsan] Format CAN_SANITIZE_LEAKS defines

Diff Detail

Event Timeline

clemenswasser requested review of this revision.Dec 11 2021, 12:55 PM
clemenswasser created this revision.
clemenswasser added a child revision: D115103: Leak Sanitizer port to Windows.
Harbormaster completed remote builds in B138814: Diff 393691.Dec 11 2021, 1:11 PM
vitalybuka accepted this revision.Dec 16 2021, 11:49 AM
This revision is now accepted and ready to land.Dec 16 2021, 11:49 AM
This revision was landed with ongoing or failed builds.Dec 16 2021, 11:55 AM
Closed by commit rG66d1fdb29b7a: [lsan] Format CAN_SANITIZE_LEAKS defines (authored by clemenswasser, committed by vitalybuka). · Explain Why
This revision was automatically updated to reflect the committed changes.
vitalybuka added a commit: rG66d1fdb29b7a: [lsan] Format CAN_SANITIZE_LEAKS defines.
Herald added a project: Restricted Project. · View Herald TranscriptDec 16 2021, 11:55 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
clemenswasser removed a child revision: D115103: Leak Sanitizer port to Windows.Apr 10 2022, 8:55 AM

Revision Contents

PathSize
compiler-rt/
lib/
lsan/
14 lines

Diff 394948

compiler-rt/lib/lsan/lsan_common.h

Show All 27 Lines
// ratio on large leaks. But we still want to have it for some 32 bit arches // ratio on large leaks. But we still want to have it for some 32 bit arches
// (e.g. x86), see https://github.com/google/sanitizers/issues/403. // (e.g. x86), see https://github.com/google/sanitizers/issues/403.
// To enable LeakSanitizer on a new architecture, one needs to implement the // To enable LeakSanitizer on a new architecture, one needs to implement the
// internal_clone function as well as (probably) adjust the TLS machinery for // internal_clone function as well as (probably) adjust the TLS machinery for
// the new architecture inside the sanitizer library. // the new architecture inside the sanitizer library.
// Exclude leak-detection on arm32 for Android because `__aeabi_read_tp` // Exclude leak-detection on arm32 for Android because `__aeabi_read_tp`
// is missing. This caused a link error. // is missing. This caused a link error.
#if SANITIZER_ANDROID && (__ANDROID_API__ < 28 || defined(__arm__)) #if SANITIZER_ANDROID && (__ANDROID_API__ < 28 || defined(__arm__))
#define CAN_SANITIZE_LEAKS 0 # define CAN_SANITIZE_LEAKS 0
#elif (SANITIZER_LINUX || SANITIZER_MAC) && (SANITIZER_WORDSIZE == 64) && \ #elif (SANITIZER_LINUX || SANITIZER_MAC) && (SANITIZER_WORDSIZE == 64) && \
(defined(__x86_64__) || defined(__mips64) || defined(__aarch64__) || \ (defined(__x86_64__) || defined(__mips64) || defined(__aarch64__) || \
defined(__powerpc64__) || defined(__s390x__)) defined(__powerpc64__) || defined(__s390x__))
#define CAN_SANITIZE_LEAKS 1 # define CAN_SANITIZE_LEAKS 1
#elif defined(__i386__) && (SANITIZER_LINUX || SANITIZER_MAC) #elif defined(__i386__) && (SANITIZER_LINUX || SANITIZER_MAC)
#define CAN_SANITIZE_LEAKS 1 # define CAN_SANITIZE_LEAKS 1
#elif defined(__arm__) && SANITIZER_LINUX #elif defined(__arm__) && SANITIZER_LINUX
#define CAN_SANITIZE_LEAKS 1 # define CAN_SANITIZE_LEAKS 1
#elif SANITIZER_RISCV64 && SANITIZER_LINUX #elif SANITIZER_RISCV64 && SANITIZER_LINUX
#define CAN_SANITIZE_LEAKS 1 # define CAN_SANITIZE_LEAKS 1
#elif SANITIZER_NETBSD || SANITIZER_FUCHSIA #elif SANITIZER_NETBSD || SANITIZER_FUCHSIA
#define CAN_SANITIZE_LEAKS 1 # define CAN_SANITIZE_LEAKS 1
#else #else
#define CAN_SANITIZE_LEAKS 0 # define CAN_SANITIZE_LEAKS 0
#endif #endif
namespace __sanitizer { namespace __sanitizer {
class FlagParser; class FlagParser;
class ThreadRegistry; class ThreadRegistry;
class ThreadContextBase; class ThreadContextBase;
struct DTLS; struct DTLS;
} }
▲ Show 20 Lines • Show All 258 Lines • Show Last 20 Lines