This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] h8300.md: Use '?' to simplify a pattern.
- From: Kazu Hirata <kazu at cs dot umass dot edu>
- To: lars at nocrew dot org
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 04 Mar 2003 09:15:30 -0500 (EST)
- Subject: Re: [patch] h8300.md: Use '?' to simplify a pattern.
- References: <20030303.201505.26295687.kazu@cs.umass.edu><851y1n5lhm.fsf@junk.nocrew.org>
Hi Lars,
> Are you sure about this? To me it seems likely that the first
> alternative (mov.b+extu.w+extu.l) should have length 6, and the second
> alternative (sub.l+mov.b) should have length 4. Your patch says the
> opposite.
Oops. Thanks for pointing this out. Condition code was also swapped.
Committed as obvious.
Kazu Hirata
2003-03-04 Kazu Hirata <kazu at cs dot umass dot edu>
* config/h8300/h8300.md (*extzv_8_8): Fix cc and lengths.
Index: h8300.md
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/h8300/h8300.md,v
retrieving revision 1.185
diff -u -r1.185 h8300.md
--- h8300.md 4 Mar 2003 01:22:01 -0000 1.185
+++ h8300.md 4 Mar 2003 13:41:45 -0000
@@ -2633,8 +2633,8 @@
"@
mov.b\\t%x1,%w0\;extu.w\\t%f0\;extu.l\\t%S0
sub.l\\t%S0,%S0\;mov.b\\t%x1,%w0"
- [(set_attr "cc" "clobber,set_znv")
- (set_attr "length" "4,6")])
+ [(set_attr "cc" "set_znv,clobber")
+ (set_attr "length" "6,4")])
(define_insn "*extzv_8_16"
[(set (match_operand:SI 0 "register_operand" "=r")