[PATCH] A new meta intrinsic header file for current and future x86 instrinsics.

Uros Bizjak ubizjak@gmail.com
Fri Dec 12 10:05:00 GMT 2008


On Thu, Dec 11, 2008 at 8:30 PM, H.J. Lu <hjl.tools@gmail.com> wrote:

>> Ok. I have attached a patch for x86intrin.h removing #ifdef
>> __AVX__/#endif around immintrin.h and added other header files based on
>> the instruction set defined.
>>
>> Bootstrapped and tested on amd64-linux. Is this Ok?
>>
>
> That is fine with me except
>
> +#ifndef _x86INTRIN_H_INCLUDED
> +#define _x86INTRIN_H_INCLUDED
>
> I think it should be _X86INTRIN_H_INCLUDED. But it isn't really
> important.
>
> BTW,  I can't approve it.

If this is finally the approach that everybody interested agreed to,
it is OK for mainline.

Thanks,
Uros.



More information about the Gcc-patches mailing list