From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 76096 invoked by alias); 28 Mar 2015 01:32:36 -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 76086 invoked by uid 89); 28 Mar 2015 01:32:35 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: Yes, score=6.7 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-OMC2S1.hotmail.com Received: from blu004-omc2s1.hotmail.com (HELO BLU004-OMC2S1.hotmail.com) (65.55.111.76) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA256 encrypted) ESMTPS; Sat, 28 Mar 2015 01:32:33 +0000 Received: from BLU436-SMTP209 ([65.55.111.72]) by BLU004-OMC2S1.hotmail.com over TLS secured channel with Microsoft SMTPSVC(7.5.7601.22751); Fri, 27 Mar 2015 18:32:31 -0700 X-TMN: [3VlGHmvNdMMd7WRNqgT8AR2nnQHcyHQL] Message-ID: Date: Sat, 28 Mar 2015 01:32: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: <5515553A.7080303@codesourcery.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-SW-Source: 2015-03/txt/msg01474.txt.bz2 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). Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed