From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-437251-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 59000 invoked by alias); 29 Sep 2016 07:55:49 -0000
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
Received: (qmail 58978 invoked by uid 89); 29 Sep 2016 07:55:47 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-3.9 required=5.0 tests=BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=
X-HELO: foss.arm.com
Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 29 Sep 2016 07:55:46 +0000
Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])	by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 6E8F722E;	Thu, 29 Sep 2016 00:55:44 -0700 (PDT)
Received: from [10.2.207.77] (e100706-lin.cambridge.arm.com [10.2.207.77])	by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id AE20B3F218;	Thu, 29 Sep 2016 00:55:43 -0700 (PDT)
Message-ID: <57ECC8FE.3040602@foss.arm.com>
Date: Thu, 29 Sep 2016 08:24:00 -0000
From: Kyrill Tkachov <kyrylo.tkachov@foss.arm.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0
MIME-Version: 1.0
To: Richard Biener <rguenther@suse.de>,  Pat Haugen <pthaugen@linux.vnet.ibm.com>
CC: GCC Patches <gcc-patches@gcc.gnu.org>, wschmidt@linux.vnet.ibm.com
Subject: Re: [PATCH][v4] GIMPLE store merging pass
References: <57EBE7A6.3040103@foss.arm.com> <d5080ceb-eb5a-c68a-f44c-694f14c17f4e@linux.vnet.ibm.com> <alpine.LSU.2.11.1609290934210.26629@t29.fhfr.qr>
In-Reply-To: <alpine.LSU.2.11.1609290934210.26629@t29.fhfr.qr>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
X-SW-Source: 2016-09/txt/msg02214.txt.bz2


On 29/09/16 08:36, Richard Biener wrote:
> On Wed, 28 Sep 2016, Pat Haugen wrote:
>
>> On 09/28/2016 10:54 AM, Kyrill Tkachov wrote:
>>> +fstore-merging
>>> +Common Var(flag_store_merging) Optimization
>>> +Use the tree store merging pass.
>>> +
>> Did you purposely leave off "Report" for this option? I noticed the option didn't show up in the "options enabled:" section of the .s file when -fverbose-asm is specified.
> Report should be added.  Also please don't mention "tree" -- this is
> user facing documentation which shouldn't talk about GCC internals.
> Thus for example "Merge adjacent stores." would be more appropriate here.

Thanks, I'll update this.

Kyrill

> Richard.