From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 119466 invoked by alias); 9 Jun 2017 10:35:32 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 119455 invoked by uid 89); 9 Jun 2017 10:35:31 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2 spammy=Realistic X-HELO: mo4-p00-ob.smtp.rzone.de Received: from mo4-p00-ob.smtp.rzone.de (HELO mo4-p00-ob.smtp.rzone.de) (81.169.146.216) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 09 Jun 2017 10:35:29 +0000 X-RZG-AUTH: :LXoWVUeid/7A29J/hMvvT3ol15ykJcYwTPLBCxG2PQt6BpGCfQ== X-RZG-CLASS-ID: mo00 Received: from [192.168.0.123] (ip5f587033.dynamic.kabel-deutschland.de [95.88.112.51]) by smtp.strato.de (RZmta 40.9 DYNA|AUTH) with ESMTPSA id 504d65t59AZV9hA (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA (curve secp521r1 with 521 ECDH bits, eq. 15360 bits RSA)) (Client did not present a certificate) for ; Fri, 9 Jun 2017 12:35:31 +0200 (CEST) Subject: Ping #1: [patch] Fix PR80929: Realistic PARALLEL cost in seq_cost. To: gcc-patches References: <14699546-2dd9-29d7-93f8-ef893fe513ed@gjlay.de> From: Georg-Johann Lay Message-ID: Date: Fri, 09 Jun 2017 10:35:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <14699546-2dd9-29d7-93f8-ef893fe513ed@gjlay.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2017-06/txt/msg00596.txt.bz2 Ping #1 https://gcc.gnu.org/ml/gcc-patches/2017-06/msg00096.html On 02.06.2017 09:53, Georg-Johann Lay wrote: > Hi, > > this small addition improves costs of PARALLELs in > rtlanal.c:seq_cost(). Up to now, these costs are > assumed to be 1 which gives gross inexact costs for, > e.g. divmod which is represented as PARALLEL. > > The patch just forwards cost computation to insn_rtx_cost > which uses the cost of the 1st SET (if any) and otherwise > assign costs of 1 insn. > > Bootstrapped & regtested on x86_64. > > Moreover, it fixed the division by constant on avr where > the problem popped up since PR79665. > > Ok to install? > > Johann > > gcc/ > PR middle-end/80929 > * rtlanal.c (seq_cost) [PARALLEL]: Get cost from insn_rtx_cost > instead of assuming cost of 1. >