From: LLVM Packaging Team Date: Mon, 23 Sep 2024 11:23:31 +0000 (+0200) Subject: libcxx-test-fix-lockfree-test-for-i386 X-Git-Tag: archive/raspbian/1%18.1.8-12+rpi1~59 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=0dffb89bf1cc8baccb6bf7065a8add037de20073;p=llvm-toolchain-18.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 6d6e6477bc..1e5851b9f5 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) {