From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 104773 invoked by alias); 1 Aug 2017 09:07:13 -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 104736 invoked by uid 89); 1 Aug 2017 09:07:11 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-10.4 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:1624 X-HELO: mail-qk0-f195.google.com Received: from mail-qk0-f195.google.com (HELO mail-qk0-f195.google.com) (209.85.220.195) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 01 Aug 2017 09:07:09 +0000 Received: by mail-qk0-f195.google.com with SMTP id d136so917795qkg.3 for ; Tue, 01 Aug 2017 02:07:09 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=rvCmgzYUPDNAyBsiahhu+4/XfidiX63T8erIdzb5b3Y=; b=qGa7fKgtt0jn9V3VZN2GNaxDje5zoXfD7zN1FYZI+jo7yXWyRx92fbl9euukPkqEhC qcfKMNqs8UeqXae86HREKEa/YnUnLaXZ6k9saz7N6KoAF8q2NNrCJja5VOGE2aPPqjFB Zk0BFhlkh4xhUylEhUbi1JBeaDIGBVI4tCHuHtXGr/4ihVg02w82cZEpey+cXF8rq1eU oq28ly8VieCDbLbq/xecJVAMc3DYdVlFuwT3lkFm2TpOsq8H6UoMc62w/IkEgeVWKA+J +u90DwXQK7x0GESTfs/3bn26yJZo9fqyKESlcJeeiNgC9Ai8/zG3L9WyqqDuOYN3aOLv Kozg== X-Gm-Message-State: AIVw112N4W5qJwL/HOyE3IgoGElYy/NQ7AMQd1m0w4YZSBDkgT8umQoC FrnoQkp2tVosp1kUXi+mdAMBKaFxnQ== X-Received: by 10.55.40.104 with SMTP id o101mr24422203qkh.311.1501578427911; Tue, 01 Aug 2017 02:07:07 -0700 (PDT) MIME-Version: 1.0 Received: by 10.140.43.35 with HTTP; Tue, 1 Aug 2017 02:07:07 -0700 (PDT) In-Reply-To: References: From: Richard Biener Date: Tue, 01 Aug 2017 09:07:00 -0000 Message-ID: Subject: Re: [PATCH 0/5] RFC, WIP: RTL cost improvements To: Segher Boessenkool Cc: GCC Patches Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2017-08/txt/msg00038.txt.bz2 On Tue, Aug 1, 2017 at 1:24 AM, Segher Boessenkool wrote: > This series creates pattern_cost and insn_cost functions that together > replace the existing insn_rtx_cost function. > > pattern_cost is like the old insn_rtx_cost function; insn_cost takes > an actual rtx_insn * as input, not just a pattern. > > Also a targetm.insn_cost is added, which targets can use to implement > a more exact cost more easily. > > The combine patch is pretty gross (but functional), it needs some > refactoring (to not call recog so often). The rs6000 patch is very > much a work in progress. > > How does this look? Is this the right direction? I think it is the right direction. I'll leave the details to somebody more familiar with RTL. Richard. > > Segher > > > Segher Boessenkool (5): > Rename existing insn_cost to insn_sched_cost > Replace insn_rtx_cost with insn_cost and pattern_cost > combine: Use insn_cost instead of pattern_cost everywhere > Add targetm.insn_cost hook > rs6000: Implement insn_cost hook > > gcc/cfgrtl.c | 7 +++---- > gcc/combine.c | 40 +++++++++++++++++++++++++++++----------- > gcc/config/rs6000/rs6000.c | 14 ++++++++++++++ > gcc/doc/tm.texi | 12 ++++++++++++ > gcc/doc/tm.texi.in | 2 ++ > gcc/dse.c | 2 +- > gcc/haifa-sched.c | 14 +++++++------- > gcc/ifcvt.c | 12 ++++++------ > gcc/rtl.h | 3 ++- > gcc/rtlanal.c | 16 ++++++++++++++-- > gcc/sched-int.h | 2 +- > gcc/sched-rgn.c | 4 ++-- > gcc/sel-sched-ir.c | 2 +- > gcc/target.def | 14 ++++++++++++++ > 14 files changed, 108 insertions(+), 36 deletions(-) > > -- > 1.9.3 >