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: David Edelsohn <dje dot gcc at gmail dot com>
- To: Peter Bergner <bergner at vnet dot ibm 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, 2 Mar 2012 11:19:15 -0500
- Subject: Re: [PATCH, rs6000][trunk,4.7,4.6,4.5] Fix PR52457, wrong code for -O1 -m32 -mcpu=power7
- Authentication-results: mr.google.com; spf=pass (google.com: domain of dje.gcc@gmail.com designates 10.60.22.40 as permitted sender) smtp.mail=dje.gcc@gmail.com; dkim=pass header.i=dje.gcc@gmail.com
- References: <1330703856.29904.3.camel@otta>
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.
Thanks, David
- Follow-Ups:
- Re: [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