[PATCH] D26162: [tsan] Enable the tsan/libcxx/ testcase(s) on Darwin

Kuba Brecka via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 13:44:28 PDT 2016


kubabrecka created this revision.
kubabrecka added a reviewer: dvyukov.
kubabrecka added subscribers: llvm-commits, zaks.anna.
kubabrecka set the repository for this revision to rL LLVM.
kubabrecka added a project: Sanitizers.

Apparently, the `std_shared_ptr.cc` testcase works fine on Darwin, even without the instrumented libcxx.  Let's enable it.


Repository:
  rL LLVM

https://reviews.llvm.org/D26162

Files:
  test/tsan/libcxx/lit.local.cfg


Index: test/tsan/libcxx/lit.local.cfg
===================================================================
--- test/tsan/libcxx/lit.local.cfg
+++ test/tsan/libcxx/lit.local.cfg
@@ -5,6 +5,8 @@
 
 root = getRoot(config)
 
-if not root.has_libcxx:
+# Only run if we an instrumented libcxx.  On Darwin, run always (we have
+# interceptors to support the system-provided libcxx).
+if not root.has_libcxx and root.host_os != 'Darwin':
   config.unsupported = True
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26162.76468.patch
Type: text/x-patch
Size: 460 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161031/99b995fb/attachment.bin>


More information about the llvm-commits mailing list