mirror of
https://git.proxmox.com/git/llvm-toolchain
synced 2025-10-31 12:27:10 +00:00
32 lines
1.4 KiB
Diff
32 lines
1.4 KiB
Diff
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<long long>::is_always_lock_free don't match on i386. Correct test
|
|
for std::atomic<long long> 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
|
|
|
|
Index: llvm-toolchain-snapshot_17~++20230517015648+0b6264738f3d/libcxx/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp
|
|
===================================================================
|
|
--- llvm-toolchain-snapshot_17~++20230517015648+0b6264738f3d.orig/libcxx/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp
|
|
+++ llvm-toolchain-snapshot_17~++20230517015648+0b6264738f3d/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 <typename T>
|
|
void checkAlwaysLockFree() {
|
|
if (std::atomic<T>::is_always_lock_free) {
|