From patchwork Sun May 8 04:56:37 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Revital Eres X-Patchwork-Id: 1420 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:48 -0000 Delivered-To: patches@linaro.org Received: by 10.224.184.145 with SMTP id ck17cs20684qab; Sat, 7 May 2011 21:56:40 -0700 (PDT) Received: by 10.213.112.144 with SMTP id w16mr553512ebp.148.1304830599863; Sat, 07 May 2011 21:56:39 -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 q56si10738298eef.25.2011.05.07.21.56.39 (version=TLSv1/SSLv3 cipher=OTHER); Sat, 07 May 2011 21:56:39 -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 25so1460633eya.37 for ; Sat, 07 May 2011 21:56:39 -0700 (PDT) MIME-Version: 1.0 Received: by 10.213.21.2 with SMTP id h2mr528962ebb.51.1304830597815; Sat, 07 May 2011 21:56:37 -0700 (PDT) Received: by 10.213.108.203 with HTTP; Sat, 7 May 2011 21:56:37 -0700 (PDT) Date: Sun, 8 May 2011 07:56:37 +0300 Message-ID: Subject: =?UTF-8?B?W1BBVENILCBTTVMgMy8zXSBTa2lwIERFQlVHX0lOU04gaW4gbG9vcC1kb2xvb+KAi+KAiw==?= =?UTF-8?B?4oCLcC4=?= From: Revital Eres To: gcc-patches@gcc.gnu.org Cc: rakdver@kam.mff.cuni.cz, Patch Tracking , zaks@il.ibm.com Hello, (sorry for multiple copies of this email) 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)