* [Patch, AVR] Housekeeping: Hookize REGISTER_MOVE_COST, MEMORY_MOVE_COST
@ 2011-03-08 11:27 Georg-Johann Lay
2011-03-08 18:32 ` Denis Chertykov
0 siblings, 1 reply; 2+ messages in thread
From: Georg-Johann Lay @ 2011-03-08 11:27 UTC (permalink / raw)
To: gcc-patches; +Cc: Denis Chertykov, Anatoly Sokolov, Eric Weddington
[-- Attachment #1: Type: text/plain, Size: 477 bytes --]
This patch moves deprecated REGISTER_MOVE_COST resp. MEMORY_MOVE_COST
from avr.h to target hook avr_register_move_cost resp.
avr_memory_move_cost in avr.c.
No functionality added or removed; costs are unchanged.
2011-03-08 Georg-Johann Lay <avr@gjlay.de>
* config/avr/avr.h (REGISTER_MOVE_COST, MEMORY_MOVE_COST): Remove.
* config/avr/avr.c (TARGET_REGISTER_MOVE_COST,
TARGET_MEMORY_MOVE_COST): Define.
(avr_register_move_cost, avr_memory_move_cost): New Functions.
[-- Attachment #2: hookize-cost.diff --]
[-- Type: text/x-patch, Size: 2652 bytes --]
Index: config/avr/avr.c
===================================================================
--- config/avr/avr.c (revision 170704)
+++ config/avr/avr.c (working copy)
@@ -82,6 +82,8 @@ static unsigned int avr_section_type_fla
static void avr_reorg (void);
static void avr_asm_out_ctor (rtx, int);
static void avr_asm_out_dtor (rtx, int);
+static int avr_register_move_cost (enum machine_mode, reg_class_t, reg_class_t);
+static int avr_memory_move_cost (enum machine_mode, reg_class_t, bool);
static int avr_operand_rtx_cost (rtx, enum machine_mode, enum rtx_code, bool);
static bool avr_rtx_costs (rtx, int, int, int *, bool);
static int avr_address_cost (rtx, bool);
@@ -174,6 +176,10 @@ static const struct default_options avr_
#define TARGET_INSERT_ATTRIBUTES avr_insert_attributes
#undef TARGET_SECTION_TYPE_FLAGS
#define TARGET_SECTION_TYPE_FLAGS avr_section_type_flags
+#undef TARGET_REGISTER_MOVE_COST
+#define TARGET_REGISTER_MOVE_COST avr_register_move_cost
+#undef TARGET_MEMORY_MOVE_COST
+#define TARGET_MEMORY_MOVE_COST avr_memory_move_cost
#undef TARGET_RTX_COSTS
#define TARGET_RTX_COSTS avr_rtx_costs
#undef TARGET_ADDRESS_COST
@@ -5132,6 +5138,32 @@ order_regs_for_local_alloc (void)
}
+/* Implement `TARGET_REGISTER_MOVE_COST' */
+
+static int
+avr_register_move_cost (enum machine_mode mode ATTRIBUTE_UNUSED,
+ reg_class_t from, reg_class_t to)
+{
+ return (from == STACK_REG ? 6
+ : to == STACK_REG ? 12
+ : 2);
+}
+
+
+/* Implement `TARGET_MEMORY_MOVE_COST' */
+
+static int
+avr_memory_move_cost (enum machine_mode mode, reg_class_t rclass ATTRIBUTE_UNUSED,
+ bool in ATTRIBUTE_UNUSED)
+{
+ return (mode == QImode ? 2
+ : mode == HImode ? 4
+ : mode == SImode ? 8
+ : mode == SFmode ? 8
+ : 16);
+}
+
+
/* Mutually recursive subroutine of avr_rtx_cost for calculating the
cost of an RTX operand given its context. X is the rtx of the
operand, MODE is its mode, and OUTER is the rtx_code of this
Index: config/avr/avr.h
===================================================================
--- config/avr/avr.h (revision 170704)
+++ config/avr/avr.h (working copy)
@@ -447,15 +447,6 @@ do { \
#define LEGITIMATE_CONSTANT_P(X) 1
-#define REGISTER_MOVE_COST(MODE, FROM, TO) ((FROM) == STACK_REG ? 6 \
- : (TO) == STACK_REG ? 12 \
- : 2)
-
-#define MEMORY_MOVE_COST(MODE,CLASS,IN) ((MODE)==QImode ? 2 : \
- (MODE)==HImode ? 4 : \
- (MODE)==SImode ? 8 : \
- (MODE)==SFmode ? 8 : 16)
-
#define BRANCH_COST(speed_p, predictable_p) 0
#define SLOW_BYTE_ACCESS 0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [Patch, AVR] Housekeeping: Hookize REGISTER_MOVE_COST, MEMORY_MOVE_COST
2011-03-08 11:27 [Patch, AVR] Housekeeping: Hookize REGISTER_MOVE_COST, MEMORY_MOVE_COST Georg-Johann Lay
@ 2011-03-08 18:32 ` Denis Chertykov
0 siblings, 0 replies; 2+ messages in thread
From: Denis Chertykov @ 2011-03-08 18:32 UTC (permalink / raw)
To: Georg-Johann Lay; +Cc: gcc-patches, Anatoly Sokolov, Eric Weddington
2011/3/8 Georg-Johann Lay <avr@gjlay.de>:
> This patch moves deprecated REGISTER_MOVE_COST resp. MEMORY_MOVE_COST
> from avr.h to target hook avr_register_move_cost resp.
> avr_memory_move_cost in avr.c.
>
> No functionality added or removed; costs are unchanged.
>
>
> 2011-03-08 Georg-Johann Lay <avr@gjlay.de>
>
> * config/avr/avr.h (REGISTER_MOVE_COST, MEMORY_MOVE_COST): Remove.
> * config/avr/avr.c (TARGET_REGISTER_MOVE_COST,
> TARGET_MEMORY_MOVE_COST): Define.
> (avr_register_move_cost, avr_memory_move_cost): New Functions.
Applied.
Denis.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-03-08 18:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-08 11:27 [Patch, AVR] Housekeeping: Hookize REGISTER_MOVE_COST, MEMORY_MOVE_COST Georg-Johann Lay
2011-03-08 18:32 ` Denis Chertykov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).