Messages in this thread | | | Date | Tue, 20 May 2008 17:55:38 -0700 (PDT) | From | Linus Torvalds <> | Subject | Re: [PATCH] or51132.c: unaligned |
| |
On Wed, 21 May 2008, Al Viro wrote:
> On Tue, May 20, 2008 at 05:41:12PM -0700, Harvey Harrison wrote: > > > > + return buf[0] | (buf[1] << 8); > > > > return get_unaligned_le16(buf); > > And the point of that would be?
Perhaps better code generation?
I didn't look what
buf[0] | (buf[1] << 8)
generates, but on x86, get_unaligned_le16() should just boil down to
*(unsigned short *)buf
which is certainly going to mostly generate better code (smaller, faster) than at least the most likely and straigthforward of the former.
(Just checked. Gcc is not smart enough to make those two loads plus a shift be one word load. At least not my version)
Linus
| |