[PATCH] D14604: [tsan] Filter OS X architectures for unit testing

Kuba Brecka via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 12 01:56:43 PST 2015


kubabrecka added inline comments.

================
Comment at: lib/tsan/tests/CMakeLists.txt:38
@@ +37,3 @@
+  if(APPLE)
+    darwin_filter_host_archs(TSAN_SUPPORTED_ARCH TSAN_TEST_ARCH)
+  endif()
----------------
glider wrote:
> I'm not familiar with CMake infrastructure enough but I think darwin_filter_host_archs isn't the right way to say "remove ARM64 from the list".
> Can we just explicitly set TSAN_TEST_ARCH to x86_64 on Darwin?
`darwin_filter_host_archs` is already used for this purpose in `asan/tests/` and `sanitizer_common/tests/`.  It's actually doing a little more work, namely keeping both x86_64 and x86_64h in case your system is a Haswell+ machine.

But I'm actually completely fine with just doing `set(TSAN_TEST_ARCH x86_64)` here, if you still want me to.


http://reviews.llvm.org/D14604





More information about the llvm-commits mailing list