yep but my change in the file is platform specific for ARM and ARM ARCH 6 (raspberrypi processor)

the change is place into a 
ifdef __arm__ && __ARM_ARCH_6__
I hope that will have an effect ^^.
> sqCompareAndSwapRes was badly code for arm, I just change it.

> __sync_val_compare_and_swap(&(var), (old), (new))
becoming
> res = __sync_val_compare_and_swap(&(var), (old), (new))
is obvious if you compare to another implementation of sqCompareAndSwapRes (for other platform).
We can review that tomorrow together if you come to the lab. 
It is not a emergency to merge it.




On 13 Feb 2014, at 19:45, Camillo Bruni <notifications@github.com> wrote:

> In platforms/Cross/vm/sqAtomicOps.h:
> 
> > @@ -238,7 +238,7 @@
> >  	__sync_bool_compare_and_swap(&(var), (old), (new))
> >  
> >  # define sqCompareAndSwapRes(var,old,new,res) \
> > -	__sync_val_compare_and_swap(&(var), (old), (new))
> That's the only change I'd see could have side-effects...
> 
> —
> Reply to this email directly or view it on GitHub.
>