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

Matt Turner mattst88 at gmail.com
Fri Sep 15 16:39:10 UTC 2017


On Fri, Sep 15, 2017 at 2:04 AM, 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?).

Yes, I think I agree. I really like your weak symbol idea, but I don't
think we can be reasonably sure that we have implemented every
operation we may use -- and we won't find out until distributions
start building Mesa at release time.


More information about the mesa-dev mailing list