[PATCH 10/11] powerpc: Add helper functions for synthesising instructions at runtime
Segher Boessenkool
segher at kernel.crashing.org
Wed Oct 19 22:32:55 EST 2005
> I agree with you, Segher's code is obfuscated. Your current coding
> is much easier to read for someone that doesn't know the instruction
> bitfield layout by heart. The saved instructions on doing it Segher's
> way are not critical since it's not during a runtime hot path that this
> is called. Go for readability instead.
Well I think it is much more readable, we'll just have to
disagree here. That's just fine, it's not my code :-)
> Actually documenting Segher's code so it's as easy to understand will
> make it just as large as your current one (i.e. your 7 lines, his 3
> lines).
Nah, it removes more comments than it would add ;-)
Segher
More information about the Linuxppc64-dev
mailing list