[PATCH] D12841: [LLVMdev] Compiler-RT - Enabling ThreadSanitizer on PPC64(BE/LE) platforms

Bill Schmidt via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 15 14:18:18 PDT 2015


wschmidt added inline comments.

================
Comment at: test/tsan/mmap_large.cc:19
@@ -18,1 +18,3 @@
   const size_t kLog2Size = 32;
+#elif defined(__powerpc)
+  const size_t kLog2Size = 39;
----------------
Looks like you missed addressing this comment.  __powerpc is 32-bit PowerPC, and there's no intent to support TSan for that, that I'm aware of.


http://reviews.llvm.org/D12841





More information about the llvm-commits mailing list