[PATCH][wwwdocs] Mention new store merging pass for GCC 7

[PATCH][wwwdocs] Mention new store merging pass for GCC 7

[Kyrill Tkachov]

[-- Attachment #1: Type: text/plain, Size: 189 bytes --]

Hi all,

This patch adds a short entry for the store merging pass in GCC 7 to the "General Optimizer Improvements" section.

Eyeballed the result in Firefox.

Ok to commit?

Thanks,
Kyrill

[-- Attachment #2: wwwdocs-sm.patch --]
[-- Type: text/x-patch, Size: 983 bytes --]

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	20 Jan 2017 12:10:17 -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 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>
 </ul>
--->
 
 <!-- .................................................................. -->
 <h2 id="languages">New Languages and Language specific improvements</h2>

Re: [PATCH][wwwdocs] Mention new store merging pass for GCC 7

[Jeff Law]

On 01/23/2017 05:26 AM, Kyrill Tkachov wrote:
> Hi all,
>
> This patch adds a short entry for the store merging pass in GCC 7 to the
> "General Optimizer Improvements" section.
>
> Eyeballed the result in Firefox.
>
> Ok to commit?
OK.
jeff

Re: [PATCH][wwwdocs] Mention new store merging pass for GCC 7

[Gerald Pfeifer]

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>

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

Re: [PATCH][wwwdocs] Mention new store merging pass for GCC 7

[Richard Earnshaw (lists)]

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

Re: [PATCH][wwwdocs] Mention new store merging pass for GCC 7

[Kyrill Tkachov]

[-- Attachment #1: Type: text/plain, Size: 1167 bytes --]


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


[-- Attachment #2: wwwdocs-sm.patch --]
[-- Type: text/x-patch, Size: 991 bytes --]

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>