On Friday 09 March 2012 16:47:07 Ed Schouten wrote: > * Tijl Coosemans <t...@freebsd.org>, 20120309 12:48: >> Clean up x86 endian.h: > > Wouldn't it be possible to simply place these macros in <sys/endian.h> > and only let <machine/endian.h> define the assembly versions for the > non-constant cases?
Several things could be moved to sys, not just endian.h, but I'd like to finish merging amd64/i386 headers first. Also, for clang and gcc >= 4.3 it's better to use __builtin_bswapNN(x), because it can use movbe rather than mov+bswap when x is a memory operand.
signature.asc
Description: This is a digitally signed message part.
