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

H.J. Lu hjl.tools@gmail.com
Sat Nov 22 09:19:00 GMT 2008


On Fri, Nov 21, 2008 at 9:34 PM, Chris Lattner <clattner@apple.com> wrote:
> On Nov 21, 2008, at 5:54 PM, H.J. Lu wrote:
>>>>
>>>> Let me clarify a few things:
>>>>
>>>> 1. We didn't add a new file. We just renamed gmmintrin.h to
>>>> immintrin.h.
>>>> 2. Including <immintrin.h> itself doesn't enable any ISA extensions.
>>>> 3. You need to use -mXXX to get the XXX extension from
>>>> <immintrin.h>. It applies to -mmmx, -msse, ... -mavx.
>>>
>>> Sounds good.
>>>
>>>> 4. We will add future new ISA intrinics to a new header file
>>>> and include it from <immintrin.h>.
>>>
>>> Sounds bad.  Why not just add everything to mmxintrin.h?  How is this
>>> morally different?
>>>
>>
>> Please follow
>>
>> http://gcc.gnu.org/ml/gcc/2008-11/msg00031.html
>
> Followed.  What is the point?  Can you please answer my question?  Thank
> you,
>

There is no mmxintrin.h. Did you mean "add everything to immintrin.h diirectly"?
The consensus is put AVX intrinsics into a separate file and include it in
immintrin.h. The same applies to future extensions.


-- 
H.J.



More information about the Gcc-patches mailing list