This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Turn on -fomit-frame-pointer by default for 32bit Linux/x86
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: Richard Guenther <richard dot guenther at gmail dot com>, Andrew Pinski <pinskia at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 3 Aug 2010 11:18:10 +0200
- Subject: Re: PATCH: Turn on -fomit-frame-pointer by default for 32bit Linux/x86
- References: <AANLkTinR6uC2_Vqd+_R17KO+_zL0+FwJFPYPLxu0Jkdc@mail.gmail.com> <AANLkTimfR_+Xo1uB3M7j0Qo7BL9b9rO7Wcv1cffuGDWN@mail.gmail.com>
On Mon, Aug 2, 2010 at 7:54 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
>> Here are the updated patch and spread sheet for SPEC CPU 2000/2006 on
>> Intel Core i7 which shows that -fomit-frame-pointer -fasynchronous-unwind-tables
>> improves performance by up to 12% at -O2 and 8% at -O3. For -Os, it improves
>> performance by up to 20%, but also increases code by up to 36%.
>>
>> This patch turns on -fomit-frame-pointer -fasynchronous-unwind-tables only if
>> -Os isn't used.
>>
>> Tested on Linux/ia32 and Linux/x86-64 with -m32. OK for trunk?
>
> Any comments on this patch. ?Any objections?
IMO, this should be approved by Release Managers, since this decision
affects distributions and their users.
I have no _technical_ objections to this patch, but I won't approve it
either due to the reason above.
Thanks,
Uros.