Re: [PATCH] Combine related fail of gcc.target/powerpc/ti_math1.c

2015-06-22 Thread Alan Modra
On Mon, Jun 22, 2015 at 09:24:07AM +0200, Eric Botcazou wrote: > > * rtlanal.c (commutative_operand_precedence): Correct comments. > > * simplify-rtx.c (simplify_plus_minus_op_data_cmp): Delete forward > > declaration. Return an int. Distinguish REG,REG return from > > others. > >

Re: [PATCH] Combine related fail of gcc.target/powerpc/ti_math1.c

2015-06-22 Thread Eric Botcazou
> This patch fixes > FAIL: gcc.target/powerpc/ti_math1.c scan-assembler-times adde 1 > a failure caused by combine simplifying this i2src > > (plus:DI (plus:DI (reg:DI 165 [ val+8 ]) > (reg:DI 169 [+8 ])) > (reg:DI 76 ca)) > > to this > > (plus:DI (plus:DI (reg:DI 76 ca) > (r

Re: [PATCH] Combine related fail of gcc.target/powerpc/ti_math1.c

2015-06-21 Thread Alan Modra
On Fri, May 22, 2015 at 04:04:19PM +0930, Alan Modra wrote: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02055.html > * rtlanal.c (commutative_operand_precedence): Correct comments. > * simplify-rtx.c (simplify_plus_minus_op_data_cmp): Delete forward > declaration. Return an in

[PATCH] Combine related fail of gcc.target/powerpc/ti_math1.c

2015-05-21 Thread Alan Modra
This patch fixes FAIL: gcc.target/powerpc/ti_math1.c scan-assembler-times adde 1 a failure caused by combine simplifying this i2src (plus:DI (plus:DI (reg:DI 165 [ val+8 ]) (reg:DI 169 [+8 ])) (reg:DI 76 ca)) to this (plus:DI (plus:DI (reg:DI 76 ca) (reg:DI 165 [ val+8 ]))