diff mbox

[doc] obvious partial fix for other/48254

Message ID AANLkTimszxein5V3pH4=nnuex-73c6LMAkAL0somwByN@mail.gmail.com
State New
Headers show

Commit Message

Jonathan Wakely March 23, 2011, 10:03 p.m. UTC
Committed to 4.4 and 4.5 branches as obvious, will apply to 4.6 after
4.6.0 is released. The option's gone on the trunk so not relevant
there.

2011-03-23  Jonathan Wakely  <jwakely.gcc@gmail.com>

       PR other/48254
       * doc/invoke.texi (-fipa-struct-reorg): Fix typo.

Comments

Richard Biener March 24, 2011, 10:12 a.m. UTC | #1
On Wed, Mar 23, 2011 at 11:03 PM, Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
> Committed to 4.4 and 4.5 branches as obvious, will apply to 4.6 after
> 4.6.0 is released. The option's gone on the trunk so not relevant
> there.

If you apply anything to 4.6 it should probably simply change
the docs to say "This option does nothing.".  Or maybe don't
pretend it exists in any useful form and remove it completely.

Richard.

> 2011-03-23  Jonathan Wakely  <jwakely.gcc@gmail.com>
>
>       PR other/48254
>       * doc/invoke.texi (-fipa-struct-reorg): Fix typo.
>
>
>
> Index: doc/invoke.texi
> ===================================================================
> --- doc/invoke.texi     (revision 171365)
> +++ doc/invoke.texi     (working copy)
> @@ -6121,7 +6121,7 @@ Enabled by default at @option{-O} and hi
>  @opindex fipa-struct-reorg
>  Perform structure reorganization optimization, that change C-like structures
>  layout in order to better utilize spatial locality.  This transformation is
> -affective for programs containing arrays of structures.  Available in two
> +effective for programs containing arrays of structures.  Available in two
>  compilation modes: profile-based (enabled with @option{-fprofile-generate})
>  or static (which uses built-in heuristics).  Require @option{-fipa-type-escape}
>  to provide the safety of this transformation.  It works only in whole program
>
Jonathan Wakely March 24, 2011, 10:14 a.m. UTC | #2
On 24 March 2011 10:12, Richard Guenther wrote:
> On Wed, Mar 23, 2011 at 11:03 PM, Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
>> Committed to 4.4 and 4.5 branches as obvious, will apply to 4.6 after
>> 4.6.0 is released. The option's gone on the trunk so not relevant
>> there.
>
> If you apply anything to 4.6 it should probably simply change
> the docs to say "This option does nothing.".  Or maybe don't
> pretend it exists in any useful form and remove it completely.

Ah, I didn't realise that was the case, thanks!
diff mbox

Patch

Index: doc/invoke.texi
===================================================================
--- doc/invoke.texi     (revision 171365)
+++ doc/invoke.texi     (working copy)
@@ -6121,7 +6121,7 @@  Enabled by default at @option{-O} and hi
 @opindex fipa-struct-reorg
 Perform structure reorganization optimization, that change C-like structures
 layout in order to better utilize spatial locality.  This transformation is
-affective for programs containing arrays of structures.  Available in two
+effective for programs containing arrays of structures.  Available in two
 compilation modes: profile-based (enabled with @option{-fprofile-generate})
 or static (which uses built-in heuristics).  Require @option{-fipa-type-escape}
 to provide the safety of this transformation.  It works only in whole program