Atomic ops: Fix atomic_add_uint32 and atomic_sub_uint32 in Windows
authorBenoit Bolsee <benoit.bolsee@online.be>
Thu, 9 Jun 2016 22:00:33 +0000 (00:00 +0200)
committerBenoit Bolsee <benoit.bolsee@online.be>
Sat, 11 Jun 2016 20:15:25 +0000 (22:15 +0200)
    The assembler version in Windows used to return the previous value
    of the variable while all the other versions return the new value.
    This is now fixed for consistency.
    Note: this bug had no effect on blender because no part of the code
    use the return value of these functions, but the future BGE DeckLink
    module makes use of it to implement reference counter.

intern/atomic/intern/atomic_ops_unix.h

index bf54750ea0deaac8e52cf9e22e562e6c0cab5c73..55c00024244eb6553ecc04d7e96e9cf07e119d42 100644 (file)
@@ -129,23 +129,24 @@ ATOMIC_INLINE uint32_t atomic_cas_uint32(uint32_t *v, uint32_t old, uint32_t _ne
 #elif (defined(__i386__) || defined(__amd64__) || defined(__x86_64__))
 ATOMIC_INLINE uint32_t atomic_add_uint32(uint32_t *p, uint32_t x)
 {
+       uint32_t ret = x;
        asm volatile (
            "lock; xaddl %0, %1;"
-           : "+r" (x), "=m" (*p) /* Outputs. */
+           : "+r" (ret), "=m" (*p) /* Outputs. */
            : "m" (*p) /* Inputs. */
            );
-       return x;
+       return ret+x;
 }
 
 ATOMIC_INLINE uint32_t atomic_sub_uint32(uint32_t *p, uint32_t x)
 {
-       x = (uint32_t)(-(int32_t)x);
+       ret = (uint32_t)(-(int32_t)x);
        asm volatile (
            "lock; xaddl %0, %1;"
-           : "+r" (x), "=m" (*p) /* Outputs. */
+           : "+r" (ret), "=m" (*p) /* Outputs. */
            : "m" (*p) /* Inputs. */
            );
-       return x;
+       return ret-x;
 }
 
 ATOMIC_INLINE uint32_t atomic_cas_uint32(uint32_t *v, uint32_t old, uint32_t _new)