[FFmpeg-cvslog] lavu/atomic: add support for the new memory model aware gcc built-ins

James Almer git at videolan.org
Wed Oct 29 18:10:09 CET 2014

ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Mon Oct 27 22:48:08 2014 -0300| [faa9d2982969c999ab0e443a226eff116f7f8e4b] | committer: James Almer

lavu/atomic: add support for the new memory model aware gcc built-ins

__sync built-ins are considered legacy and will be deprecated.
These new memory model aware built-ins have been available since GCC 4.7.0

Use them by default when available except for __atomic_compare_exchange_n(),
which is slower, and is instead implemented as a fallback for when and if gcc
removes the legacy __sync built-ins.

Reviewed-by: Michael Niedermayer <michaelni at gmx.at>
Signed-off-by: James Almer <jamrial at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=faa9d2982969c999ab0e443a226eff116f7f8e4b

 configure              |    4 +++-
 libavutil/atomic_gcc.h |   17 +++++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/configure b/configure
index 0a1b154..1913cb5 100755
--- a/configure
+++ b/configure
@@ -1602,6 +1602,7 @@ ARCH_FEATURES="
+    atomic_compare_exchange
@@ -2028,7 +2029,7 @@ simd_align_16_if_any="altivec neon sse"
 symver_if_any="symver_asm_label symver_gnu_asm"
 # threading support
+atomics_gcc_if_any="sync_val_compare_and_swap atomic_compare_exchange"
 atomics_suncc_if="atomic_cas_ptr machine_rw_barrier"
@@ -4681,6 +4682,7 @@ if ! disabled network; then
 check_builtin atomic_cas_ptr atomic.h "void **ptr; void *oldval, *newval; atomic_cas_ptr(ptr, oldval, newval)"
+check_builtin atomic_compare_exchange "" "int *ptr, *oldval; int newval; __atomic_compare_exchange_n(ptr, oldval, newval, 0, __ATOMIC_SEQ_CST, __ATOMIC_SEQ_CST)"
 check_builtin machine_rw_barrier mbarrier.h "__machine_rw_barrier()"
 check_builtin MemoryBarrier windows.h "MemoryBarrier()"
 check_builtin sarestart signal.h "SA_RESTART"
diff --git a/libavutil/atomic_gcc.h b/libavutil/atomic_gcc.h
index 2bb43c3..5f9fc49 100644
--- a/libavutil/atomic_gcc.h
+++ b/libavutil/atomic_gcc.h
@@ -28,27 +28,40 @@
 #define avpriv_atomic_int_get atomic_int_get_gcc
 static inline int atomic_int_get_gcc(volatile int *ptr)
+    return __atomic_load_n(ptr, __ATOMIC_SEQ_CST);
     return *ptr;
 #define avpriv_atomic_int_set atomic_int_set_gcc
 static inline void atomic_int_set_gcc(volatile int *ptr, int val)
+    __atomic_store_n(ptr, val, __ATOMIC_SEQ_CST);
     *ptr = val;
 #define avpriv_atomic_int_add_and_fetch atomic_int_add_and_fetch_gcc
 static inline int atomic_int_add_and_fetch_gcc(volatile int *ptr, int inc)
+    return __atomic_add_fetch(ptr, inc, __ATOMIC_SEQ_CST);
     return __sync_add_and_fetch(ptr, inc);
 #define avpriv_atomic_ptr_cas atomic_ptr_cas_gcc
 static inline void *atomic_ptr_cas_gcc(void * volatile *ptr,
                                        void *oldval, void *newval)
     // armcc will throw an error if ptr is not an integer type
     volatile uintptr_t *tmp = (volatile uintptr_t*)ptr;
@@ -56,6 +69,10 @@ static inline void *atomic_ptr_cas_gcc(void * volatile *ptr,
     return __sync_val_compare_and_swap(ptr, oldval, newval);
+    __atomic_compare_exchange_n(ptr, &oldval, newval, 0, __ATOMIC_SEQ_CST, __ATOMIC_SEQ_CST);
+    return oldval;
 #endif /* AVUTIL_ATOMIC_GCC_H */

More information about the ffmpeg-cvslog mailing list