[patch] mn10300.md: Remove a define_peephole.

Kazu Hirata kazu@cs.umass.edu
Fri Mar 5 04:25:00 GMT 2004


Hi,

Attached is a patch to remove a define_peephole.

This peephole is effectively dead because we have combine_instructions
and combine_stack_adjustments.

Compiling a bunch of real-world testcases, I saw this trigger only
twice out of almost a million lines of assembly code.

Tested on mn10300-elf.  OK to apply?

Kazu Hirata

2004-03-04  Kazu Hirata  <kazu@cs.umass.edu>

	* config/mn10300/mn10300.md: Remove a define_peephole.

Index: mn10300.md
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/mn10300/mn10300.md,v
retrieving revision 1.54
diff -u -r1.54 mn10300.md
--- mn10300.md	4 Feb 2004 19:15:24 -0000	1.54
+++ mn10300.md	5 Mar 2004 02:57:06 -0000
@@ -2484,23 +2484,6 @@
 }"
   [(set_attr "cc" "clobber")])
 
-;; Try to combine consecutive updates of the stack pointer (or any
-;; other register for that matter).
-(define_peephole
-  [(set (match_operand:SI 0 "register_operand" "=dxay")
-	(plus:SI (match_dup 0)
-		 (match_operand 1 "const_int_operand" "")))
-   (set (match_dup 0)
-	(plus:SI (match_dup 0)
-		 (match_operand 2 "const_int_operand" "")))]
-  ""
-  "*
-{
-  operands[1] = GEN_INT (INTVAL (operands[2]) + INTVAL (operands[1]));
-  return \"add %1,%0\";
-}"
-  [(set_attr "cc" "clobber")])
-
 ;;
 ;; We had patterns to check eq/ne, but the they don't work because
 ;; 0x80000000 + 0x80000000 = 0x0 with a carry out.



More information about the Gcc-patches mailing list