From patchwork Fri Nov 19 18:56:17 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Uros Bizjak X-Patchwork-Id: 72293 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 86495B70FD for ; Sat, 20 Nov 2010 05:56:30 +1100 (EST) Received: (qmail 11687 invoked by alias); 19 Nov 2010 18:56:26 -0000 Received: (qmail 11679 invoked by uid 22791); 19 Nov 2010 18:56:24 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, TW_ZJ, T_TO_NO_BRKTS_FREEMAIL X-Spam-Check-By: sourceware.org Received: from mail-px0-f175.google.com (HELO mail-px0-f175.google.com) (209.85.212.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 19 Nov 2010 18:56:19 +0000 Received: by pxi17 with SMTP id 17so1196298pxi.20 for ; Fri, 19 Nov 2010 10:56:17 -0800 (PST) MIME-Version: 1.0 Received: by 10.142.170.15 with SMTP id s15mr2014955wfe.276.1290192977530; Fri, 19 Nov 2010 10:56:17 -0800 (PST) Received: by 10.143.161.2 with HTTP; Fri, 19 Nov 2010 10:56:17 -0800 (PST) Date: Fri, 19 Nov 2010 19:56:17 +0100 Message-ID: Subject: [PATCH]: Use FOR_BB_INSNS in a couple of places From: Uros Bizjak To: gcc-patches@gcc.gnu.org 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 Hello! 2010-11-19 Uros Bizjak * mode-switching.c (optimize_mode_switching): Use FOR_BB_INSNS instead of open-coded for loop. * cfgrtl.c (redirect_branch_edge): Ditto. Tested on x86_64-pc-linux-gnu {,-m32}, committed to mainline SVN. Uros. Index: mode-switching.c =================================================================== --- mode-switching.c (revision 166931) +++ mode-switching.c (working copy) @@ -518,9 +518,7 @@ optimize_mode_switching (void) } } - for (insn = BB_HEAD (bb); - insn != NULL && insn != NEXT_INSN (BB_END (bb)); - insn = NEXT_INSN (insn)) + FOR_BB_INSNS (bb, insn) { if (INSN_P (insn)) { Index: cfgrtl.c =================================================================== --- cfgrtl.c (revision 166931) +++ cfgrtl.c (working copy) @@ -1062,8 +1062,7 @@ redirect_branch_edge (edge e, basic_bloc /* When expanding this BB might actually contain multiple jumps (i.e. not yet split by find_many_sub_basic_blocks). Redirect all of those that match our label. */ - for (insn = BB_HEAD (src); insn != NEXT_INSN (BB_END (src)); - insn = NEXT_INSN (insn)) + FOR_BB_INSNS (src, insn) if (JUMP_P (insn) && !patch_jump_insn (insn, old_label, target)) return NULL;