svn commit: r232721 - head/sys/x86/include

Tijl Coosemans tijl at freebsd.org
Fri Mar 9 16:11:57 UTC 2012


On Friday 09 March 2012 16:47:07 Ed Schouten wrote:
> * Tijl Coosemans <tijl at 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 228 bytes
Desc: This is a digitally signed message part.
Url : http://lists.freebsd.org/pipermail/svn-src-all/attachments/20120309/b279c22e/attachment.pgp


More information about the svn-src-all mailing list