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