Re: [PATCH, i386, testsuite] FMA intrinsics

[Uros Bizjak <ubizjak at gmail dot com>]

On Wed, Aug 24, 2011 at 2:39 PM, Ilya Tocar <tocarip.intel@gmail.com> 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.