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

Reply via email to