[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