From patchwork Tue Aug 31 15:00:56 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Nathan Froyd X-Patchwork-Id: 63275 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 355D9B6F0D for ; Wed, 1 Sep 2010 01:01:04 +1000 (EST) Received: (qmail 7610 invoked by alias); 31 Aug 2010 15:01:02 -0000 Received: (qmail 7600 invoked by uid 22791); 31 Aug 2010 15:01:01 -0000 X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 tests=AWL, BAYES_00, T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 31 Aug 2010 15:00:57 +0000 Received: (qmail 30365 invoked from network); 31 Aug 2010 15:00:56 -0000 Received: from unknown (HELO localhost) (froydnj@127.0.0.2) by mail.codesourcery.com with ESMTPA; 31 Aug 2010 15:00:56 -0000 Date: Tue, 31 Aug 2010 08:00:56 -0700 From: Nathan Froyd To: gcc-patches@gcc.gnu.org Subject: [obv,fr30] fix some warnings in fr30.c Message-ID: <20100831150055.GY16898@codesourcery.com> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.17+20080114 (2008-01-14) X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Noticed while testing other things: gcc/config/fr30/fr30.c: In function ‘fr30_move_double’: gcc/config/fr30/fr30.c:885: warning: ISO C90 forbids mixed declarations and code gcc/config/fr30/fr30.c:886: warning: ISO C90 forbids mixed declarations and code gcc/config/fr30/fr30.c:883: warning: unused variable ‘dregno’ dregno is truly dead, its last use was deleted by: 2007-11-26 Rask Ingemann Lambertsen PR target/34174 * config/fr30/fr30.c (fr30_move_double): Sanitize mem->reg case. Copy the address before it is clobbered. Fixed like so. Committed. -Nathan * config/fr30/fr30.c (fr30_move_double): Delete `dregno' and extra semicolons. Index: config/fr30/fr30.c =================================================================== --- config/fr30/fr30.c (revision 163673) +++ config/fr30/fr30.c (working copy) @@ -844,9 +844,8 @@ fr30_move_double (rtx * operands) else if (src_code == MEM) { rtx addr = XEXP (src, 0); - int dregno = REGNO (dest); - rtx dest0 = operand_subword (dest, 0, TRUE, mode);; - rtx dest1 = operand_subword (dest, 1, TRUE, mode);; + rtx dest0 = operand_subword (dest, 0, TRUE, mode); + rtx dest1 = operand_subword (dest, 1, TRUE, mode); rtx new_mem; gcc_assert (GET_CODE (addr) == REG);