Patchwork [ARM] committed - Remove superfluous definition of COSTS_N_INSNS in the ARM backend.

login
register
mail settings
Submitter Ramana Radhakrishnan
Date Aug. 2, 2010, 8:52 a.m.
Message ID <1280739154.9309.7.camel@e102325-lin.cambridge.arm.com>
Download mbox | patch
Permalink /patch/60522/
State New
Headers show

Comments

Ramana Radhakrishnan - Aug. 2, 2010, 8:52 a.m.
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  <ramana.radhakrishnan@arm.com>

	* config/arm/arm.c (COSTS_N_INSNS): Remove definition.

Patch

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)
 {