On Wed, Aug 24, 2011 at 11:31 AM, Ilya Tocar <[email protected]> wrote:
> 2011/8/24 Jakub Jelinek <[email protected]>:
>> On Wed, Aug 24, 2011 at 12:48:06PM +0400, Ilya Tocar wrote:
>>> Removed extra blank lines and pass tests through "indent".
>>
>> You haven't:
> Ah sorry only noticed one in sse.md.
>>
>> @@ -25113,6 +25125,9 @@ static const struct builtin_description
>> bdesc_multi_arg[] =
>> "__builtin_ia32_vfmaddpd256", IX86_BUILTIN_VFMADDPD256,
>> UNKNOWN, (int)MULTI_ARG_3_DF2 },
>>
>> +
>> +
>> +
>> { OPTION_MASK_ISA_FMA | OPTION_MASK_ISA_FMA4, CODE_FOR_fmaddsub_v4sf,
>> "__builtin_ia32_vfmaddsubps", IX86_BUILTIN_VFMADDSUBPS,
>> UNKNOWN, (int)MULTI_ARG_3_SF },
>>
>> 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.
