Merge pull request arsenm#14 from amin-jabri/tsan_osx

Reenable TSan on macOS as newer Xcode toolchains support it.
9fcc · Oct 13, 2017 · 6b9a849 · 6b9a849
2 parents c5055c4 + b719221
commit 6b9a849
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -0,0 +1,3 @@
+# out-of-source build top-level folders.
+build/
+_build/
diff --git a/cmake/FindTSan.cmake b/cmake/FindTSan.cmake
@@ -39,9 +39,10 @@ endif ()
 include(sanitize-helpers)
 
 if (SANITIZE_THREAD)
-    if (NOT ${CMAKE_SYSTEM_NAME} STREQUAL "Linux")
+  if (NOT ${CMAKE_SYSTEM_NAME} STREQUAL "Linux" AND
+      NOT ${CMAKE_SYSTEM_NAME} STREQUAL "Darwin")
         message(WARNING "ThreadSanitizer disabled for target ${TARGET} because "
-            "ThreadSanitizer is supported for Linux systems only.")
+          "ThreadSanitizer is supported for Linux systems and macOS only.")
         set(SANITIZE_THREAD Off CACHE BOOL
             "Enable ThreadSanitizer for sanitized targets." FORCE)
     elseif (NOT ${CMAKE_SIZEOF_VOID_P} EQUAL 8)