From patchwork Sun May 8 04:40:29 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Revital Eres X-Patchwork-Id: 1416 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 ck17cs20533qab; Sat, 7 May 2011 21:40:32 -0700 (PDT) Received: by 10.213.14.2 with SMTP id e2mr552757eba.147.1304829632180; Sat, 07 May 2011 21:40:32 -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 r5si10706386eef.106.2011.05.07.21.40.31 (version=TLSv1/SSLv3 cipher=OTHER); Sat, 07 May 2011 21:40:32 -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 eya25 with SMTP id 25so1459242eya.37 for ; Sat, 07 May 2011 21:40:31 -0700 (PDT) MIME-Version: 1.0 Received: by 10.213.27.19 with SMTP id g19mr551743ebc.127.1304829629714; Sat, 07 May 2011 21:40:29 -0700 (PDT) Received: by 10.213.108.203 with HTTP; Sat, 7 May 2011 21:40:29 -0700 (PDT) In-Reply-To: References: Date: Sun, 8 May 2011 07:40:29 +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)