From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 128876 invoked by alias); 28 Mar 2015 06:24:06 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 128867 invoked by uid 89); 28 Mar 2015 06:24:06 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: Yes, score=6.8 required=5.0 tests=AWL,BAYES_00,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,KAM_FROM_URIBL_PCCC,MEDICAL_SUBJECT,RCVD_IN_DNSWL_NONE,SPF_PASS,T_RP_MATCHES_RCVD autolearn=no version=3.3.2 X-HELO: BLU004-OMC2S32.hotmail.com Received: from blu004-omc2s32.hotmail.com (HELO BLU004-OMC2S32.hotmail.com) (65.55.111.107) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA256 encrypted) ESMTPS; Sat, 28 Mar 2015 06:24:05 +0000 Received: from BLU436-SMTP183 ([65.55.111.73]) by BLU004-OMC2S32.hotmail.com over TLS secured channel with Microsoft SMTPSVC(7.5.7601.22751); Fri, 27 Mar 2015 23:24:03 -0700 X-TMN: [3U9D2hl7RZNVHp8+OT88paffEonJzZ7T] Message-ID: Date: Sat, 28 Mar 2015 06:24:00 -0000 From: Chen Gang User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Bernd Schmidt , Jeff Law CC: gcc-patches List Subject: Re: [PATCH] gcc/config/c6x/c6x.md: Remove "clobber (match_scratch ...)" in "movmisalign_store". References: <5515553A.7080303@codesourcery.com> In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-SW-Source: 2015-03/txt/msg01477.txt.bz2 On 3/28/15 09:32, Chen Gang wrote: > On 3/27/15 21:03, Bernd Schmidt wrote: >> On 03/27/2015 01:05 AM, Chen Gang wrote: >>> For misalignment memory access, c6x gcc will cause issue, so need remove >>> "clobber (match_scratch ...)" which will be symmetric with "movmisalign >>> _load", then pass compiling and generate correct assembly code. >>> >> >>> * config/c6x/c6x.md (movmisalign_store): Remove "clobber >>> (match_scratch ...)". >> >> No, that just will make the compiler confuse loads and stores. I've committed the following to fix it (I thought I'd done so a year ago, but probably it was one of those commit against an out-of-date tree situations and it didn't go through). >> > > OK, thanks. Your patch is OK to me. And I shall try to find another c6x > patches within this month (2015-03-31). > I have reported the related bug 65510, please continue for it, if I need to continue, please let me know. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed