On Sun, Feb 23, 2014 at 06:10:42PM -0700, Adam Conrad wrote: > + > +-#define rd_atomic_add_prev(PTR,VAL) __sync_fetch_and_add(PTR,VAL) > +-#define rd_atomic_sub_prev(PTR,VAL) __sync_fetch_and_sub(PTR,VAL) > ++#define rd_atomic_add_prev(PTR,VAL) > __atmoic_fetch_add(PTR,VAL,__ATOMIC_SEQ_CST)
Except for this hilarious typo above that didn't show up in testing, as the code never actually calls this. Obviously, this should be __atomic, not __atmoic. Fixed now in the current Ubuntu version. > ++#define rd_atomic_sub_prev(PTR,VAL) > __atomic_fetch_sub(PTR,VAL,__ATOMIC_SEQ_CST) > + > +-#define rd_atomic_set(PTR,VAL) __sync_lock_test_and_set(PTR,VAL) > ++#define rd_atomic_set(PTR,VAL) __atomic_test_and_set(PTR,__ATOMIC_SEQ_CST) ... Adam -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
