diff --git a/gcc/ChangeLog b/gcc/ChangeLog index a9edea9c28f2953b4e2b2edb413d958cc6474169..c29a3e737f9a67305d15d5b9bd068eec7ef9d0f0 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2012-03-03 Kaz Kojima <kkojima@gcc.gnu.org> + + * config/sh/sh.c (shiftcosts): Return MAX_COST when the first + operand is CONST_INT. Take COSTS_N_INSNS into account. + (sh_rtx_costs): Don't apply COSTS_N_INSNS to the return value + of shiftcosts. + 2012-03-02 Richard Henderson <rth@redhat.com> * optabs.c (expand_atomic_test_and_set): Honor diff --git a/gcc/config/sh/sh.c b/gcc/config/sh/sh.c index 00258f42b8d45bd5ebe9655fdf099519a0a43058..1b7eaae8fbe03d4481ff00362179b1b32fc85196 100644 --- a/gcc/config/sh/sh.c +++ b/gcc/config/sh/sh.c @@ -2828,22 +2828,26 @@ shiftcosts (rtx x) { int value; + /* There is no pattern for constant first operand. */ + if (CONST_INT_P (XEXP (x, 0))) + return MAX_COST; + if (TARGET_SHMEDIA) - return 1; + return COSTS_N_INSNS (1); if (GET_MODE_SIZE (GET_MODE (x)) > UNITS_PER_WORD) { if (GET_MODE (x) == DImode && CONST_INT_P (XEXP (x, 1)) && INTVAL (XEXP (x, 1)) == 1) - return 2; + return COSTS_N_INSNS (2); /* Everything else is invalid, because there is no pattern for it. */ return MAX_COST; } /* If shift by a non constant, then this will be expensive. */ if (!CONST_INT_P (XEXP (x, 1))) - return SH_DYNAMIC_SHIFT_COST; + return COSTS_N_INSNS (SH_DYNAMIC_SHIFT_COST); /* Otherwise, return the true cost in instructions. Cope with out of range shift counts more or less arbitrarily. */ @@ -2855,10 +2859,10 @@ shiftcosts (rtx x) /* If SH3, then we put the constant in a reg and use shad. */ if (cost > 1 + SH_DYNAMIC_SHIFT_COST) cost = 1 + SH_DYNAMIC_SHIFT_COST; - return cost; + return COSTS_N_INSNS (cost); } else - return shift_insns[value]; + return COSTS_N_INSNS (shift_insns[value]); } /* Return the cost of an AND/XOR/IOR operation. */ @@ -3091,7 +3095,7 @@ sh_rtx_costs (rtx x, int code, int outer_code, int opno ATTRIBUTE_UNUSED, case ASHIFT: case ASHIFTRT: case LSHIFTRT: - *total = COSTS_N_INSNS (shiftcosts (x)); + *total = shiftcosts (x); return true; case DIV: