From: Alexander Potapenko Date: Wed, 18 Aug 2021 12:33:14 +0000 (+0200) Subject: [tsan] Disable Trace.MemoryAccessSize on Mac X-Git-Tag: upstream/15.0.7~33624 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7338be0e6e8d09187f858d087df26914d201cb79;p=platform%2Fupstream%2Fllvm.git [tsan] Disable Trace.MemoryAccessSize on Mac According to comments at https://reviews.llvm.org/D107911, Trace.MemoryAccessSize fails on Mac buildbots. Because this test is newly introduced, and is the only user of the code added in that patch, disable the test on Mac till the problem is resolved. Differential Revision: https://reviews.llvm.org/D108294 --- diff --git a/compiler-rt/lib/tsan/tests/unit/tsan_trace_test.cpp b/compiler-rt/lib/tsan/tests/unit/tsan_trace_test.cpp index f9d9b64..25bc550 100644 --- a/compiler-rt/lib/tsan/tests/unit/tsan_trace_test.cpp +++ b/compiler-rt/lib/tsan/tests/unit/tsan_trace_test.cpp @@ -76,7 +76,14 @@ TEST(Trace, RestoreAccess) { run_in_thread(Thread::Func); } -TEST(Trace, MemoryAccessSize) { +#if SANITIZER_MAC +// Trace.MemoryAccessSize is currently failing on Mac. +// See https://reviews.llvm.org/D107911 for more details. +# define MAYBE_MemoryAccessSize DISABLED_MemoryAccessSize +#else +# define MAYBE_MemoryAccessSize MemoryAccessSize +#endif +TEST(Trace, MAYBE_MemoryAccessSize) { struct Thread { struct Params { uptr access_size, offset, size;