Messages in this thread |  | | Date | Wed, 20 Sep 2000 01:22:50 +0200 | From | "Andi Kleen" <> | Subject | Re: [PATCH] abuse of macros in swab.h |
| |
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/
|  |