On Tue, Nov 19, 2013 at 2:45 PM, Yuri Rumyantsev <ysrum...@gmail.com> wrote: > Resend modified ChangeLog: > > 2013-11-20 Yuri Rumyantsev <ysrum...@gmail.com> > PR target/57756 > * config/i386/i386.c (ix86_option_override_internal): Add missed > argument prefix for 'ix86_fpmath'. > * config/i386/ssemath.h: Add missed definition of > TARGET_FPMATH_DEFAULT_P macros. > > 2013/11/19 H.J. Lu <hjl.to...@gmail.com>: >> On Tue, Nov 19, 2013 at 5:31 AM, Yuri Rumyantsev <ysrum...@gmail.com> wrote: >>> Hi All, >>> >>> We found out that compiler configured with '-fpmath=sse' option does >>> not generate scalar floating-point instructions present in the SSE >>> instruction set for generic32 that leads to performance degradation >>> for Fortran benchmarks using library functions in 32-bit mode. >>> >>> This simple fix was designed to cure this issue - definition of >>> TARGET_FPMATH_DEFAULT_P macros was missed in i386/ssemath.h. Also one >>> missed fix was done. >>> >>> Bootstrapping and regression testing were successful. >>> >>> Is it OK for trunk >>> >>> 2013-11-20 Yuri Rumyantsev <ysrum...@gmail.com> >>> >>> * config/i386/i386.c (ix86_option_override_internal): Add missed >>> argument prefix for 'ix86_fpmath'. >>> * config/i386/ssemath.h: Add missed definition of >>> TARGET_FPMATH_DEFAULT_P macros.
The patch looks OK to me, but let's also ask Sriraman for his opinion. Thanks, Uros.