Re: [Patch ARM] Fix PR50022

2011-08-15 Thread Ramana Radhakrishnan
> > fails to build, rename it everywhere if needed. Yes - sorry . I had fixed this up on Friday after accidentally sending a wrong version. The final version is attached. Bootstrap with C and Ada completed ok with this patch. Regression tests on a cross-compiler showed no new failures. Committed

Re: [Patch ARM] Fix PR50022

2011-08-14 Thread Matthias Klose
On 08/12/2011 04:16 PM, Ramana Radhakrishnan wrote: >>> @@ -24183,4 +24306,13 @@ arm_attr_length_push_multi(rtx parallel_op, rtx >>> first_op) >>>return 4; >>> } >>> >>> +/* Compute the number of instructions emitted by output_move_double. */ >>> +int >>> +arm_count_output_move_double_insns

Re: [Patch ARM] Fix PR50022

2011-08-12 Thread Richard Earnshaw
On 12/08/11 10:28, Ramana Radhakrishnan wrote: > On 12 August 2011 10:19, Ramana Radhakrishnan > wrote: >> Hi, >> >> The attached patch fixes up PR50022 which exposed a problem in my reworking > > And with the patch even :( ... > > Ramana > > 2011-08-12 Ramana Radhakrishnan > >PR ta

Re: [Patch ARM] Fix PR50022

2011-08-12 Thread Ramana Radhakrishnan
On 12 August 2011 10:19, Ramana Radhakrishnan wrote: > Hi, > > The attached patch fixes up PR50022 which exposed a problem in my reworking And with the patch even :( ... Ramana 2011-08-12 Ramana Radhakrishnan PR target/50022 * config/arm/arm.c (output_move_double): Add 2 param

[Patch ARM] Fix PR50022

2011-08-12 Thread Ramana Radhakrishnan
Hi, The attached patch fixes up PR50022 which exposed a problem in my reworking the movdi_vfp_cortex_a8 patterns. The problem was the ce_count attribute wasn't being set at all for alternatives that would have more than one assembler instruction that could be generated as a result of predication.