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

2011-06-13 Thread Ramana Radhakrishnan
This caused http://gcc.gnu.org/PR49398. You might also consider putting the PR numbers in your commit log in the future that these patches attempt to fix so that the PR's in bugzilla get the associated commit message. Please add links to this commit to the individual PRs this commit attempts to fi

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

2011-06-07 Thread Nick Clifton
Hi 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): Add condition !arm_arch7 to these insns. (ldrd, ldrd_reg1, ldrd_reg2 and peephole2): New insn patte

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): Add co

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

2011-03-23 Thread Mike Stump
On Jan 18, 2011, at 6:59 AM, Carrot Wei wrote: > +(define_insn "*ldrd" > + [(parallel [(set (match_operand:SI 0 "arm_hard_register_operand" "") parallel is implicit, you can safely remove it from all define_insns.

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

2011-03-15 Thread Carrot Wei
The trunk is opened again, could any maintainers continue to review this patch? thanks Carrot On Tue, Jan 18, 2011 at 10:59 PM, Carrot Wei wrote: > Ramana's method is to put the instruction output and counting in on place. > So it's easy to keep them synchronized. > > My latest version of patch