On Tue, Sep 19, 2000 at 07:13:31PM -0400, Alexander Viro wrote:
> Nice spotting, but bad fix, IMO. swab...() stuff is a perfect example of
> the dangerous use of macros. BTW, 2.4 has the same problem.
inlines usually generate worse code than macros (the gcc manual lies on that),
e.g. the register allocation is usually worse and CSE doesn't work that well.
Normally it makes not that much difference, but these macros are rather
performance critical.
Better would be to use statement blocks like
#define bla(x) ({ __u32 tmp__ = (x); ....; tmp__; })
I would prefer to fix the callers anyways, because it is clearer this way.
-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sat Sep 23 2000 - 21:00:21 EST