From 4246e0c55bf669cfa59fc68e6dedea70a6cabdbd Mon Sep 17 00:00:00 2001
From: Jeff Law <law@gcc.gnu.org>
Date: Mon, 16 Dec 1996 10:18:18 -0700
Subject: [PATCH] mn10300.c (expand_epilogue): Restore registers in the "ret"
 instruction instead of a separate movm instruction.

        * mn10300/mn10300.c (expand_epilogue): Restore registers in the
        "ret" instruction instead of a separate movm instruction.
        Support possible stack deallocation in "ret" instruction too.
        * mn10300.md (return_internal): Use "ret" instead of "rets";
        restore registers and deallocate stack as needed.
        (load_movm): Delete unused pattern.

        * mn10300/mn10300.h (SMALL_REGISTER_CLASSES): Define.

From-SVN: r13311
---
 gcc/config/mn10300/mn10300.c  | 19 ++++++++++---------
 gcc/config/mn10300/mn10300.h  |  3 +++
 gcc/config/mn10300/mn10300.md | 11 ++++-------
 3 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/gcc/config/mn10300/mn10300.c b/gcc/config/mn10300/mn10300.c
index 9218584baf0..19f57c0a632 100644
--- a/gcc/config/mn10300/mn10300.c
+++ b/gcc/config/mn10300/mn10300.c
@@ -244,17 +244,18 @@ expand_epilogue ()
 			     frame_pointer_rtx,
 			     GEN_INT (-20)));
       emit_move_insn (stack_pointer_rtx, frame_pointer_rtx);
+      size = 0;
+    }
+  else if (size > 255)
+    {
+      emit_insn (gen_addsi3 (stack_pointer_rtx,
+			     stack_pointer_rtx,
+			     GEN_INT (size)));
+      size = 0;
     }
-  else if (size)
-    emit_insn (gen_addsi3 (stack_pointer_rtx,
-			   stack_pointer_rtx,
-			   GEN_INT (size)));
-
-  /* And restore the registers.  */
-  emit_insn (gen_load_movm ());
 
-  /* And return.  */
-  emit_jump_insn (gen_return_internal ());
+  /* Deallocate remaining stack, restore registers and return.  And return.  */
+  emit_jump_insn (gen_return_internal (GEN_INT (size)));
 }
 
 /* Update the condition code from the insn.  */
diff --git a/gcc/config/mn10300/mn10300.h b/gcc/config/mn10300/mn10300.h
index b8b416b99af..d4bce39ad5a 100644
--- a/gcc/config/mn10300/mn10300.h
+++ b/gcc/config/mn10300/mn10300.h
@@ -177,6 +177,9 @@ extern int target_flags;
 #define MODES_TIEABLE_P(MODE1, MODE2) \
   (MODE1 == MODE2 || GET_MODE_SIZE (MODE1) <= 4 && GET_MODE_SIZE (MODE2) <= 4)
 
+/* 4 data, and effectively 3 address registers is small as far as I'm
+   concerned.  */
+#define SMALL_REGISTER_CLASSES 1
 
 /* Define the classes of registers for register constraints in the
    machine description.  Also define ranges of constants.
diff --git a/gcc/config/mn10300/mn10300.md b/gcc/config/mn10300/mn10300.md
index c3306386782..443ee999c19 100644
--- a/gcc/config/mn10300/mn10300.md
+++ b/gcc/config/mn10300/mn10300.md
@@ -655,11 +655,14 @@
   "rets"
   [(set_attr "cc" "clobber")])
 
+;; This insn restores the callee saved registers and does a return, it
+;; can also deallocate stack space.
 (define_insn "return_internal"
   [(const_int 0)
+   (match_operand:SI 0  "const_int_operand" "i")
    (return)]
   ""
-  "rets"
+  "ret [d2,d3,a2,a3],%0"
   [(set_attr "cc" "clobber")])
 
 (define_insn "store_movm"
@@ -667,9 +670,3 @@
   ""
   "movm [d2,d3,a2,a3],(sp)"
   [(set_attr "cc" "none")])
-
-(define_insn "load_movm"
-  [(const_int 2)]
-  ""
-  "movm (sp),[d2,d3,a2,a3]"
-  [(set_attr "cc" "none")])
-- 
2.30.2