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: Peter Bergner <bergner at vnet dot ibm dot com>
- To: David Edelsohn <dje dot gcc at gmail dot com>
- Cc: "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, 02 Mar 2012 12:36:20 -0600
- 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>
On Fri, 2012-03-02 at 11:19 -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?
> >
> > * 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.
Ok, I have committed the patch to trunk and the 4.7 branch given Jakub's
ok in his email. I'll commit the patch to the 4.6 and 4.5 branches once
my testing has completed. Thanks.
Peter
- 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