Re: PATCH: Add bits/byteswap-16.h and use GCC builtin 32/64-bit bswap

[Chung-Lin Tang <cltang at codesourcery dot com>]

On 2012/4/7 03:40 AM, H.J. Lu wrote:
> 2012-04-06  H.J. Lu  <hongjiu.lu@intel.com>
> 
> 	* bits/byteswap.h (__bswap_16): Removed.
> 	Include <bits/byteswap-16.h> to get __bswap_16.
> 	(__bswap_32): Use __builtin_bswap32 for GCC >= 4.2.
> 	(__bswap_64): Use __builtin_bswap64 for GCC >= 4.2.
> 	* bits/byteswap-16.h: New file.

For SH, this triggered a build fail, due to those inline C functions
being added to a assembly file (the SH strlen.S use of endian.h) after a
complex include sequence.

Of course, there's nothing wrong with those __bswap* functions. It seems
that string/endian.h includes a bunch of stuff not usable anyways for
assembler, so here's trivial patch to correct this.

Thanks,
Chung-Lin

2012-04-24  Chung-Lin Tang  <cltang@codesourcery.com>

	* string/endian.h: Add !__ASSEMBLER__ condition for including
	conversion interfaces.

Attachment: asm.patch
Description: Text document