(shiftcosts): For SH3, max cost of arithmetic right shift is 3.

(shiftcosts): For SH3, max cost of arithmetic right
shift is 3.
(expand_ashiftrt): For SH3, if shift cost is more than 3, then
call gen_ashrsi3_d to use shad instruction.

From-SVN: r10674

diff --git a/gcc/config/sh/sh.c b/gcc/config/sh/sh.c
index 5f072d80221edc996a9790a5f584c6935149cd71..e89280e499fae0e63ea98d44e2f7b01266fc4cdd 100644 (file)
--- a/gcc/config/sh/sh.c
+++ b/gcc/config/sh/sh.c
@@ -785,7 +785,13 @@ shiftcosts (x)
 
   /* Otherwise, return the true cost in instructions.  */
   if (GET_CODE (x) == ASHIFTRT)
-    return ashiftrt_insns[value];
+    {
+      int cost = ashiftrt_insns[value];
+      /* If SH3, then we put the constant in a reg and use shad.  */
+      if (TARGET_SH3 && cost > 3)
+       cost = 3;
+      return cost;
+    }
   else
     return shift_insns[value];
 }
@@ -881,9 +887,6 @@ gen_ashift (type, n, reg)
 /* Output RTL to split a constant shift into its component SH constant
    shift instructions.  */
    
-/* ??? For SH3, should reject constant shifts when slower than loading the
-   shift count into a register?  */
-
 int
 gen_shifty_op (code, operands)
      int code;
@@ -931,12 +934,21 @@ expand_ashiftrt (operands)
   tree func_name;
   int value;
 
-  if (TARGET_SH3 && GET_CODE (operands[2]) != CONST_INT)
+  if (TARGET_SH3)
     {
-      rtx count = copy_to_mode_reg (SImode, operands[2]);
-      emit_insn (gen_negsi2 (count, count));
-      emit_insn (gen_ashrsi3_d (operands[0], operands[1], count));
-      return 1;
+      if (GET_CODE (operands[2]) != CONST_INT)
+       {
+         rtx count = copy_to_mode_reg (SImode, operands[2]);
+         emit_insn (gen_negsi2 (count, count));
+         emit_insn (gen_ashrsi3_d (operands[0], operands[1], count));
+         return 1;
+       }
+      else if (ashiftrt_insns[INTVAL (operands[2])] > 3)
+       {
+         rtx count = force_reg (SImode, GEN_INT (- INTVAL (operands[2])));
+         emit_insn (gen_ashrsi3_d (operands[0], operands[1], count));
+         return 1;
+       }
     }
   if (GET_CODE (operands[2]) != CONST_INT)
     return 0;