Re: [PATCH,RFC] combine: Remove use_crosses_set_p

2017-12-06 Thread Kyrill Tkachov
On 06/12/17 11:51, Christophe Lyon wrote: Hi, On 4 December 2017 at 18:18, Eric Botcazou wrote: >> Since there are no comments, I'll commit it now. > > The idea looks interesting but the timing is a bit more questionable. > > -- > Eric Botcazou Since this was committed (r255384), I've noticed

Re: [PATCH,RFC] combine: Remove use_crosses_set_p

2017-12-06 Thread Christophe Lyon
Hi, On 4 December 2017 at 18:18, Eric Botcazou wrote: >> Since there are no comments, I'll commit it now. > > The idea looks interesting but the timing is a bit more questionable. > > -- > Eric Botcazou Since this was committed (r255384), I've noticed a regression on arm-none-linux-gnueabi --wit

Re: [PATCH,RFC] combine: Remove use_crosses_set_p

2017-12-04 Thread Eric Botcazou
> Since there are no comments, I'll commit it now. The idea looks interesting but the timing is a bit more questionable. -- Eric Botcazou

Re: [PATCH,RFC] combine: Remove use_crosses_set_p

2017-12-04 Thread Segher Boessenkool
On Thu, Nov 30, 2017 at 09:26:37AM +, Segher Boessenkool wrote: > This removes use_crosses_set_p, and uses modified_between_p instead > everywhere it was used. This improves optimisation. I'm a little bit > worried it might increase compile time; so far I haven't seen it do > that though. >