Message ID | 5885F65B.2070001@foss.arm.com |
---|---|
State | New |
Headers | show |
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
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
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
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>