[Mesa-dev] [PATCH 2/2] util/u_atomic: Add implementation of __sync_val_compare_and_swap_8

Grazvydas Ignotas notasas at gmail.com
Fri Sep 15 09:14:27 UTC 2017


On Fri, Sep 15, 2017 at 12:04 PM, Grazvydas Ignotas <notasas at gmail.com> wrote:
> It was because on some platforms ./configure test looking for
> __atomic_* passes because it's a compile (not link) test, even when
> the compiler inserts library calls instead of actual implementation.
> It looks like the correct solution is to link against libatomic (which
> I was not aware of at that time) and drop these from u_atomic.c (amend
> it to 1/2?).

Whoops somehow I misread 1/2 as adding -latomic , so ignore the
"amend" part of this comment. It probably came as a result of thinking
about bug 102573 and some lack of sleep.

>
> GraÅžvydas
>
>
> On Thu, Sep 14, 2017 at 10:08 PM, Matt Turner <mattst88 at gmail.com> wrote:
>> Grazvydas,
>>
>> I noticed that there are some __atomic functions in this file, but I'm
>> not sure what they do or why they're necessary. Remind me?
>>
>> Thanks,
>> Matt


More information about the mesa-dev mailing list