[PATCH] Implement `internal_start/join_thread` on Mac OS X

Ismail Pazarbasi ismail.pazarbasi at gmail.com
Wed May 13 12:06:46 PDT 2015


================
Comment at: lib/sanitizer_common/sanitizer_mac.cc:340
@@ +339,3 @@
+void *internal_start_thread(void(*func)(void *arg), void *arg) {
+#if __IPHONE_OS_VERSION_MIN_REQUIRED >= 80000
+  // Start the thread with signals blocked, otherwise it can steal user signals.
----------------
glider wrote:
> Why is this disabled on certain iOS versions?
> Do we really need TSan on iOS? (note that the shadow memory overhead is quite big and the 32-bit address space may be too little).
I didn't try porting TSan to iOS, but I thought about trying it out for iPhone 6, which has 64-bit processor. I have no idea whether this is possible, as I didn't investigate it in detail.

We must get it working reliably on Mac OS X first. Therefore, it might be better to replace this line with:
  #if TARGET_OS_MAC && !TARGET_OS_IPHONE && !TARGET_IPHONE_SIMULATOR

http://reviews.llvm.org/D9638

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list