From patchwork Fri Apr 1 14:48:07 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom de Vries X-Patchwork-Id: 89271 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 7C484B6F94 for ; Sat, 2 Apr 2011 01:48:15 +1100 (EST) Received: (qmail 3963 invoked by alias); 1 Apr 2011 14:48:13 -0000 Received: (qmail 3955 invoked by uid 22791); 1 Apr 2011 14:48:13 -0000 X-SWARE-Spam-Status: No, hits=-1.6 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; Fri, 01 Apr 2011 14:48:09 +0000 Received: (qmail 8580 invoked from network); 1 Apr 2011 14:48:08 -0000 Received: from unknown (HELO ?192.168.1.66?) (vries@127.0.0.2) by mail.codesourcery.com with ESMTPA; 1 Apr 2011 14:48:08 -0000 Message-ID: <4D95E5A7.7090201@codesourcery.com> Date: Fri, 01 Apr 2011 16:48:07 +0200 From: Tom de Vries User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.14) Gecko/20110223 Lightning/1.0b2 Thunderbird/3.1.8 MIME-Version: 1.0 To: gcc-patches@gcc.gnu.org, ebotcazou@libertysurf.fr CC: Maxim Kuvyrkov , Bernd Schmidt Subject: Re: [PATCH, PR43920, 3/9] Cleanup. References: <4D94C603.7080505@codesourcery.com> <4D94C80C.10100@codesourcery.com> In-Reply-To: <4D94C80C.10100@codesourcery.com> 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 Reposting, with ChangeLog. 2011-04-01 Tom de Vries PR target/43920 * gcc/cfgcleanup.c (try_crossjump_bb): Remove 2 superfluous variables. diff -u gcc/cfgcleanup.c gcc/cfgcleanup.c --- gcc/cfgcleanup.c (working copy) +++ gcc/cfgcleanup.c (working copy) @@ -1961,7 +1961,6 @@ edge e, e2, fallthru; bool changed; unsigned max, ix, ix2; - basic_block ev, ev2; /* Nothing to do if there is not at least two incoming edges. */ if (EDGE_COUNT (bb->preds) < 2) @@ -2001,9 +2000,9 @@ fallthru = find_fallthru_edge (bb->preds); changed = false; - for (ix = 0, ev = bb; ix < EDGE_COUNT (ev->preds); ) + for (ix = 0; ix < EDGE_COUNT (bb->preds);) { - e = EDGE_PRED (ev, ix); + e = EDGE_PRED (bb, ix); ix++; /* As noted above, first try with the fallthru predecessor (or, a @@ -2021,7 +2020,6 @@ { changed = true; ix = 0; - ev = bb; continue; } } @@ -2045,10 +2043,9 @@ if (EDGE_SUCC (e->src, 0) != e) continue; - for (ix2 = 0, ev2 = bb; ix2 < EDGE_COUNT (ev2->preds); ) + for (ix2 = 0; ix2 < EDGE_COUNT (bb->preds); ix2++) { - e2 = EDGE_PRED (ev2, ix2); - ix2++; + e2 = EDGE_PRED (bb, ix2); if (e2 == e) continue; @@ -2071,7 +2068,6 @@ if (try_crossjump_to_edge (mode, e, e2)) { changed = true; - ev2 = bb; ix = 0; break; }