diff mbox

[doc] Correct optimisation levels documentation for -fstore-merging

Message ID 588879F6.2010905@foss.arm.com
State New
Headers show

Commit Message

Kyrill Tkachov Jan. 25, 2017, 10:12 a.m. UTC 
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.tkachov@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.tkachov@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 mbox

Patch

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