[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