[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