On 23/01/17 23:39, Jeff Law wrote:
On 01/23/2017 10:28 AM, Kyrill Tkachov wrote:
Hi all,

I had forgotten to update the -fstore-merging documentation from a
previous iteration of the pass
and it says that it's enabled at -O and higher. The option is in fact
enabled at -O2 and higher, as well as -Os.
This patch clarifies that.

Is this ok? Or is there a more preferred style of listing optimisation
levels?

Thanks,
Kyrill

2016-01-23  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>

    * doc/invoke.texi (-fstore-merging): Correct default optimization
    levels at which it is enabled.
I think you also need to remove -fstore-merging from list of options turned on 
by -O:

@option{-O} turns on the following optimization flags:
[ ... ]
-fstore-merging @gol


And instead add it to the list of options enabled at -O2 and higher which 
immediately follows.

OK with those changes.

Thanks Jeff, Sandra.
I've done that. Committing this version to trunk.

Kyrill

2016-01-25  Kyrylo Tkachov  <kyrylo.tkac...@arm.com>

    * doc/invoke.texi (-fstore-merging): Correct default optimization
    levels at which it is enabled.
    (-O): Move -fstore-merging from list to...
    (-O2): ... Here.


jeff

diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 45af80c..59ab394 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -7012,7 +7012,6 @@ compilation time.
 -fsplit-wide-types @gol
 -fssa-backprop @gol
 -fssa-phiopt @gol
--fstore-merging @gol
 -ftree-bit-ccp @gol
 -ftree-ccp @gol
 -ftree-ch @gol
@@ -7072,6 +7071,7 @@ also turns on the following optimization flags:
 -frerun-cse-after-loop  @gol
 -fsched-interblock  -fsched-spec @gol
 -fschedule-insns  -fschedule-insns2 @gol
+-fstore-merging @gol
 -fstrict-aliasing -fstrict-overflow @gol
 -ftree-builtin-call-dce @gol
 -ftree-switch-conversion -ftree-tail-merge @gol
@@ -8342,7 +8342,7 @@ early.  This flag is enabled by default at @option{-O} and higher.
 Perform merging of narrow stores to consecutive memory addresses.  This pass
 merges contiguous stores of immediate values narrower than a word into fewer
 wider stores to reduce the number of instructions.  This is enabled by default
-at @option{-O} and higher.
+at @option{-O2} and higher as well as @option{-Os}.
 
 @item -ftree-ter
 @opindex ftree-ter

Reply via email to