This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][wwwdocs] Mention new store merging pass for GCC 7
- From: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>
- To: Gerald Pfeifer <gerald at pfeifer dot com>, Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 24 Jan 2017 13:44:09 +0000
- Subject: Re: [PATCH][wwwdocs] Mention new store merging pass for GCC 7
- Authentication-results: sourceware.org; auth=none
- References: <5885F65B.2070001@foss.arm.com> <alpine.LSU.2.20.1701231743340.2449@anthias.pfeifer.com>
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