[PATCH] bitops: fix issue in parallel atomic tests
Mattias Rönnblom
hofors at lysator.liu.se
Mon Oct 14 16:33:31 CEST 2024
On 2024-10-14 16:16, David Marchand wrote:
> On Sun, Oct 13, 2024 at 5:37 PM Morten Brørup <mb at smartsharesystems.com> wrote:
>>> I'm far from sure it's a compiler bug. Just look at the base rate: how
>>> often does the code you just wrote fail because of a bug in your code,
>>> and how often is the root cause to be found in the compiler or the
>>> standard libraries.
>>>
>>
>> A strong argument for rootcausing exactly what the specific compiler gets wrong when compiling the code triggering the error.
>>
>
> I agree that root causing it is preferrable.
> I merged this change in any case to resolve the false positives in CI.
>
>
The change is reworking some pretty awkward, potentially incorrect (if
GCC 12.2 is right and I am wrong) code. So it should be merged in any
case. The reason the reference word was always 64 bit was just an
oversight on my part.
More information about the dev
mailing list