This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Committed] Fix my do_compare_rtx_and_jump breakage.
- From: Roger Sayle <roger at eyesopen dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Tue, 14 Feb 2006 11:12:16 -0700 (MST)
- Subject: Re: [Committed] Fix my do_compare_rtx_and_jump breakage.
On Tue, 14 Feb 2006, Roger Sayle wrote:
> Sorry for not noticing this ommission/latent bug.
> I've committed the following obvious fix as revision 110983.
And equally as obvious, do_compare_rtx_and_jump should also support
GTU and GEU. It's just not my day. Sorry again for the inconvenience.
On the plus side, we clearly missed a lot of optimization opportunities
before! :-}
2006-02-14 Roger Sayle <roger@eyesopen.com>
* dojump.c (do_compare_rtx_and_jump): Also handle multi-word GTU
and GEU.
Index: dojump.c
===================================================================
*** dojump.c (revision 110989)
--- dojump.c (working copy)
*************** do_compare_rtx_and_jump (rtx op0, rtx op
*** 916,921 ****
--- 916,931 ----
if_true_label, if_false_label);
break;
+ case GTU:
+ do_jump_by_parts_greater_rtx (mode, 1, op0, op1,
+ if_false_label, if_true_label);
+ break;
+
+ case GEU:
+ do_jump_by_parts_greater_rtx (mode, 1, op1, op0,
+ if_true_label, if_false_label);
+ break;
+
case LT:
do_jump_by_parts_greater_rtx (mode, 0, op1, op0,
if_false_label, if_true_label);
Roger
--