From: LLVM Packaging Team Date: Thu, 11 Jan 2024 13:15:09 +0000 (+0100) Subject: libcxx-test-fix-lockfree-test-for-i386 X-Git-Tag: archive/raspbian/1%19.1.3-1+rpi1~2^2~51 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=8d63d23eb0799aeafb1082e9f47d3f0405ab5489;p=llvm-toolchain-19.git libcxx-test-fix-lockfree-test-for-i386 Lock is_always_lock free test fails on i386 because std::atomic is aligned Lock is_always_lock free test fails on i386 because std::atomic is aligned to 8 bytes while long long is aligned to 4 bytes. clang can't generate inline code for unaligned 8 byte atomics even tough instruction set and gcc support it. That makes it expected thaqt ATOMIC_LLONG_LOCK_FREE and std::atomic::is_always_lock_free don't match on i386. Correct test for std::atomic is to check if target cpu support cmpxchg8 instruction. To set instruction support one can check __GCC_HAVE_SYNC_COMPARE_AND_SWAP_8 define. Bug: https://llvm.org/bugs/show_bug.cgi?id=19355 Gbp-Pq: Topic libcxx Gbp-Pq: Name libcxx-test-fix-lockfree-test-for-i386.patch --- diff --git a/libcxx/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp b/libcxx/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp index b2d83f0a6f..f4fa0fecc0 100644 --- a/libcxx/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp +++ b/libcxx/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp @@ -18,6 +18,14 @@ #include "test_macros.h" +#if defined(__i386__) && defined(__GCC_HAVE_SYNC_COMPARE_AND_SWAP_8) +/* Fix for clang setting __GCC_ATOMIC_LLONG_LOCK_FREE incorecctly for x86 + * https://llvm.org/bugs/show_bug.cgi?id=19355 + */ +#undef ATOMIC_LLONG_LOCK_FREE +#define ATOMIC_LLONG_LOCK_FREE 2 +#endif + template void checkAlwaysLockFree() { if (std::atomic::is_always_lock_free) {