[Asterisk-code-review] Build System: Add support for atomic built-in operators. (asterisk[master])

[Corey Farrell]

Corey Farrell has posted comments on this change. ( https://gerrit.asterisk.org/8046 )

Change subject: Build System: Add support for __atomic built-in operators.
......................................................................


Patch Set 1:

Changes could also be added to spinlock.h but I'm not sure of the correct way to do that.  Since spinlock.h isn't included by anything I'd rather not touch it.

https://gcc.gnu.org/onlinedocs/gcc/_005f_005fatomic-Builtins.html

To preemptively answer about the preference order having __atomic first, the gcc docs for __sync are labeled "Legacy __sync Built-in Functions for Atomic Memory Access", it says new code should use __atomic builtins instead.

https://gcc.gnu.org/onlinedocs/gcc/_005f_005fsync-Builtins.html


-- 
To view, visit https://gerrit.asterisk.org/8046
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I65b4feb02bae368904ed0fb03f585c05f50a690e
Gerrit-Change-Number: 8046
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Comment-Date: Thu, 25 Jan 2018 02:27:55 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180124/01fdf82b/attachment.html>