Re: [ping #4][patch] Fix PR80929: Realistic PARALLEL cost in seq_cost.

2017-07-13 Thread Georg-Johann Lay
On 12.07.2017 21:36, Segher Boessenkool wrote: On Wed, Jul 12, 2017 at 03:30:00PM +0200, Georg-Johann Lay wrote: On 12.07.2017 14:11, Segher Boessenkool wrote: On Tue, Jul 11, 2017 at 10:47:27AM +0200, Georg-Johann Lay wrote: This small addition improves costs of PARALLELs in rtlanal.c:seq_cos

Re: [ping #4][patch] Fix PR80929: Realistic PARALLEL cost in seq_cost.

2017-07-12 Thread Segher Boessenkool
On Wed, Jul 12, 2017 at 03:30:00PM +0200, Georg-Johann Lay wrote: > On 12.07.2017 14:11, Segher Boessenkool wrote: > >On Tue, Jul 11, 2017 at 10:47:27AM +0200, Georg-Johann Lay wrote: > >>This small addition improves costs of PARALLELs in > >>rtlanal.c:seq_cost(). Up to now, these costs are > >>as

Re: [ping #4][patch] Fix PR80929: Realistic PARALLEL cost in seq_cost.

2017-07-12 Thread Georg-Johann Lay
On 12.07.2017 14:11, Segher Boessenkool wrote: Hi, On Tue, Jul 11, 2017 at 10:47:27AM +0200, Georg-Johann Lay wrote: 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 rep

Re: [ping #4][patch] Fix PR80929: Realistic PARALLEL cost in seq_cost.

2017-07-12 Thread Segher Boessenkool
Hi, On Tue, Jul 11, 2017 at 10:47:27AM +0200, Georg-Johann Lay wrote: > 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. insn_rtx_cost retur

[ping #4][patch] Fix PR80929: Realistic PARALLEL cost in seq_cost.

2017-07-11 Thread Georg-Johann Lay
Ping #4 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