This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix shufpd operands (PR target/46880)
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, gcc-patches at gcc dot gnu dot org
- Date: Tue, 21 Dec 2010 23:19:04 +0100
- Subject: Re: [PATCH] Fix shufpd operands (PR target/46880)
- References: <20101221205225.GF16156@tyan-ft48-01.lab.bos.redhat.com>
On Tue, Dec 21, 2010 at 9:52 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> In these two insns, shufpd insn is used in an alternative which has
> constraint 0 on operand %2 and constraint x on operand %1, so it
> is obviously wrong to use operand %2 as source operand, because
> that's the same register as %0 and shuftpd $2, %xmm0, %xmm0
> (or any other where src == dst) is a nop).
>
> Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for
> trunk?
>
> 2010-12-21 ?Jakub Jelinek ?<jakub@redhat.com>
>
> ? ? ? ?PR target/46880
> ? ? ? ?* config/i386/sse.md (sse2_loadlpd, sse2_movsd): Fix shufpd source
> ? ? ? ?operand.
>
> ? ? ? ?* gcc.target/i386/pr46880.c: New test.
OK.
Thanks,
Uros.