Re: ARM: Emit conditions in push_multi

2011-09-12 Thread Jakub Jelinek
Hi! This last hunk is wrong, for sprintf you need to use %% instead of %. On Wed, Sep 07, 2011 at 09:23:36PM +0200, Bernd Schmidt wrote: > @@ -10631,7 +10633,7 @@ (define_insn "*push_fp_multi" >{ > char pattern[100]; > > -sprintf (pattern, \"sfmfd\\t%%1, %d, [%%m0]!\", XVECLEN (ope

Re: ARM: Emit conditions in push_multi

2011-09-08 Thread Ramana Radhakrishnan
> New patch below. Tested on arm-eabi sim with a few multilibs. OK. Ramana

Re: ARM: Emit conditions in push_multi

2011-09-07 Thread Bernd Schmidt
On 09/07/11 10:28, Ramana Radhakrishnan wrote: > On 2 September 2011 12:42, Bernd Schmidt wrote: >> On 09/02/11 12:35, Ramana Radhakrishnan wrote: >>> On 1 September 2011 12:50, Bernd Schmidt wrote: Shrink-wrapping tests on ARM had one additional failure, which I could track down to a s

Re: ARM: Emit conditions in push_multi

2011-09-07 Thread Ramana Radhakrishnan
On 2 September 2011 12:42, Bernd Schmidt wrote: > On 09/02/11 12:35, Ramana Radhakrishnan wrote: >> On 1 September 2011 12:50, Bernd Schmidt wrote: >>> Shrink-wrapping tests on ARM had one additional failure, which I could >>> track down to a stmfd instruction being emitted where an stmhifd was >

Re: ARM: Emit conditions in push_multi

2011-09-02 Thread Bernd Schmidt
On 09/02/11 12:35, Ramana Radhakrishnan wrote: > On 1 September 2011 12:50, Bernd Schmidt wrote: >> Shrink-wrapping tests on ARM had one additional failure, which I could >> track down to a stmfd instruction being emitted where an stmhifd was >> intended. The following patch fixes the testcase; fu

Re: ARM: Emit conditions in push_multi

2011-09-02 Thread Ramana Radhakrishnan
On 1 September 2011 12:50, Bernd Schmidt wrote: > Shrink-wrapping tests on ARM had one additional failure, which I could > track down to a stmfd instruction being emitted where an stmhifd was > intended. The following patch fixes the testcase; full tests running > now. Ok? IIUC this should have b