This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: combiner bug and fix
- From: Denis Chertykov <denisc at overta dot ru>
- To: Andrew Pinski <pinskia at physics dot uc dot edu>
- Cc: Denis Chertykov <denisc at overta dot ru>, kenner at vlsi1 dot ultra dot nyu dot edu (Richard Kenner), gcc-patches at gcc dot gnu dot org
- Date: 30 Jun 2003 21:25:18 +0400
- Subject: Re: combiner bug and fix
- References: <2268AF12-AB10-11D7-A233-000393A6D2F2@physics.uc.edu>
Andrew Pinski <pinskia@physics.uc.edu> writes:
> Somebody needs to file a bug emacs saying the copyright line is now
> multiline
> but they do not handle it because this is at least the third time I
> seen this.
Oops.
The right patch.
Is this ok for mainline and 3.3 branch ?
2003-06-30 Denis Chertykov <denisc@overta.ru>
Richard Kenner <kenner@vlsi1.ultra.nyu.edu>
* combine.c (gen_binary): Handle the CLOBBER rtx and
don't build a binary operation with it.
Index: combine.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/combine.c,v
retrieving revision 1.364
diff -c -3 -p -r1.364 combine.c
*** combine.c 27 Jun 2003 19:42:24 -0000 1.364
--- combine.c 30 Jun 2003 17:15:53 -0000
*************** gen_binary (enum rtx_code code, enum mac
*** 10170,10175 ****
--- 10170,10180 ----
rtx result;
rtx tem;
+ if (GET_CODE (op0) == CLOBBER)
+ return op0;
+ else if (GET_CODE (op1) == CLOBBER)
+ return op1;
+
if (GET_RTX_CLASS (code) == 'c'
&& swap_commutative_operands_p (op0, op1))
tem = op0, op0 = op1, op1 = tem;