This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] More cleanup: inline preserve_subexpressions_p into its callers
- From: Paolo Bonzini <paolo dot bonzini at polimi dot it>
- To: Richard Earnshaw <rearnsha at buzzard dot freeserve dot co dot uk>
- Cc: Paolo Bonzini <bonzini at gnu dot org>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 18 Aug 2004 09:02:05 +0200
- Subject: Re: [PATCH] More cleanup: inline preserve_subexpressions_p into its callers
- References: <200408172302.i7HN2Yig025902@merlin.buzzard.freeserve.co.uk>
I'll fix the arm port, but note that c4x also uses
preserve_subexpressions_p.
Fixed thus, committed as obvious (optimize is already checked in the
conditional).
Paolo
Index: c4x/c4x.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/c4x/c4x.c,v
retrieving revision 1.159
diff -u -r1.159 c4x.c
--- c4x/c4x.c 15 Aug 2004 15:45:09 -0000 1.159
+++ c4x/c4x.c 18 Aug 2004 06:55:59 -0000
@@ -3932,7 +3932,6 @@
&& TARGET_HOIST
&& optimize > 0
&& GET_CODE (operands[1]) == CONST_INT
- && preserve_subexpressions_p ()
&& rtx_cost (operands[1], code) > 1)
operands[1] = force_reg (mode, operands[1]);
@@ -3950,7 +3949,6 @@
&& TARGET_HOIST
&& optimize > 1
&& GET_CODE (operands[2]) == CONST_INT
- && preserve_subexpressions_p ()
&& rtx_cost (operands[2], code) > 1)
operands[2] = force_reg (mode, operands[2]);