On Wed, Aug 24, 2011 at 2:39 PM, Ilya Tocar <[email protected]> wrote:
>>>> Also, why is fmaintrin.h including immintrin.h? You can't include
>>>> fmaintrin.h
>>>> directly and x86intrin.h has already included it before including
>>>> fmaintrin.h.
>>> Makes sense. Removed it.
>>
>> Do we need to add -mfma to gcc.target/i386/sse-{12,13,14,22,23}.c and
>> g++.dg/other/i386-{2,3}.C to activate and test new intrinsics?
>>
>> Uros.
>>
> I think yes, otherwise fmaintrin.h won't be included and intrinsics
> won't be tested.
> Added -mfma to those tests.
The patch is OK for mainline with the proper ChangeLog.
Thanks,
Uros.
