On Thu, Feb 23, 2017 at 04:31:34AM +1100, Richard Henderson wrote: > On 02/23/2017 01:22 AM, Stafford Horne wrote: > > > static inline int __atomic_add_unless(atomic_t *v, int a, int u) > > > { > > > int old, tmp; > > > > > > __asm__ __volatile__( > > > "1: l.lwa %0, 0(%2) \n" > > > " l.sfeq %0, %4 \n" > > > " l.bf 2f \n" > > > " l.nop \n" > > > " l.add %1, %0, %3 \n" > > You can move this add into the delay slot and drop the preceding nop.
Thanks, Thats right, also here the 2: label being after the l.nop can be applied. I should have thought about it. I made the change, Ill also fix/look again in the other places. -Stafford > > r~