PATCH: Use __builtin_bswap32/64 for GCC >= 4.2

H.J. Lu hjl.tools@gmail.com
Thu Apr 5 22:42:00 GMT 2012


On Thu, Apr 5, 2012 at 2:06 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, Apr 5, 2012 at 2:01 PM, Roland McGrath <roland@hack.frob.com> wrote:
>> Any static inlines need to use __attribute__ ((__unused__)).
>
> __attribute__ ((__unused__)) may not work for all compilers.
> Should we use __extern_inline since it is a user visible header
> file?

__extern_inline  doesn't work with all compilers.  I checked
all user-visible glibc header files with inline functions.  They
are declared with static __inline, __extern_always_inline or
__extern_inline.  None of them use __attribute__ ((__unused__)).
If static __inline should be changed, I believe it should be done
with a separate patch.


-- 
H.J.



More information about the Libc-alpha mailing list