[patch] Add bit_FXSAVE as an alias for bit_FXSR, for gcc compat

Roman Divacky rdivacky at freebsd.org
Mon Dec 16 08:37:57 PST 2013


I am fine with that.

On Sat, Dec 14, 2013 at 10:48:05AM -0800, Nico Weber wrote:
> Hi,
> 
> it looks like gcc spells bit_FXSR like bit_FXSAVE by mistake, but they also
> don't want to fix this (
> http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02158.html ). This causes for
> example a compiler ifdef in breakpad ( https://breakpad.appspot.com/924002/).
> 
> To make things simpler, the attached patch adds support for gcc's spelling
> too.
> 
> Nico

> Index: lib/Headers/cpuid.h
> ===================================================================
> --- lib/Headers/cpuid.h	(revision 197300)
> +++ lib/Headers/cpuid.h	(working copy)
> @@ -79,6 +79,7 @@
>  #define bit_ACPI        0x00400000
>  #define bit_MMX         0x00800000
>  #define bit_FXSR        0x01000000
> +#define bit_FXSAVE      bit_FXSR
>  #define bit_SSE         0x02000000
>  #define bit_SSE2        0x04000000
>  #define bit_SS          0x08000000




More information about the cfe-commits mailing list