Re: [PATCH 0/5] RFC, WIP: RTL cost improvements

[Richard Biener <richard.guenther@gmail.com>]

On Tue, Aug 1, 2017 at 1:24 AM, Segher Boessenkool
<segher@kernel.crashing.org> 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
>