On 5/24/23 11:32, Richard Henderson wrote:
Changes for v3: * Most of the v2 patch set merged, except x86_64 atomic128-ldst.h, which failed testing with clang-11 with debian 11.* New patch to change __int128_t detection. * This in turn enabled CONFIG_ATOMIC128, which was not ideal. This clang bug/mis-feature of using a cmpxchg sequence for implementing __atomic_load_n was already noted for aarch64, so I should have expected it would also be true for x86_64. Given that I am adding inline assembly for CPUINFO_ATOMIC_VMOVDQA anyway, this isn't a big deal, but I did need to adjust the ifdefs. r~ Richard Henderson (2): meson: Split test for __int128_t type from __int128_t arithmetic qemu/atomic128: Add x86_64 atomic128-ldst.h meson.build | 15 +++-- host/include/x86_64/host/atomic128-ldst.h | 68 +++++++++++++++++++++++ include/qemu/int128.h | 4 +- 3 files changed, 80 insertions(+), 7 deletions(-) create mode 100644 host/include/x86_64/host/atomic128-ldst.h
Superseded by Message-Id: <20230526002334.1760495-1-richard.hender...@linaro.org> r~
