Page MenuHomeFreeBSD

stdatomic: Only use clang atomics if __clang__ is defined
ClosedPublic

Authored by jhb on Jul 17 2024, 7:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 27, 2:55 PM
Unknown Object (File)
Mon, Sep 23, 9:35 PM
Unknown Object (File)
Mon, Sep 16, 5:12 AM
Unknown Object (File)
Sun, Sep 15, 10:14 PM
Unknown Object (File)
Sun, Sep 8, 5:25 AM
Unknown Object (File)
Sun, Sep 8, 3:16 AM
Unknown Object (File)
Sat, Sep 7, 7:46 AM
Unknown Object (File)
Aug 31 2024, 4:13 AM
Subscribers
None

Details

Summary

GCC 14 defines has_extension(c_atomic) but does not support
c11_atomic_*, so require clang for the CLANG_ATOMICS case.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Jul 17 2024, 7:14 PM
jhb created this revision.

We likely don't need sync atomics given the scopeof where it would be needed... but im not sure.

This revision is now accepted and ready to land.Jul 17 2024, 7:55 PM

I think as part of your cleanup of cdefs.h, we can probably drop the pre-GCC 4.7 path entirely. We already don't support versions of GCC that old in cdefs.h now I think?