Commit 77a103dd authored by dbates@webkit.org's avatar dbates@webkit.org

Only #include <signal.h> and require SA_RESTART when building with JSC_MULTIPLE_THREADS

https://bugs.webkit.org/show_bug.cgi?id=66617

Both <signal.h> and SA_RESTART usage are guarded behind ENABLE(JSC_MULTIPLE_THREADS).
But we cause a compile error if the platform doesn't support SA_RESTART regardless of
whether JSC_MULTIPLE_THREADS is enabled for the port. Instead, we shouldn't require
SA_RESTART support unless we are building with JSC_MULTIPLE_THREADS enabled.

Reviewed by Antonio Gomes.

* heap/MachineStackMarker.cpp:


git-svn-id: svn://svn.chromium.org/blink/trunk@93467 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 504fdb68
2011-08-19 Daniel Bates <dbates@webkit.org>
Only #include <signal.h> and require SA_RESTART when building with JSC_MULTIPLE_THREADS
https://bugs.webkit.org/show_bug.cgi?id=66617
Both <signal.h> and SA_RESTART usage are guarded behind ENABLE(JSC_MULTIPLE_THREADS).
But we cause a compile error if the platform doesn't support SA_RESTART regardless of
whether JSC_MULTIPLE_THREADS is enabled for the port. Instead, we shouldn't require
SA_RESTART support unless we are building with JSC_MULTIPLE_THREADS enabled.
Reviewed by Antonio Gomes.
* heap/MachineStackMarker.cpp:
2011-08-19 Filip Pizlo <fpizlo@apple.com>
The JSC JIT currently has no facility to profile and report
......
......@@ -76,7 +76,7 @@
#include <errno.h>
#endif
#if USE(PTHREADS) && !OS(WINDOWS) && !OS(DARWIN)
#if ENABLE(JSC_MULTIPLE_THREADS) && USE(PTHREADS) && !OS(WINDOWS) && !OS(DARWIN)
#include <signal.h>
#ifndef SA_RESTART
#error MachineThreads requires SA_RESTART
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment