This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] h8300.md: Optimize stack pushes.
Hi Jeff,
> >I don't think that would be a problem unless the interrupt service
> >routine starts digging into the stack upward and tries to do something
> >with it.
> That's not an unusual thing to do :-)
Oh, no! :-)
> I think you'd be better off keeping stack adjustments as atomic operations.
Reverted like so.
2003-07-07 Kazu Hirata <kazu@cs.umass.edu>
* config/h8300/h8300.md (pushqi1_h8300hs): Revert my patch
today.
(pushhi1_h8300hs): Likewise.
Index: h8300.md
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/h8300/h8300.md,v
retrieving revision 1.230
retrieving revision 1.229
diff -u -r1.230 -r1.229
--- h8300.md 7 Jul 2003 15:46:02 -0000 1.230
+++ h8300.md 6 Jul 2003 23:05:09 -0000 1.229
@@ -126,7 +126,7 @@
(match_operand:QI 0 "register_operand" "r"))])]
"(TARGET_H8300H || TARGET_H8300S)
&& REGNO (operands[0]) != SP_REG"
- "mov.w\\t%f0,@-er7\;subs\\t#2,er7"
+ "mov.l\\t%S0,@-er7"
[(set_attr "length" "4")
(set_attr "cc" "clobber")])
@@ -218,7 +218,7 @@
(match_operand:HI 0 "register_operand" "r"))])]
"(TARGET_H8300H || TARGET_H8300S)
&& REGNO (operands[0]) != SP_REG"
- "mov.w\\t%f0,@-er7\;subs\\t#2,er7"
+ "mov.l\\t%S0,@-er7"
[(set_attr "length" "4")
(set_attr "cc" "clobber")])