This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][lower-subreg] Fix PR87507
- From: Peter Bergner <bergner at linux dot ibm dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: gcc-patches at gcc dot gnu dot org, "ian at airs dot com" <ian at airs dot com>, Richard Henderson <rth at twiddle dot net>, Segher Boessenkool <segher at kernel dot crashing dot org>
- Date: Tue, 13 Nov 2018 20:21:22 -0600
- Subject: Re: [PATCH][lower-subreg] Fix PR87507
- References: <27077db7-b4bf-689c-2438-a5bab4753871@linux.ibm.com> <4040235.T82DKmk4EN@polaris> <fed50e49-267b-c39a-320c-9db1a368f4f0@linux.ibm.com> <4868166.Ub7ZXv77v3@polaris>
On 11/13/18 6:06 PM, Eric Botcazou wrote:
>> PR rtl-optimization/87507
>> * lower-subreg.c (operand_for_simple_move_operator): New function.
>> (simple_move): Strip simple operators.
>> (find_pseudo_copy): Likewise.
>> (resolve_operand_for_simple_move_operator): New function.
>> (resolve_simple_move): Strip simple operators and swap operands.
>>
>> gcc/testsuite/
>> PR rtl-optimization/87507
>> * gcc.target/powerpc/pr87507.c: New test.
>> * gcc.target/powerpc/pr68805.c: Update expected results.
>
> OK with the s/simple/swap/ change suggested by Richard.
Ok, I used operand_for_swap_move_operator like Richard suggested and also
did a similar change for resolve_operand_for_swap_move_operator to keep
things consistent. Now committed. Thanks!
Peter