This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [x86,PATCH] Additional fix for 57756.


Resend modified ChangeLog:

2013-11-20  Yuri Rumyantsev  <ysrumyan@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.tools@gmail.com>:
> On Tue, Nov 19, 2013 at 5:31 AM, Yuri Rumyantsev <ysrumyan@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  <ysrumyan@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.
>
> Please add "PR target/57756" in ChangeLog entry.
>
> Thanks.
>
> --
> H.J.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]