2011/8/24 Jakub Jelinek : > 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 >