Re: [PATCH] Fix PR83008

2018-02-05 Thread Christophe Lyon
On 31 January 2018 at 16:01, Richard Biener wrote: > On Wed, 31 Jan 2018, Christophe Lyon wrote: > > > On 30 January 2018 at 11:47, Jakub Jelinek wrote: > > > On Tue, Jan 30, 2018 at 11:07:50AM +0100, Richard Biener wrote: > > >> > > >> I have been asked to push this change, fixing (somewhat) th

Re: [PATCH] Fix PR83008

2018-01-31 Thread Richard Biener
On Wed, 31 Jan 2018, Christophe Lyon wrote: > On 30 January 2018 at 11:47, Jakub Jelinek wrote: > > On Tue, Jan 30, 2018 at 11:07:50AM +0100, Richard Biener wrote: > >> > >> I have been asked to push this change, fixing (somewhat) the impreciseness > >> of costing constant/invariant vector uses i

Re: [PATCH] Fix PR83008

2018-01-31 Thread Christophe Lyon
On 30 January 2018 at 11:47, Jakub Jelinek wrote: > On Tue, Jan 30, 2018 at 11:07:50AM +0100, Richard Biener wrote: >> >> I have been asked to push this change, fixing (somewhat) the impreciseness >> of costing constant/invariant vector uses in SLP stmts. The previous >> code always just consider

Re: [PATCH] Fix PR83008

2018-01-30 Thread Jakub Jelinek
On Tue, Jan 30, 2018 at 11:07:50AM +0100, Richard Biener wrote: > > I have been asked to push this change, fixing (somewhat) the impreciseness > of costing constant/invariant vector uses in SLP stmts. The previous > code always just considered a single constant to be generated in the > prologue i

[PATCH] Fix PR83008

2018-01-30 Thread Richard Biener
I have been asked to push this change, fixing (somewhat) the impreciseness of costing constant/invariant vector uses in SLP stmts. The previous code always just considered a single constant to be generated in the prologue irrespective of how many we'd need. With this patch we properly handle thi