From patchwork Thu Sep 20 22:06:51 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bernd Schmidt X-Patchwork-Id: 185515 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 C1B032C0080 for ; Fri, 21 Sep 2012 08:07:23 +1000 (EST) Comment: DKIM? See http://www.dkim.org DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=gcc.gnu.org; s=default; x=1348783644; h=Comment: DomainKey-Signature:Received:Received:Received:Received:Received: Received:Message-ID:Date:From:User-Agent:MIME-Version:To:CC: Subject:References:In-Reply-To:Content-Type:Mailing-List: Precedence:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:Sender:Delivered-To; bh=7upL4Nsg+gJqjD+BId93LVSFHeg=; b=ZlpOX4AT1FhfFh3UuA6yKVoqfRmfDOaGwOh0l2kdfGBt93fSsv8dkqYNpBjO9u q1BCi3b/ugAHdYu9ozbtDEtiMfA6uGrnEOxLHHXsNOwHojcIvqesc9x2OFQEulBJ 65j3zK+VLva69nmvJh7C21eibvnOlM2i8M7dK6o4hbG8c= Comment: DomainKeys? See http://antispam.yahoo.com/domainkeys DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=gcc.gnu.org; h=Received:Received:X-SWARE-Spam-Status:X-Spam-Check-By:Received:Received:Received:Received:Message-ID:Date:From:User-Agent:MIME-Version:To:CC:Subject:References:In-Reply-To:Content-Type:Mailing-List:Precedence:List-Id:List-Unsubscribe:List-Archive:List-Post:List-Help:Sender:Delivered-To; b=Fwvqk4BN9I2W5Ux6BsGXVOggF7kuWg4VZL4wCysl5srKA1A4xO0w8SUaAQWOiP T3al/x+0RUm7ZS6ZlujupJ2vutKh6Apl5lNjBfkXYC4TuEY3FbZgwn9U3QlgsvBP SY+JhI6jorTBOWgA0bqlu1dSUiUh4duoPs68d4fS+xsMs=; Received: (qmail 21439 invoked by alias); 20 Sep 2012 22:07:20 -0000 Received: (qmail 21420 invoked by uid 22791); 20 Sep 2012 22:07:18 -0000 X-SWARE-Spam-Status: No, hits=-4.4 required=5.0 tests=AWL, BAYES_00, KHOP_RCVD_UNTRUST, KHOP_THREADED, RCVD_IN_HOSTKARMA_W, RCVD_IN_HOSTKARMA_WL X-Spam-Check-By: sourceware.org Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 20 Sep 2012 22:07:04 +0000 Received: from svr-orw-exc-10.mgc.mentorg.com ([147.34.98.58]) by relay1.mentorg.com with esmtp id 1TEotj-00021e-2W from Bernd_Schmidt@mentor.com ; Thu, 20 Sep 2012 15:07:03 -0700 Received: from SVR-IES-FEM-01.mgc.mentorg.com ([137.202.0.104]) by SVR-ORW-EXC-10.mgc.mentorg.com with Microsoft SMTPSVC(6.0.3790.4675); Thu, 20 Sep 2012 15:07:03 -0700 Received: from [127.0.0.1] (137.202.0.76) by SVR-IES-FEM-01.mgc.mentorg.com (137.202.0.104) with Microsoft SMTP Server id 14.1.289.1; Thu, 20 Sep 2012 23:07:01 +0100 Message-ID: <505B937B.3080506@codesourcery.com> Date: Fri, 21 Sep 2012 00:06:51 +0200 From: Bernd Schmidt User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.7) Gecko/20120912 Thunderbird/10.0.7 MIME-Version: 1.0 To: "H.J. Lu" CC: GCC Patches Subject: Re: Scheduler: Allow breaking dependencies by modifying patterns References: <501BBE7B.7030104@codesourcery.com> In-Reply-To: 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 On 09/20/2012 07:42 PM, H.J. Lu wrote: > On Fri, Aug 3, 2012 at 5:05 AM, Bernd Schmidt wrote: >> This patch allows us to change >> >> rn++ >> rm=[rn] >> >> into >> >> rm=[rn + 4] >> rn++ >> > This caused: > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54645 The jury is still out on that, but it did cause PR54643. This seems to be some kind of interaction with the new SCHED_PRESSURE_MODEL code. For now it seems best to just disable the new optimization if scheduling for pressure. It will still trigger during sched2. I've bootstrapped the following on x86_64-linux and verified that the arm build gets further (it now fails in libbacktrace). Committed. Bernd Index: gcc/ChangeLog =================================================================== --- gcc/ChangeLog (revision 191592) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,9 @@ +2012-09-20 Bernd Schmidt + + PR bootstrap/54643 + * haifa-sched.c (schedule_block): Skip find_modifiable_mems if using + SCHED_PRESSURE_MODEL. + 2012-09-20 Joern Rennecke * rtlanal.c (dead_or_set_regno_p): Fix COND_EXEC handling. Index: gcc/haifa-sched.c =================================================================== --- gcc/haifa-sched.c (revision 191582) +++ gcc/haifa-sched.c (working copy) @@ -5854,7 +5854,8 @@ schedule_block (basic_block *target_bb) rtx head = NEXT_INSN (prev_head); rtx tail = PREV_INSN (next_tail); - if ((current_sched_info->flags & DONT_BREAK_DEPENDENCIES) == 0) + if ((current_sched_info->flags & DONT_BREAK_DEPENDENCIES) == 0 + && sched_pressure != SCHED_PRESSURE_MODEL) find_modifiable_mems (head, tail); /* We used to have code to avoid getting parameters moved from hard