From 3dbe0d9517935d549f2630728500b6f275750ec8 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Thu, 20 Mar 2003 09:03:23 +0000 Subject: [PATCH] Change atomic_exchange_and_add to return the old value. --- sysdeps/i386/i486/bits/atomic.h | 33 ++++++++++++--------------------- 1 file changed, 12 insertions(+), 21 deletions(-) diff --git a/sysdeps/i386/i486/bits/atomic.h b/sysdeps/i386/i486/bits/atomic.h index 80a49872e1..87739c6cc6 100644 --- a/sysdeps/i386/i486/bits/atomic.h +++ b/sysdeps/i386/i486/bits/atomic.h @@ -123,33 +123,24 @@ typedef uintmax_t uatomic_max_t; ({ __typeof (*mem) result; \ __typeof (value) addval = (value); \ if (sizeof (*mem) == 1) \ - { \ - __asm __volatile (LOCK "xaddb %b0, %1" \ - : "=r" (result), "=m" (*mem) \ - : "0" (addval), "1" (*mem)); \ - result += addval; \ - } \ + __asm __volatile (LOCK "xaddb %b0, %1" \ + : "=r" (result), "=m" (*mem) \ + : "0" (addval), "1" (*mem)); \ else if (sizeof (*mem) == 2) \ - { \ - __asm __volatile (LOCK "xaddw %w0, %1" \ - : "=r" (result), "=m" (*mem) \ - : "0" (addval), "1" (*mem)); \ - result += addval; \ - } \ + __asm __volatile (LOCK "xaddw %w0, %1" \ + : "=r" (result), "=m" (*mem) \ + : "0" (addval), "1" (*mem)); \ else if (sizeof (*mem) == 4) \ - { \ - __asm __volatile (LOCK "xaddl %0, %1" \ - : "=r" (result), "=m" (*mem) \ - : "0" (addval), "1" (*mem)); \ - result += addval; \ - } \ + __asm __volatile (LOCK "xaddl %0, %1" \ + : "=r" (result), "=m" (*mem) \ + : "0" (addval), "1" (*mem)); \ else \ { \ - __typeof (*mem) oldval; \ __typeof (mem) memp = (mem); \ do \ - result = (oldval = *memp) + addval; \ - while (__arch_compare_and_exchange_64_acq (memp, result, oldval)); \ + result = *memp; \ + while (__arch_compare_and_exchange_64_acq (memp, result + addval, \ + result)); \ } \ result; })