Re: [PATCH 7/8] [RS6000] rs6000_rtx_costs reduce cost for SETs

2021-01-21 Thread Alan Modra via Gcc-patches
Ping. On Tue, Jan 12, 2021 at 02:02:27PM +1030, Alan Modra wrote: > Ping > https://gcc.gnu.org/pipermail/gcc-patches/2020-October/555758.html > > On Thu, Oct 08, 2020 at 09:27:59AM +1030, Alan Modra wrote: > > The aim of this patch is to make rtx_costs for SETs closer to > > insn_cost for SETs.

Re: [PATCH 7/8] [RS6000] rs6000_rtx_costs reduce cost for SETs

2020-10-08 Thread will schmidt via Gcc-patches
On Thu, 2020-10-08 at 09:27 +1030, Alan Modra via Gcc-patches wrote: > The aim of this patch is to make rtx_costs for SETs closer to > insn_cost for SETs. One visible effect on powerpc code is increased > if-conversion. > > * config/rs6000/rs6000.c (rs6000_rtx_costs): Reduce cost of SET >

[PATCH 7/8] [RS6000] rs6000_rtx_costs reduce cost for SETs

2020-10-07 Thread Alan Modra via Gcc-patches
The aim of this patch is to make rtx_costs for SETs closer to insn_cost for SETs. One visible effect on powerpc code is increased if-conversion. * config/rs6000/rs6000.c (rs6000_rtx_costs): Reduce cost of SET operands. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/r

Re: [RS6000] rs6000_rtx_costs reduce cost for SETs

2020-09-21 Thread Alan Modra via Gcc-patches
On Fri, Sep 18, 2020 at 01:13:18PM -0500, Segher Boessenkool wrote: > Thanks (to both of you). Interesting! Which of these unrelated changes > does this come from? Most of the changes I saw in code generation (not in spec, I didn't look there, but in gcc) came down to this change to the cost for

Re: [RS6000] rs6000_rtx_costs reduce cost for SETs

2020-09-18 Thread Segher Boessenkool
On Fri, Sep 18, 2020 at 01:08:42PM +0930, Alan Modra wrote: > On Thu, Sep 17, 2020 at 12:51:25PM -0500, Segher Boessenkool wrote: > > > - if (CONST_INT_P (XEXP (x, 1)) > > > - && satisfies_constraint_I (XEXP (x, 1))) > > > + if (!speed) > > > + /* A little more than one insn so that not

Re: [RS6000] rs6000_rtx_costs reduce cost for SETs

2020-09-17 Thread Alan Modra via Gcc-patches
On Thu, Sep 17, 2020 at 12:51:25PM -0500, Segher Boessenkool wrote: > Hi! > > On Tue, Sep 15, 2020 at 10:49:45AM +0930, Alan Modra wrote: > > Also use rs6000_cost only for speed. > > More directly: use something completely different for !speed, namely, > code size. Yes, that might be better. >

Re: [RS6000] rs6000_rtx_costs reduce cost for SETs

2020-09-17 Thread Segher Boessenkool
Hi! On Tue, Sep 15, 2020 at 10:49:45AM +0930, Alan Modra wrote: > Also use rs6000_cost only for speed. More directly: use something completely different for !speed, namely, code size. > - if (CONST_INT_P (XEXP (x, 1)) > - && satisfies_constraint_I (XEXP (x, 1))) > + if (!speed) >

[RS6000] rs6000_rtx_costs reduce cost for SETs

2020-09-14 Thread Alan Modra via Gcc-patches
Also use rs6000_cost only for speed. * config/rs6000/rs6000.c (rs6000_rtx_costs): Reduce cost for SETs when insn operation cost handled on recursive call. Only use rs6000_cost for speed. Tidy break/return. Tidy AND costing. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/