2011/8/24 Jakub Jelinek <ja...@redhat.com>:
> 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.
>
>        Jakub
>

Attachment: patch
Description: Binary data



Reply via email to