diff mbox

[doc] : Document -fomit-frame-pointer default change for 32bit Linux and Darwin targets.

Message ID AANLkTikNhiUqryEtC7UViNmhvmJjuX0_BhjgVm3qa1tM@mail.gmail.com
State New
Headers show

Commit Message

Uros Bizjak Aug. 15, 2010, 6:59 p.m. UTC 
Hello!

Attached patch adds documentation about -fomit-frame-pointer default
change on selected x86 targets.

2010-08-15  Uros Bizjak  <ubizjak@gmail.com>

	* doc/invoke.texi (-fomit-frame-pointer): Document that starting
	from GCC version 4.6, the default setting (when not optimizing
	for size) for 32-bit Linux x86 and 32-bit Darwin x86 targets has
	been changed to -fomit-frame-pointer.

Patch was tested by make info. OK for mainline?

Uros.

Comments

Gerald Pfeifer Aug. 15, 2010, 7:14 p.m. UTC | #1 
On Sun, 15 Aug 2010, Uros Bizjak wrote:
> Attached patch adds documentation about -fomit-frame-pointer default
> change on selected x86 targets.

Thanks for thinking of this, Uros!  Basically the same text looks good
for the GCC 4.6 release notes (http://gcc.gnu.org/gcc-4.6/changes.html)
I think, and I can apply it there for you.

> 	* doc/invoke.texi (-fomit-frame-pointer): Document that starting
> 	from GCC version 4.6, the default setting (when not optimizing

Is it starting "from" or "with"?  Native speakers welcome! :-)

> 	for size) for 32-bit Linux x86 and 32-bit Darwin x86 targets has
> 	been changed to -fomit-frame-pointer.

GNU/Linux or really Linux (that is, everything running a Linux kernel)?

+Starting from GCC version 4.6, the default setting (when not optimizing for
+size) for 32-bit Linux x86 and 32-bit Darwin x86 targets has been changed to
+@option{-fomit-frame-pointer}.  New behavior can be reverted back to
+@option{-fno-omit-frame-pointer} by configuring GCC with the
+@option{--enable-frame-pointer} configure option.

Same comments as for the ChangeLog apply here.


And I suggest to slightly change the last sentence a bit to read

  "The default can be reverted to @option{-fno-omit-frame-pointer} by 
  configuring..."

but that's more a question of style, and thus personal preference.

Gerald
diff mbox

Patch

Index: doc/invoke.texi
===================================================================
--- doc/invoke.texi	(revision 163266)
+++ doc/invoke.texi	(working copy)
@@ -5993,6 +5993,12 @@ 
 whether a target machine supports this flag.  @xref{Registers,,Register
 Usage, gccint, GNU Compiler Collection (GCC) Internals}.
 
+Starting from GCC version 4.6, the default setting (when not optimizing for
+size) for 32-bit Linux x86 and 32-bit Darwin x86 targets has been changed to
+@option{-fomit-frame-pointer}.  New behavior can be reverted back to
+@option{-fno-omit-frame-pointer} by configuring GCC with the
+@option{--enable-frame-pointer} configure option.
+
 Enabled at levels @option{-O}, @option{-O2}, @option{-O3}, @option{-Os}.
 
 @item -foptimize-sibling-calls