PATCH: Add x32 to x86_64/bits/sigcontext.h
H.J. Lu
hjl.tools@gmail.com
Wed May 16 20:44:00 GMT 2012
On Wed, May 16, 2012 at 1:06 PM, Roland McGrath <roland@hack.frob.com> wrote:
>> This is a public header file. Do all compilers support this?
>
> It should get an __extension__ prefix just to be pedantic. It was a GNU
> extension but is now in C11. I don't really care if we support compilers
> that don't handle the simplest basic GNU extensions that have been in GCC
> for over 20 years. If someone is concerned about that you could do:
>
> union
> {
> struct _fpstate *fpstate;
> __uint64_t __fpstate_word;
> } __fpstate_u;
> # define fpstate __fpstate_u.fpstate
>
> But that is worse in other ways.
>
> Unless somebody else has a strong objection with good rationale,
> I think we can use the anonymous union feature in public headers.
>
__extension__ union works for me. I will make the change.
Thanks.
--
H.J.
More information about the Libc-alpha
mailing list