This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000][trunk,4.7,4.6,4.5] Fix PR52457, wrong code for -O1 -m32 -mcpu=power7
- From: Jakub Jelinek <jakub at redhat dot com>
- To: David Edelsohn <dje dot gcc at gmail dot com>
- Cc: Peter Bergner <bergner at vnet dot ibm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Michael Meissner <meissner at linux dot ibm dot com>
- Date: Fri, 2 Mar 2012 18:15:46 +0100
- Subject: Re: [PATCH, rs6000][trunk,4.7,4.6,4.5] Fix PR52457, wrong code for -O1 -m32 -mcpu=power7
- References: <1330703856.29904.3.camel@otta> <CAGWvnym24pk9vQLH26GnHyV=R3zc1avYF1jk5UaFs9r3qQUxRw@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Mar 02, 2012 at 11:19:15AM -0500, David Edelsohn wrote:
> On Fri, Mar 2, 2012 at 10:57 AM, Peter Bergner <bergner@vnet.ibm.com> wrote:
> > The patch below fixes a typo in the vsx_set_<mode> pattern that causes
> > wrong code to be generated when using -mcpu=power7. ?This passed bootstrap
> > and regression testing on trunk using powerpc64-linux. ?Ok for trunk?
> >
> > This is also broken on the 4.7, 4.6 and 4.5 branches, is this ok for
> > those release branches once the branches are open for fixes and my
> > bootstrapping/regtesting are complete?
> >
> > Peter
> >
> > ? ? ? ?* gcc/config/rs6000/vsx.md (vsx_set_<mode>): Reorder operands.
> > ? ? ? ?* gcc/testsuite/gcc.target/powerpc/pr52457.c: New test.
>
> Okay everywhere, but please make sure that Richi and Jakub are okay
> with the patch for GCC 4.7.
This is ok for 4.7.0.
Jakub
- References:
- [PATCH, rs6000][trunk,4.7,4.6,4.5] Fix PR52457, wrong code for -O1 -m32 -mcpu=power7
- Re: [PATCH, rs6000][trunk,4.7,4.6,4.5] Fix PR52457, wrong code for -O1 -m32 -mcpu=power7