Re: Move code out of vect_slp_analyze_bb_1

2019-10-21 Thread Richard Biener
On October 21, 2019 4:06:49 PM GMT+02:00, Richard Sandiford wrote: >Richard Biener writes: >> On Mon, Oct 21, 2019 at 12:08 PM Richard Sandiford >> wrote: >>> >>> Richard Biener writes: >>> > On October 20, 2019 2:54:48 PM GMT+02:00, Richard Sandiford > wrote: >>> >>Richard Biener writes: >>>

Re: Move code out of vect_slp_analyze_bb_1

2019-10-21 Thread Richard Sandiford
Richard Biener writes: > On Mon, Oct 21, 2019 at 12:08 PM Richard Sandiford > wrote: >> >> Richard Biener writes: >> > On October 20, 2019 2:54:48 PM GMT+02:00, Richard Sandiford >> > wrote: >> >>Richard Biener writes: >> >>> On October 19, 2019 5:06:40 PM GMT+02:00, Richard Sandiford >> >> w

Re: Move code out of vect_slp_analyze_bb_1

2019-10-21 Thread Richard Biener
On Mon, Oct 21, 2019 at 12:08 PM Richard Sandiford wrote: > > Richard Biener writes: > > On October 20, 2019 2:54:48 PM GMT+02:00, Richard Sandiford > > wrote: > >>Richard Biener writes: > >>> On October 19, 2019 5:06:40 PM GMT+02:00, Richard Sandiford > >> wrote: > After the previous patc

Re: Move code out of vect_slp_analyze_bb_1

2019-10-21 Thread Richard Sandiford
Richard Biener writes: > On October 20, 2019 2:54:48 PM GMT+02:00, Richard Sandiford > wrote: >>Richard Biener writes: >>> On October 19, 2019 5:06:40 PM GMT+02:00, Richard Sandiford >> wrote: After the previous patch, it seems more natural to apply the PARAM_SLP_MAX_INSNS_IN_BB thresho

Re: Move code out of vect_slp_analyze_bb_1

2019-10-20 Thread Richard Biener
On October 20, 2019 2:54:48 PM GMT+02:00, Richard Sandiford wrote: >Richard Biener writes: >> On October 19, 2019 5:06:40 PM GMT+02:00, Richard Sandiford > wrote: >>>After the previous patch, it seems more natural to apply the >>>PARAM_SLP_MAX_INSNS_IN_BB threshold as soon as we know what >>>the

Re: Move code out of vect_slp_analyze_bb_1

2019-10-20 Thread Richard Sandiford
Richard Biener writes: > On October 19, 2019 5:06:40 PM GMT+02:00, Richard Sandiford > wrote: >>After the previous patch, it seems more natural to apply the >>PARAM_SLP_MAX_INSNS_IN_BB threshold as soon as we know what >>the region is, rather than delaying it to vect_slp_analyze_bb_1. >>(But rat

Re: Move code out of vect_slp_analyze_bb_1

2019-10-19 Thread Richard Biener
On October 19, 2019 5:06:40 PM GMT+02:00, Richard Sandiford wrote: >After the previous patch, it seems more natural to apply the >PARAM_SLP_MAX_INSNS_IN_BB threshold as soon as we know what >the region is, rather than delaying it to vect_slp_analyze_bb_1. >(But rather than carve out the biggest r