From patchwork Mon Aug 2 08:52:34 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [ARM] committed - Remove superfluous definition of COSTS_N_INSNS in the ARM backend. Date: Sun, 01 Aug 2010 22:52:34 -0000 From: Ramana Radhakrishnan X-Patchwork-Id: 60522 Message-Id: <1280739154.9309.7.camel@e102325-lin.cambridge.arm.com> To: gcc-patches@gcc.gnu.org Hi, This removes a superfluous definition of COSTS_N_INSNS in the ARM backend. This bit of code dates back to 2000 when the ARM and Thumb backends were merged but wasn't removed when the definition of COSTS_N_INSNS was moved to rtl.h in Sept. 2000. I can't see how this definition could have been exercised in the last few years and hence committed as obvious. cheers Ramana 2010-08-02 Ramana Radhakrishnan * config/arm/arm.c (COSTS_N_INSNS): Remove definition. Index: config/arm/arm.c =================================================================== --- config/arm/arm.c (revision 162810) +++ config/arm/arm.c (working copy) @@ -6293,9 +6293,6 @@ arm_cannot_force_const_mem (rtx x) #define REG_OR_SUBREG_RTX(X) \ (GET_CODE (X) == REG ? (X) : SUBREG_REG (X)) -#ifndef COSTS_N_INSNS -#define COSTS_N_INSNS(N) ((N) * 4 - 2) -#endif static inline int thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer) {