-
Notifications
You must be signed in to change notification settings - Fork 13.5k
[LSAN] Add extra suppressions on Apple Aarch64 #117478
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@llvm/pr-subscribers-compiler-rt-sanitizer Author: Mads Marquart (madsmtm) Changes
Fixes #115992. Full diff: https://github.com/llvm/llvm-project/pull/117478.diff 1 Files Affected:
diff --git a/compiler-rt/lib/lsan/lsan_common.cpp b/compiler-rt/lib/lsan/lsan_common.cpp
index 7ab9e4ff2ac9fd..56b943fa05b459 100644
--- a/compiler-rt/lib/lsan/lsan_common.cpp
+++ b/compiler-rt/lib/lsan/lsan_common.cpp
@@ -120,7 +120,14 @@ static const char kStdSuppressions[] =
# endif // SANITIZER_SUPPRESS_LEAK_ON_PTHREAD_EXIT
# if SANITIZER_APPLE
// For Darwin and os_log/os_trace: https://reviews.llvm.org/D35173
+# if SANITIZER_ARM64
+ // Apple Aarch64 leaks on startup in _fetchInitializingClassList and when
+ // using thread locals in dyld4::RuntimeState::_instantiateTLVs.
+ // See https://github.com/llvm/llvm-project/issues/115992.
+ "leak:*_os_trace*\nleak:*_fetchInitializingClassList*\nleak:*dyld4::RuntimeState::_instantiateTLVs*\n"
+# else
"leak:*_os_trace*\n"
+# endif
# endif
// TLS leak in some glibc versions, described in
// https://sourceware.org/bugzilla/show_bug.cgi?id=12650.
|
✅ With the latest revision this PR passed the C/C++ code formatter. |
4a1855f
to
eec7727
Compare
- _fetchInitializingClassList for startup code in dyld. - dyld4::RuntimeState::_instantiateTLVs for thread locals.
eec7727
to
dee3070
Compare
FYI: |
Is this will merge into 19.1.6 ? |
Fixes #115992, now it shows the following instead (in a project using thread locals):