Re: [PATCH 8/8] Use rank_for_schedule to as tie-breaker in model_order_p

2014-10-21 Thread Maxim Kuvyrkov
On Oct 22, 2014, at 9:24 AM, Richard Sandiford wrote: > Maxim Kuvyrkov writes: >> On Oct 21, 2014, at 9:11 PM, Richard Sandiford >> wrote: >> >>> Maxim Kuvyrkov writes: This patch improves model_order_p to use non-reg-pressure version of rank_for_schedule when it needs to break the

Re: [PATCH 8/8] Use rank_for_schedule to as tie-breaker in model_order_p

2014-10-21 Thread Richard Sandiford
Maxim Kuvyrkov writes: > On Oct 21, 2014, at 9:11 PM, Richard Sandiford > wrote: > >> Maxim Kuvyrkov writes: >>> This patch improves model_order_p to use non-reg-pressure version of >>> rank_for_schedule when it needs to break the tie. At the moment it is >>> comparing INSN_PRIORITY by itself,

Re: [PATCH 8/8] Use rank_for_schedule to as tie-breaker in model_order_p

2014-10-21 Thread Maxim Kuvyrkov
On Oct 21, 2014, at 9:11 PM, Richard Sandiford wrote: > Maxim Kuvyrkov writes: >> This patch improves model_order_p to use non-reg-pressure version of >> rank_for_schedule when it needs to break the tie. At the moment it is >> comparing INSN_PRIORITY by itself, and it seems prudent to outsourc

Re: [PATCH 8/8] Use rank_for_schedule to as tie-breaker in model_order_p

2014-10-21 Thread Richard Sandiford
Maxim Kuvyrkov writes: > This patch improves model_order_p to use non-reg-pressure version of > rank_for_schedule when it needs to break the tie. At the moment it is > comparing INSN_PRIORITY by itself, and it seems prudent to outsource > that to rank_for_schedule. Do you have an example of wher

[PATCH 8/8] Use rank_for_schedule to as tie-breaker in model_order_p

2014-10-20 Thread Maxim Kuvyrkov
Hi, This patch improves model_order_p to use non-reg-pressure version of rank_for_schedule when it needs to break the tie. At the moment it is comparing INSN_PRIORITY by itself, and it seems prudent to outsource that to rank_for_schedule. Bootstrapped on x86_64-linux-gnu and regtested on arm-