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
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>
