On Sun, 2007-12-16 at 15:48 -0800, H. Peter Anvin wrote: > Harvey Harrison wrote: > > > > Do you have a stylistic preference between these two options: > > > > Option 1) Rely on CPP string constant concatenation > > > > // possibly include trailing space here to avoid remembering > > // leading space on the register names > > # define _ASM_INC "incl" > > > > static inline void local_inc(local_t *l) > > { > > __asm__ __volatile__( > > _ASM_INC " %0" > > :"+m" (l->a.counter)); > > } > > > > This is what I have used up to this point, except including the space in > the macro: > > #define _ASM_INC " incl " > > _ASM_INC "%0"
Not sure if you were just tossing a space on the end of my example, but do you also put a leading space on the " incl " in addition to the trailing space? Harvey -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/