From patchwork Sun May 8 04:52:53 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Revital Eres X-Patchwork-Id: 1421 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 ck17cs20650qab; Sat, 7 May 2011 21:52:56 -0700 (PDT) Received: by 10.14.3.150 with SMTP id 22mr179230eeh.196.1304830375880; Sat, 07 May 2011 21:52:55 -0700 (PDT) Received: from mail-ew0-f50.google.com (mail-ew0-f50.google.com [209.85.215.50]) by mx.google.com with ESMTPS id f1si6630507eeb.91.2011.05.07.21.52.55 (version=TLSv1/SSLv3 cipher=OTHER); Sat, 07 May 2011 21:52:55 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.215.50 is neither permitted nor denied by best guess record for domain of revital.eres@linaro.org) client-ip=209.85.215.50; Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.215.50 is neither permitted nor denied by best guess record for domain of revital.eres@linaro.org) smtp.mail=revital.eres@linaro.org Received: by ewy10 with SMTP id 10so1460861ewy.37 for ; Sat, 07 May 2011 21:52:55 -0700 (PDT) MIME-Version: 1.0 Received: by 10.213.27.19 with SMTP id g19mr555381ebc.127.1304830373947; Sat, 07 May 2011 21:52:53 -0700 (PDT) Received: by 10.213.108.203 with HTTP; Sat, 7 May 2011 21:52:53 -0700 (PDT) Date: Sun, 8 May 2011 07:52:53 +0300 Message-ID: Subject: =?UTF-8?B?W1BBVENILCBTTVMgMy8zXSBTa2lwIERFQlVHX0lOU04gaW4gbG9vcC1kb2xvb+KAi+KAiw==?= =?UTF-8?B?cC4=?= 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)