diff mbox

[wwwdocs] Mention new store merging pass for GCC 7

Message ID 58887983.9010803@foss.arm.com
State New
Headers show

Commit Message

Kyrill Tkachov Jan. 25, 2017, 10:10 a.m. UTC 
On 24/01/17 13:44, Richard Earnshaw (lists) wrote:
> On 23/01/17 16:45, Gerald Pfeifer wrote:
>> Hi Kyrill,
>>
>> On Mon, 23 Jan 2017, Kyrill Tkachov wrote:
>>> This patch adds a short entry for the store merging pass in GCC 7 to the
>>> "General Optimizer Improvements" section.
>> +  <li>A new store merging pass has been added.  It will attempt to merge
>> +  constant stores to adjacent memory locations into fewer wider stores.
>> +  It can be enabled by using the <code>-fstore-merging</code> option
>> and is
>> +  enabled by default at the <code>-O2</code> optimization level or
>> +  higher.</li>
> I also think you should either use 'fewer, wider, stores' (with commas)
> or, if you don't like the commas: 'a smaller number of wider stores'.
>
> R.
>> Here I'd say "it attempts to merge" or, better yet, let's just say
>> "it merges".
>>
>> Let's not be too shy. :-)  (This still does not claim that it always
>> succeeds or anything like that, mind.)
>>
>> Okay, with that note taken into consideration.

Thanks, Gerald, Richard. I've done that, and also added that it's enabled at -Os as well.
Committing this to the repo.

Kyrill

>> Thanks,
>> Gerald
diff mbox

Patch

Index: htdocs/gcc-7/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-7/changes.html,v
retrieving revision 1.39
diff -U 3 -r1.39 changes.html
--- htdocs/gcc-7/changes.html	17 Jan 2017 21:26:31 -0000	1.39
+++ htdocs/gcc-7/changes.html	24 Jan 2017 13:45:11 -0000
@@ -40,11 +40,14 @@ 
 
 <!-- .................................................................. -->
 <h2 id="general">General Optimizer Improvements</h2>
-<!--
+
 <ul>
-  <li></li>
+  <li>A new store merging pass has been added.  It merges constant stores to
+  adjacent memory locations into fewer, wider, stores.
+  It can be enabled by using the <code>-fstore-merging</code> option and is
+  enabled by default at <code>-Os</code> and the <code>-O2</code> optimization
+  level or higher.</li>
 </ul>
--->
 
 <!-- .................................................................. -->
 <h2 id="languages">New Languages and Language specific improvements</h2>