fix inconsistency in a_and and a_or argument types on x86[_64]
authorRich Felker <dalias@aerifal.cx>
Wed, 20 May 2015 04:17:35 +0000 (00:17 -0400)
committerRich Felker <dalias@aerifal.cx>
Wed, 20 May 2015 04:17:35 +0000 (00:17 -0400)
conceptually, and on other archs, these functions take a pointer to
int, but in the i386, x86_64, and x32 versions of atomic.h, they took
a pointer to void instead.

arch/i386/atomic.h
arch/x32/atomic.h
arch/x86_64/atomic.h

index 4fe7bded328befa977a175a54b162be00a1bd6d4..95fecbdcc51eb7faee9fff264388a3f7bc6d8bcd 100644 (file)
@@ -50,16 +50,16 @@ static inline int a_cas(volatile int *p, int t, int s)
        return t;
 }
 
-static inline void a_or(volatile void *p, int v)
+static inline void a_or(volatile int *p, int v)
 {
        __asm__( "lock ; orl %1, %0"
-               : "=m"(*(int *)p) : "r"(v) : "memory" );
+               : "=m"(*p) : "r"(v) : "memory" );
 }
 
-static inline void a_and(volatile void *p, int v)
+static inline void a_and(volatile int *p, int v)
 {
        __asm__( "lock ; andl %1, %0"
-               : "=m"(*(int *)p) : "r"(v) : "memory" );
+               : "=m"(*p) : "r"(v) : "memory" );
 }
 
 static inline int a_swap(volatile int *x, int v)
index 333098c399e95d065f51f9f9628663a906e7c887..b2014cc0da38bdb14dd605a0a32ac48cff181a95 100644 (file)
@@ -47,16 +47,16 @@ static inline int a_cas(volatile int *p, int t, int s)
        return t;
 }
 
-static inline void a_or(volatile void *p, int v)
+static inline void a_or(volatile int *p, int v)
 {
        __asm__( "lock ; or %1, %0"
-               : "=m"(*(int *)p) : "r"(v) : "memory" );
+               : "=m"(*p) : "r"(v) : "memory" );
 }
 
-static inline void a_and(volatile void *p, int v)
+static inline void a_and(volatile int *p, int v)
 {
        __asm__( "lock ; and %1, %0"
-               : "=m"(*(int *)p) : "r"(v) : "memory" );
+               : "=m"(*p) : "r"(v) : "memory" );
 }
 
 static inline int a_swap(volatile int *x, int v)
index 333098c399e95d065f51f9f9628663a906e7c887..b2014cc0da38bdb14dd605a0a32ac48cff181a95 100644 (file)
@@ -47,16 +47,16 @@ static inline int a_cas(volatile int *p, int t, int s)
        return t;
 }
 
-static inline void a_or(volatile void *p, int v)
+static inline void a_or(volatile int *p, int v)
 {
        __asm__( "lock ; or %1, %0"
-               : "=m"(*(int *)p) : "r"(v) : "memory" );
+               : "=m"(*p) : "r"(v) : "memory" );
 }
 
-static inline void a_and(volatile void *p, int v)
+static inline void a_and(volatile int *p, int v)
 {
        __asm__( "lock ; and %1, %0"
-               : "=m"(*(int *)p) : "r"(v) : "memory" );
+               : "=m"(*p) : "r"(v) : "memory" );
 }
 
 static inline int a_swap(volatile int *x, int v)