<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 11, 2022, 3:03 AM Jin-oh Kang <<a href="mailto:jinoh.kang.kr@gmail.com">jinoh.kang.kr@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 11, 2022, 2:50 AM Rémi Bernon <<a href="mailto:rbernon@codeweavers.com" target="_blank" rel="noreferrer">rbernon@codeweavers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2/10/22 18:43, Rémi Bernon wrote:<br>
> On 2/10/22 18:36, Jin-oh Kang wrote:<br>
>><br>
>> We still support GCC 4.x (an in-support RHEL/CentOS release uses it), <br>
>> so I<br>
>> think we still need some wrappers around it. That and the GCC bug.<br>
>><br>
> <br>
> The __sync intrinsices seems to have be there since GCC 4.2, then <br>
> deprecated when __atomic where introduced in GCC 4.7. What's that bug?<br>
> <br>
<br>
Nvm I missed that part of the first patch.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><a href="https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81316" target="_blank" rel="noreferrer">https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81316</a> says the bug only affects the __ATOMIC_RELEASE case, but it affects __ATOMIC_SEQ_CST too on ARM64: <a href="https://godbolt.org/z/aTaaYaoGK" target="_blank" rel="noreferrer">https://godbolt.org/z/aTaaYaoGK</a></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Which is to say, this is still a meaningful fix for non-x86 architectures.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-- <br>
Rémi Bernon <<a href="mailto:rbernon@codeweavers.com" rel="noreferrer noreferrer" target="_blank">rbernon@codeweavers.com</a>><br>
<br>
</blockquote></div></div></div>
</blockquote></div></div></div>