From: Ivan Maidanski Date: Sat, 17 Aug 2013 09:58:59 +0000 (+0400) Subject: Fix ARM double-CAS operands width (GCC/Clang) X-Git-Tag: libatomic_ops-7_2e~5 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b60ce287b2fdd12a522932f5707b320cba181904;p=libatomic_ops Fix ARM double-CAS operands width (GCC/Clang) (Apply commit a81940c from 'master' branch.) * src/atomic_ops/sysdeps/gcc/arm.h (AO_double_compare_and_swap): Swap assembly code operands ("new_val" and "addr") to prevent Clang3.3 warning about operand truncation. Conflicts: src/atomic_ops/sysdeps/gcc/arm.h --- diff --git a/src/atomic_ops/sysdeps/gcc/arm.h b/src/atomic_ops/sysdeps/gcc/arm.h index 1c5af93..0aa5e7c 100644 --- a/src/atomic_ops/sysdeps/gcc/arm.h +++ b/src/atomic_ops/sysdeps/gcc/arm.h @@ -295,9 +295,9 @@ AO_compare_and_swap(volatile AO_t *addr, AO_t old_val, AO_t new_val) if (tmp != old_val) break; __asm__ __volatile__( - " strexd %0, %2, %H2, [%3]\n" /* store new one if matched */ + " strexd %0, %3, %H3, [%2]\n" /* store new one if matched */ : "=&r"(result), "+m"(*addr) - : "r"(new_val), "r"(addr) + : "r" (addr), "r" (new_val) : "cc"); } while (result); return !result; /* if succeded, return 1 else 0 */