Re: [ARM] Fix a performance regression from the fix for PR49030

2011-09-22 Thread Richard Sandiford
Ramana Radhakrishnan writes: > On 20 September 2011 09:38, Richard Sandiford >> Otherwise, all expanders use expandable_comparison_operator instead. >> This restores the previous behaviour for them, and I went through each >> one to try to make sure that it was handled correctly. >> >> Tested on a

Re: [ARM] Fix a performance regression from the fix for PR49030

2011-09-21 Thread Ramana Radhakrishnan
On 20 September 2011 09:38, Richard Sandiford wrote: > My fix for PR49030 had the unintended side-effect of forcing libcalls > to be used for some DImode comparisons.  The problem (which I should > have noticed at the time, sorry) is that arm_comparison_operator is > used for both expanders and in

[ARM] Fix a performance regression from the fix for PR49030

2011-09-20 Thread Richard Sandiford
My fix for PR49030 had the unintended side-effect of forcing libcalls to be used for some DImode comparisons. The problem (which I should have noticed at the time, sorry) is that arm_comparison_operator is used for both expanders and insns. The patch fixed the definition for insns, but made it to