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: PATCH: Turn on -fomit-frame-pointer by default for 32bit Linux/x86


On Thu, Aug 12, 2010 at 6:19 PM, H.J. Lu <hjl.tools@gmail.com> wrote:

> 2010-08-12 ?H.J. Lu ?<hongjiu.lu@intel.com>
> ? ? ? ? ? ?Uros Bizjak ?<ubizjak@gmail.com>
>
> ? ? ? ?* config.gcc: Handle --enable-frame-pointer.
>
> ? ? ? ?* configure.ac: Add --enable-frame-pointer.
> ? ? ? ?* configure: Regenerated.
>
> ? ? ? ?* config/i386/i386.c (USE_IX86_FRAME_POINTER): Default to 0.
> ? ? ? ?(override_options): Enable -fomit-frame-pointer for 32bit code
> ? ? ? ?if compiling for TARGET_MACHO and not optimizing for size
> ? ? ? ?unless configured with --enable-frame-pointer. ?Enable
> ? ? ? ?-fasynchronous-unwind-tables unless configured with
> ? ? ? ?--enable-frame-pointer. ?Enable -maccumulate-outgoing-args
> ? ? ? ?by default unless configured with --enable-frame-pointer.
>

Please change the ChangeLog text to something like:

If not configured with --enable-frame-pointer, enable
-fomit-frame-pointer (but not for TARGET_MACHO or when optimizing for
size), enable -fasynchronous-unwind-tables and
-maccumulate-outgoing-args by default.

Thanks,
Uros.


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