Re: [PATCH 08/11] x86: use asm .macro instead of cpp #define in entry_32.S

From: Jeremy Fitzhardinge
Date: Mon Feb 09 2009 - 13:35:09 EST


Tejun Heo wrote:
Impact: cleanup

Use .macro instead of cpp #define where approriate. This cleans up
code and will ease future changes.

I agree with this in principle - cpp macros are indeed very ugly - but in practise I've found asm macros to be very hard to work with. When things go wrong, the gas error messages are completely useless (it only reports the error on the line where the macro was used, without any detail about where in the macro the problem lies), and as far as I know, there's no way to see the source with macros expanded.

J
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/