This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

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

From: Carrot Wei <carrot at google dot com>
To: Mike Stump <mikestump at comcast dot net>
Cc: Richard Earnshaw <rearnsha at arm dot com>, Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>, Paul Brook <paul at codesourcery dot com>, Nick Clifton <nickc at redhat dot com>, gcc-patches at gcc dot gnu dot org, ramana dot radhakrishnan at arm dot com
Date: Tue, 29 Mar 2011 18:14:17 +0800
Subject: Re: [PATCH: ARM] PR 45335 Use ldrd and strd to access two consecutive words
References: <AANLkTim+nD_6m5PwDBkz0BVz1Ftr+DYyih17nXX5LMue@mail.gmail.com> <AANLkTi=rdaKqDmLppspFbPuVKdtuw-PiXF89+w3Jk6u=@mail.gmail.com> <4D2C66FB.9090100@redhat.com> <201101121319.53218.paul@codesourcery.com> <1294844819.15132.1.camel@e102346-lin.cambridge.arm.com> <AANLkTi=8GfdUNfme1GgjAXL+S+N+N11PdzJq90AQ6qqo@mail.gmail.com> <AANLkTimMD0tzDsLuXQZwJPaS7V=r6=_d2p-1iXuy=P9d@mail.gmail.com> <AANLkTi=qH6Svg9fjCyDX1ctYHNd2N4=ax6nn2bZo8R-p@mail.gmail.com> <1294998680.28104.7.camel@e102346-lin.cambridge.arm.com> <AANLkTi=yvvsJVJwymkE5nTL-q8RnW+WXMf5GNw1sTqpe@mail.gmail.com> <AD81B65F-6171-4E83-A1B8-78C0A69F54F6@comcast.net>

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  <carrot@google.com>

        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 patterns and
        related peephole2.
        (strd, strd_reg1, strd_reg2 and peephole2): New insn patterns and
        related peephole2.
        * gcc/config/arm/arm-protos.h (arm_check_ldrd_operands): New prototype.
        (arm_legitimate_ldrd_p): New prototype.
        (arm_output_ldrd): New prototype.
        * gcc/config/arm/arm.c (arm_check_ldrd_operands): New function.
        (arm_legitimate_ldrd_p): New function.
        (arm_output_ldrd): New function.


2011-03-29  Wei Guozhi  <carrot@google.com>

        PR target/45335
        * gcc.target/arm/pr45335.c: New test.
        * gcc.target/arm/pr45335-2.c: New test.
        * gcc.target/arm/pr45335-3.c: New test.
        * gcc.target/arm/pr40457-1.c: Add another possible output "ldrd".
        * gcc.target/arm/pr40457-2.c: Changed to store 3 words.
        * gcc.target/arm/pr40457-3.c: Changed to store 3 words.


On Thu, Mar 24, 2011 at 8:25 AM, Mike Stump <mikestump@comcast.net> wrote:
> 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.
>

Attachment: pr45335.txt
Description: Text document

References:
- Re: [PATCH: ARM] PR 45335 Use ldrd and strd to access two consecutive words
  - From: Mike Stump

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]