[PATCH] D14947: [tsan] Port tsan_rtl_amd64.S to OS X to add support for setjmp/longjmp

Kuba Brecka via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 26 08:23:10 PST 2015


kubabrecka added inline comments.

================
Comment at: lib/tsan/rtl/tsan_rtl_amd64.S:297
@@ +296,3 @@
+#else
+  jmp ASM_TSAN_SYMBOL(setjmp)
+#endif
----------------
dvyukov wrote:
> Will it work with pie binaries?.. are there pie binaries on mac?
Yes, actually, PIE binaries are the default on OS X (at least for dylibs).


http://reviews.llvm.org/D14947





More information about the llvm-commits mailing list