From patchwork Sun May 8 04:37:38 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Revital Eres X-Patchwork-Id: 1417 Return-Path: Delivered-To: unknown Received: from imap.gmail.com (74.125.159.109) by localhost6.localdomain6 with IMAP4-SSL; 08 Jun 2011 14:51:47 -0000 Delivered-To: patches@linaro.org Received: by 10.224.184.145 with SMTP id ck17cs20498qab; Sat, 7 May 2011 21:37:40 -0700 (PDT) Received: by 10.14.50.15 with SMTP id y15mr2683188eeb.149.1304829460146; Sat, 07 May 2011 21:37:40 -0700 (PDT) Received: from mail-ey0-f178.google.com (mail-ey0-f178.google.com [209.85.215.178]) by mx.google.com with ESMTPS id e72si7172795eee.14.2011.05.07.21.37.39 (version=TLSv1/SSLv3 cipher=OTHER); Sat, 07 May 2011 21:37:40 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.215.178 is neither permitted nor denied by best guess record for domain of revital.eres@linaro.org) client-ip=209.85.215.178; Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.215.178 is neither permitted nor denied by best guess record for domain of revital.eres@linaro.org) smtp.mail=revital.eres@linaro.org Received: by mail-ey0-f178.google.com with SMTP id 25so1458964eya.37 for ; Sat, 07 May 2011 21:37:39 -0700 (PDT) MIME-Version: 1.0 Received: by 10.213.21.2 with SMTP id h2mr523610ebb.51.1304829458241; Sat, 07 May 2011 21:37:38 -0700 (PDT) Received: by 10.213.108.203 with HTTP; Sat, 7 May 2011 21:37:38 -0700 (PDT) Date: Sun, 8 May 2011 07:37:38 +0300 Message-ID: Subject: [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloop.c From: Revital Eres To: gcc-patches@gcc.gnu.org Cc: rakdver@kam.mff.cuni.cz, Patch Tracking , zaks@il.ibm.com Hello, This small fix was inserted to skip DEBUG_INSNs while recognizing doloop pattern in loop-doloop.c file. It's a fix for the already approved do-loop patch (not in mainline yet, http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c The patch was tested together with the rest of the patches in this series and on top of the patch to support do-loop for ARM (not yet in mainline, but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html). On ppc64-redhat-linux regtest as well as bootstrap with SMS flags enabling SMS also on loops with stage count 1. Regtested on SPU. On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS flags enabling SMS also on loops with stage count 1. OK for mainline? Thanks, Revital Changelog: * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn instead of PREV_INSN. --- loop-doloop.c 2011-05-07 16:08:27.000000000 +0300 +++ loop-doloop_new.c 2011-05-07 16:07:48.000000000 +0300 @@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat) inc = XVECEXP (PATTERN (prev_insn), 0, 1); } else - inc = PATTERN (PREV_INSN (doloop_pat)); + inc = PATTERN (prev_insn); /* We expect the condition to be of the form (reg != 0) */ cond = XEXP (SET_SRC (cmp), 0); if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)