This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Fix PR target/72804: Poor code gen with -mvsx-timode
- From: Peter Bergner <bergner at vnet dot ibm dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>
- Date: Wed, 16 Aug 2017 17:56:09 -0500
- Subject: Re: [PATCH, rs6000] Fix PR target/72804: Poor code gen with -mvsx-timode
- Authentication-results: sourceware.org; auth=none
- References: <9356351a-ef7e-05ff-9f13-34d4b2756f15@vnet.ibm.com> <20170816223014.GX13471@gate.crashing.org>
On 8/16/17 5:30 PM, Segher Boessenkool wrote:
> On Mon, Aug 14, 2017 at 04:28:25PM -0500, Peter Bergner wrote:
>> + mr %0,%L1; mr %L0,%1
>
> mr %0,%L1\;mr %L0,%1
So you want the ';' escaped and the space removed? Ok.
>> + [(set (match_operand:VSX_TI 0 "int_reg_operand" "")
>
> You can leave out the default ""?
Cut/paste of another peephole2. I'll try without it, thanks.
>> +/* { dg-do compile { target { powerpc64*-*-* } } } */
>
> powerpc64*-*-* is never correct, we are biarch. { target lp64 } instead?
Ah yes, that is better.
I'll make the above changes and commit after another quick test cycle.
Thanks!
Peter