On Thu Feb 6 18:10:41 2025 +0000, William Horvath wrote:
Also, note that these aren't C11 atomics, which would be `atomic_store` and `atomic_load`, without the `__` prefix. Underscore prefixed atomics are compiler builtins. I'm not particularly opposed to the idea, but without a convincing case for it I'm not sure further modification will be well received. (I'm not a maintainer, so maybe ask for others' opinion.)
Maybe when we drop support for GCC <8, then `__sync_` variants as well as open-coded inline asms could be dropped at once, although we may still need a stronger case for it. Note that the 32-bit atomics have been effectively tested for years. ReadNoFence64 was relatively new though. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7281#note_93985