Re: [PATCH][i386] Fix PR83008

2017-12-08 Thread Richard Biener
On December 8, 2017 5:45:23 PM GMT+01:00, Jan Hubicka wrote: >> On December 8, 2017 4:51:16 PM GMT+01:00, Jan Hubicka > wrote: >> >> >> >> This restores the vec_construct cost dependence on the vector >element >> >> count. Honza removed this (accidentially?) during the rework. >> >> >> >> Boots

Re: [PATCH][i386] Fix PR83008

2017-12-08 Thread Jan Hubicka
> On December 8, 2017 4:51:16 PM GMT+01:00, Jan Hubicka wrote: > >> > >> This restores the vec_construct cost dependence on the vector element > >> count. Honza removed this (accidentially?) during the rework. > >> > >> Bootstrap / regtest running on x86_64-unknown-linux-gnu, ok for > >trunk? >

Re: [PATCH][i386] Fix PR83008

2017-12-08 Thread Richard Biener
On December 8, 2017 4:51:16 PM GMT+01:00, Jan Hubicka wrote: >> >> This restores the vec_construct cost dependence on the vector element >> count. Honza removed this (accidentially?) during the rework. >> >> Bootstrap / regtest running on x86_64-unknown-linux-gnu, ok for >trunk? > >Hmm, the fal

Re: [PATCH][i386] Fix PR83008

2017-12-08 Thread Jan Hubicka
> > This restores the vec_construct cost dependence on the vector element > count. Honza removed this (accidentially?) during the rework. > > Bootstrap / regtest running on x86_64-unknown-linux-gnu, ok for trunk? Hmm, the false parameter to ix86_vec_cost is supposed to do that. It uses: if (!

[PATCH][i386] Fix PR83008

2017-12-08 Thread Richard Biener
This restores the vec_construct cost dependence on the vector element count. Honza removed this (accidentially?) during the rework. Bootstrap / regtest running on x86_64-unknown-linux-gnu, ok for trunk? Thanks, Richard. 2017-12-08 Richard Biener PR target/83008 * config/i38