Re: Add missing cases to vect_get_smallest_scalar_type (PR 85286)

2018-04-10 Thread Richard Biener
On Tue, Apr 10, 2018 at 12:40 PM, Richard Sandiford wrote: > Jakub Jelinek writes: >> On Mon, Apr 09, 2018 at 06:47:45PM +0100, Richard Sandiford wrote: >>> In this PR we used WIDEN_SUM_EXPR to vectorise: >>> >>> short i, y; >>> int sum; >>>

Re: Add missing cases to vect_get_smallest_scalar_type (PR 85286)

2018-04-10 Thread Richard Sandiford
Jakub Jelinek writes: > On Mon, Apr 09, 2018 at 06:47:45PM +0100, Richard Sandiford wrote: >> In this PR we used WIDEN_SUM_EXPR to vectorise: >> >> short i, y; >> int sum; >> [...] >> for (i = x; i > 0; i--) >> sum += y; >> >> with 4 ints and 8 shorts per vector.

Re: Add missing cases to vect_get_smallest_scalar_type (PR 85286)

2018-04-10 Thread Richard Biener
On Mon, Apr 9, 2018 at 7:47 PM, Richard Sandiford wrote: > In this PR we used WIDEN_SUM_EXPR to vectorise: > > short i, y; > int sum; > [...] > for (i = x; i > 0; i--) > sum += y; > > with 4 ints and 8 shorts per vector. The problem was that we set > the

Re: Add missing cases to vect_get_smallest_scalar_type (PR 85286)

2018-04-09 Thread Jakub Jelinek
On Mon, Apr 09, 2018 at 06:47:45PM +0100, Richard Sandiford wrote: > In this PR we used WIDEN_SUM_EXPR to vectorise: > > short i, y; > int sum; > [...] > for (i = x; i > 0; i--) > sum += y; > > with 4 ints and 8 shorts per vector. The problem was that we set > the VF based only on