On 07/02/2012 04:27 PM, Alexander Graf wrote: > > On 02.07.2012, at 23:26, Benjamin Herrenschmidt wrote: > >> On Mon, 2012-07-02 at 22:59 +0200, Alexander Graf wrote: >>> This should go into an #ifdef __ASSEMBLY__ block, right? :) >> >> We almost never use #ifdef __ASSEMBLY__, we use it the other way >> around, to prevent C stuff from being included in assembly. The >> other way around is legit since things might be used in inline asm >> for example. > > I'm not sure I want to see this bit of code used as is in inline asm > :). I don't even think it's possible, since it's a full statement. > Either way, it's safer with the guard.
Safer from what? It won't be expanded unless referenced. How is this better than putting ifdefs on #includes, prototypes, struct definitions, etc.? The ifdef is just clutter. -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev