From: Chris Lattner Date: Mon, 23 Mar 2009 05:42:29 +0000 (+0000) Subject: factorize signal registration, part of PR3848. X-Git-Tag: android-x86-6.0-r1~1003^2~21658 X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=1c4d8a00aba2b216dfd7c2b95f01752465037d70;p=android-x86%2Fexternal-llvm.git factorize signal registration, part of PR3848. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@67508 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/System/Unix/Signals.inc b/lib/System/Unix/Signals.inc index e031e85b425..60e3a26314e 100644 --- a/lib/System/Unix/Signals.inc +++ b/lib/System/Unix/Signals.inc @@ -31,6 +31,8 @@ #endif using namespace llvm; +static RETSIGTYPE SignalHandler(int Sig); // defined below. + /// InterruptFunction - The function to call if ctrl-c is pressed. static void (*InterruptFunction)() = 0; @@ -55,18 +57,34 @@ static const int KillSigs[] = { static const int *const KillSigsEnd = KillSigs + sizeof(KillSigs) / sizeof(KillSigs[0]); +// Just call signal +static void RegisterHandler(int Signal) { + signal(Signal, SignalHandler); +} + +static void RegisterHandlers() { + std::for_each(IntSigs, IntSigsEnd, RegisterHandler); + std::for_each(KillSigs, KillSigsEnd, RegisterHandler); +} + static void UnregisterHandler(int Signal) { - signal(Signal, SIG_DFL); + signal(Signal, SIG_DFL); +} + +static void UnregisterHandlers() { + std::for_each(KillSigs, KillSigsEnd, UnregisterHandler); + std::for_each(IntSigs, IntSigsEnd, UnregisterHandler); } + // SignalHandler - The signal handler that runs. static RETSIGTYPE SignalHandler(int Sig) { // Restore the signal behavior to default, so that the program actually // crashes when we return and the signal reissues. This also ensures that if // we crash in our signal handler that the program will terminate immediately // instead of recursing in the signal handler. - std::for_each(KillSigs, KillSigsEnd, UnregisterHandler); + UnregisterHandlers(); // Unmask all potentially blocked kill signals. sigset_t SigMask; @@ -95,15 +113,11 @@ static RETSIGTYPE SignalHandler(int Sig) { (*CallBacksToRun)[i].first((*CallBacksToRun)[i].second); } -// Just call signal -static void RegisterHandler(int Signal) { - signal(Signal, SignalHandler); -} void llvm::sys::SetInterruptFunction(void (*IF)()) { InterruptFunction = IF; - RegisterHandler(SIGINT); + RegisterHandlers(); } // RemoveFileOnSignal - The public API @@ -114,8 +128,7 @@ bool llvm::sys::RemoveFileOnSignal(const sys::Path &Filename, FilesToRemove->push_back(Filename); - std::for_each(IntSigs, IntSigsEnd, RegisterHandler); - std::for_each(KillSigs, KillSigsEnd, RegisterHandler); + RegisterHandlers(); return false; } @@ -126,7 +139,7 @@ void llvm::sys::AddSignalHandler(void (*FnPtr)(void *), void *Cookie) { if (CallBacksToRun == 0) CallBacksToRun = new std::vector >(); CallBacksToRun->push_back(std::make_pair(FnPtr, Cookie)); - std::for_each(KillSigs, KillSigsEnd, RegisterHandler); + RegisterHandlers(); }