From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1859 invoked by alias); 26 Sep 2011 18:21:17 -0000 Received: (qmail 1847 invoked by uid 22791); 26 Sep 2011 18:21:15 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,TW_HR,TW_RQ X-Spam-Check-By: sourceware.org Received: from mail-ey0-f175.google.com (HELO mail-ey0-f175.google.com) (209.85.215.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 26 Sep 2011 18:20:59 +0000 Received: by eya25 with SMTP id 25so3917671eya.20 for ; Mon, 26 Sep 2011 11:20:58 -0700 (PDT) Received: by 10.14.3.210 with SMTP id 58mr2179947eeh.29.1317061258190; Mon, 26 Sep 2011 11:20:58 -0700 (PDT) MIME-Version: 1.0 Received: by 10.14.189.1 with HTTP; Mon, 26 Sep 2011 11:20:38 -0700 (PDT) In-Reply-To: <4E80A0CF.2030806@gjlay.de> References: <4E80A0CF.2030806@gjlay.de> From: Denis Chertykov Date: Mon, 26 Sep 2011 19:36:00 -0000 Message-ID: Subject: Re: [Patch,AVR]: Housekeeping: Clean-up notice_update_cc To: Georg-Johann Lay Cc: gcc-patches@gcc.gnu.org, Anatoly Sokolov , Eric Weddington Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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 X-SW-Source: 2011-09/txt/msg01635.txt.bz2 2011/9/26 Georg-Johann Lay : > This patchlet represents the impact of insn ashrqi3 on CC by means of att= ribute > "cc" instead of hard-coding it in notice_update_cc. > > Testsuite passes fine. Moreover, tested against code like > > char c; > > void func_1 (char a) > { > =C2=A0 =C2=A0a =3D a >> 7; > =C2=A0 =C2=A0if (a) > =C2=A0 =C2=A0 =C2=A0 =C2=A0c =3D a; > } > > that triggered PR39633. > > Ok? > > Johann > > =C2=A0 =C2=A0 =C2=A0 =C2=A0* config/avr/avr.md (ashrqi3): Split alternati= ve "n" > =C2=A0 =C2=A0 =C2=A0 =C2=A0into its remaining parts C03, C04, C05, C06, C= 07 and describe > =C2=A0 =C2=A0 =C2=A0 =C2=A0impact on CC by attribute "cc" appropriately. > =C2=A0 =C2=A0 =C2=A0 =C2=A0* config/avr/avr.c (notice_update_cc): Clean-u= p: Don't patch CC0 > =C2=A0 =C2=A0 =C2=A0 =C2=A0by digging RTX. > Approved. Denis.