[PATCH] D9638: Implement `internal_start/join_thread` on Mac OS X
Ismail Pazarbasi
ismail.pazarbasi at gmail.com
Fri Jul 24 12:57:59 PDT 2015
ismailp updated this revision to Diff 30596.
ismailp added a comment.
- Removed preprocessor directives guarding pthread.
http://reviews.llvm.org/D9638
Files:
lib/sanitizer_common/sanitizer_mac.cc
Index: lib/sanitizer_common/sanitizer_mac.cc
===================================================================
--- lib/sanitizer_common/sanitizer_mac.cc
+++ lib/sanitizer_common/sanitizer_mac.cc
@@ -365,8 +365,20 @@
return info.resident_size;
}
-void *internal_start_thread(void (*func)(void *arg), void *arg) { return 0; }
-void internal_join_thread(void *th) { }
+// FIXME: Move internal_start_thread and internal_join_thread from both Mac and
+// Linux into sanitizer_posix.cc.
+void *internal_start_thread(void(*func)(void *arg), void *arg) {
+ // Start the thread with signals blocked, otherwise it can steal user signals.
+ __sanitizer_sigset_t set, old;
+ internal_sigfillset(&set);
+ internal_sigprocmask(SIG_SETMASK, &set, &old);
+ pthread_t th;
+ pthread_create(&th, 0, (void*(*)(void *arg))func, arg);
+ internal_sigprocmask(SIG_SETMASK, &old, 0);
+ return th;
+}
+
+void internal_join_thread(void *th) { pthread_join((pthread_t)th, 0); }
void GetPcSpBp(void *context, uptr *pc, uptr *sp, uptr *bp) {
ucontext_t *ucontext = (ucontext_t*)context;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9638.30596.patch
Type: text/x-patch
Size: 1079 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150724/cad57253/attachment.bin>
More information about the llvm-commits
mailing list