[compiler-rt] 6789e1b - [sanitizer_common] Fix forward: limit #98200 to Linux only
Thurston Dang via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 10 14:24:22 PDT 2024
Author: Thurston Dang
Date: 2024-07-10T21:22:58Z
New Revision: 6789e1bb77c030c25ba5686c784bf4966770c324
URL: https://github.com/llvm/llvm-project/commit/6789e1bb77c030c25ba5686c784bf4966770c324
DIFF: https://github.com/llvm/llvm-project/commit/6789e1bb77c030c25ba5686c784bf4966770c324.diff
LOG: [sanitizer_common] Fix forward: limit #98200 to Linux only
My patch broke Solaris: https://github.com/llvm/llvm-project/pull/98200#issuecomment-2221463400
Fixing forward by limiting the change to Linux only.
Added:
Modified:
compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
Removed:
################################################################################
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
index 54f972c74794b..f7f5698a5f32d 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
@@ -170,7 +170,6 @@ void BlockSignals(__sanitizer_sigset_t *oldset) {
// If this signal is blocked, such calls cannot be handled and the process may
// hang.
internal_sigdelset(&set, 31);
-# endif
// Don't block synchronous signals
internal_sigdelset(&set, SIGSEGV);
@@ -180,6 +179,7 @@ void BlockSignals(__sanitizer_sigset_t *oldset) {
internal_sigdelset(&set, SIGABRT);
internal_sigdelset(&set, SIGFPE);
internal_sigdelset(&set, SIGPIPE);
+# endif
SetSigProcMask(&set, oldset);
}
More information about the llvm-commits
mailing list