Re: [PATCH: PR target/46975] Replace 32 bit instructions with 16 bit instructions in thumb2

2011-03-18 Thread Carrot Wei
Ping On Sat, Dec 18, 2010 at 3:30 AM, Carrot Wei wrote: > On Fri, Dec 17, 2010 at 4:18 AM, Richard Earnshaw wrote: >> >> On Thu, 2010-12-16 at 14:45 -0800, Carrot Wei wrote: >>> Hi >>> >>> Compile the following c code with options -march=

[PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-03-26 Thread Carrot Wei
Hi As described in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47855, there are many insn patterns don't compute attribute length correctly. This patch is the first and simplest part of the fixing. This patch has been tested on qemu. thanks Carrot ChangeLog: 2011-03-26 Wei G

Re: [PATCH: ARM] PR 45335 Use ldrd and strd to access two consecutive words

2011-03-29 Thread Carrot Wei
Thank you for the knowledge. I've updated the insn patterns accordingly. Again tested on arm qemu. thanks Carrot ChangeLog: 2011-03-29 Wei Guozhi PR target/45335 * gcc/config/arm/ldmstm.md (ldm2_ia, stm2_ia, ldm2_ib, stm2_ib, ldm2_da, stm2_da, ldm2_db, stm2_db)

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-03-30 Thread Carrot Wei
Hi Ramana On Wed, Mar 30, 2011 at 6:35 AM, Ramana Radhakrishnan wrote: > Hi Carrot, > > >        How about adding an alternative only enabled for T2 that uses the `l' > constraint and inventing new constraints for some of the constant values > that are valid for 16 bit in

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-03-31 Thread Carrot Wei
Hi Ramana On Thu, Mar 31, 2011 at 11:57 PM, Ramana Radhakrishnan wrote: >>> Hi Carrot, >>>        How about adding an alternative only enabled for T2 that uses the >>> `l' >>> constraint and inventing new constraints for some of the constant values &

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-04-05 Thread Carrot Wei
Committed as r172017. thanks Carrot On Tue, Apr 5, 2011 at 9:32 PM, Ramana Radhakrishnan wrote: > >> >> The patch was tested on arm qemu without regression, OK for install? > > Ok if no regressions. > > cheers > Ramana >

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-04-07 Thread Carrot Wei
On Thu, Apr 7, 2011 at 5:31 PM, Richard Sandiford wrote: > Hi Carrot, > > Sorry if this has already been reported, but the patch breaks bootstrap > of arm-linux-gnueabi (or cross builds with --enable-werror).  The problem > is that this... > > uses a statement expression --

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-04-08 Thread Carrot Wei
Hi This patch moves the length computation of push_multi into a C function to avoid the usage of GNU extension. Tested on qemu without regression. OK to install? thanks Carrot ChangeLog: 2011-04-08 Wei Guozhi PR target/47855 * config/arm/arm-protos.h

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

2011-04-08 Thread Carrot Wei
Sorry, which pattern in thumb2.md? thanks Carrot On Fri, Apr 8, 2011 at 4:35 PM, Ramana Radhakrishnan wrote: > On 08/04/11 08:36, Carrot Wei wrote: >> >> Hi >> >> This patch moves the length computation of push_multi into a C >> function to avoid the usage of

[PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns, 2/3

2011-04-08 Thread Carrot Wei
ength is computed in separate C functions. The patch has been tested on qemu with both ARM and thumb mode. thanks Carrot ChangeLog: 2011-04-08 Wei Guozhi PR target/47855 * config/arm/arm-protos.h (arm_attr_length_movqi): New prototype. (arm_attr_length_addsi3): Likewise.

<    1   2