[compiler-rt] Revert "[compiler-rt] Also consider SIGPROF as a synchronous signal (#85188)" (PR #85296)

via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 14 11:59:43 PDT 2024


https://github.com/dyung created https://github.com/llvm/llvm-project/pull/85296

This reverts commit 6f3f659ce9ab91002b4a490b0ce4b085981383cd.

This change is causing TSAN failures on a bot as reported in #85188:
https://lab.llvm.org/buildbot/#/builders/247/builds/15279

>From 9f9dc1d1dcf6d3eb2a71ddbeaf881064d8043df4 Mon Sep 17 00:00:00 2001
From: Douglas Yung <dyung+github at fastmail.fm>
Date: Thu, 14 Mar 2024 11:57:48 -0700
Subject: [PATCH] Revert "[compiler-rt] Also consider SIGPROF as a synchronous
 signal (#85188)"

This reverts commit 6f3f659ce9ab91002b4a490b0ce4b085981383cd.

This change is causing TSAN failures on a bot:
https://lab.llvm.org/buildbot/#/builders/247/builds/15279
---
 compiler-rt/lib/tsan/rtl/tsan_interceptors_posix.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/compiler-rt/lib/tsan/rtl/tsan_interceptors_posix.cpp b/compiler-rt/lib/tsan/rtl/tsan_interceptors_posix.cpp
index 2bebe651b994e5..8ffc703b05eace 100644
--- a/compiler-rt/lib/tsan/rtl/tsan_interceptors_posix.cpp
+++ b/compiler-rt/lib/tsan/rtl/tsan_interceptors_posix.cpp
@@ -126,7 +126,6 @@ const int SIGFPE = 8;
 const int SIGSEGV = 11;
 const int SIGPIPE = 13;
 const int SIGTERM = 15;
-const int SIGPROF = 27;
 #if defined(__mips__) || SANITIZER_FREEBSD || SANITIZER_APPLE || SANITIZER_NETBSD
 const int SIGBUS = 10;
 const int SIGSYS = 12;
@@ -2169,8 +2168,7 @@ static bool is_sync_signal(ThreadSignalContext *sctx, int sig,
     return false;
 #endif
   return sig == SIGSEGV || sig == SIGBUS || sig == SIGILL || sig == SIGTRAP ||
-         sig == SIGABRT || sig == SIGFPE || sig == SIGPIPE || sig == SIGSYS ||
-         sig == SIGPROF;
+         sig == SIGABRT || sig == SIGFPE || sig == SIGPIPE || sig == SIGSYS;
 }
 
 void sighandler(int sig, __sanitizer_siginfo *info, void *ctx) {



More information about the llvm-commits mailing list