This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[MIPS, committed] Fix some mips.c warnings
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Sun, 22 Jun 2014 10:57:31 +0100
- Subject: [MIPS, committed] Fix some mips.c warnings
- Authentication-results: sourceware.org; auth=none
Fixes unused "mode" and "in_p" argument names. Neither mips_move_to_gpr_cost
nor mips_move_from_gpr_cost care about the mode now, so I just removed it.
Thanks,
Richard
gcc/
* config/mips/mips.c (mips_move_to_gpr_cost): Remove mode argument.
(mips_move_from_gpr_cost): Likewise.
(mips_register_move_cost): Update accordingly.
(mips_secondary_reload_class): Remove name of in_p.
Index: gcc/config/mips/mips.c
===================================================================
--- gcc/config/mips/mips.c 2014-06-14 09:30:21.288981027 +0100
+++ gcc/config/mips/mips.c 2014-06-22 10:48:26.156234088 +0100
@@ -12021,13 +12021,12 @@ mips_canonicalize_move_class (reg_class_
return rclass;
}
-/* Return the cost of moving a value of mode MODE from a register of
- class FROM to a GPR. Return 0 for classes that are unions of other
- classes handled by this function. */
+/* Return the cost of moving a value from a register of class FROM to a GPR.
+ Return 0 for classes that are unions of other classes handled by this
+ function. */
static int
-mips_move_to_gpr_cost (enum machine_mode mode ATTRIBUTE_UNUSED,
- reg_class_t from)
+mips_move_to_gpr_cost (reg_class_t from)
{
switch (from)
{
@@ -12059,12 +12058,12 @@ mips_move_to_gpr_cost (enum machine_mode
}
}
-/* Return the cost of moving a value of mode MODE from a GPR to a
- register of class TO. Return 0 for classes that are unions of
- other classes handled by this function. */
+/* Return the cost of moving a value from a GPR to a register of class TO.
+ Return 0 for classes that are unions of other classes handled by this
+ function. */
static int
-mips_move_from_gpr_cost (enum machine_mode mode, reg_class_t to)
+mips_move_from_gpr_cost (reg_class_t to)
{
switch (to)
{
@@ -12125,15 +12124,15 @@ mips_register_move_cost (enum machine_mo
/* Handle cases in which only one class deviates from the ideal. */
dregs = TARGET_MIPS16 ? M16_REGS : GENERAL_REGS;
if (from == dregs)
- return mips_move_from_gpr_cost (mode, to);
+ return mips_move_from_gpr_cost (to);
if (to == dregs)
- return mips_move_to_gpr_cost (mode, from);
+ return mips_move_to_gpr_cost (from);
/* Handles cases that require a GPR temporary. */
- cost1 = mips_move_to_gpr_cost (mode, from);
+ cost1 = mips_move_to_gpr_cost (from);
if (cost1 != 0)
{
- cost2 = mips_move_from_gpr_cost (mode, to);
+ cost2 = mips_move_from_gpr_cost (to);
if (cost2 != 0)
return cost1 + cost2;
}
@@ -12158,7 +12157,7 @@ mips_memory_move_cost (enum machine_mode
enum reg_class
mips_secondary_reload_class (enum reg_class rclass,
- enum machine_mode mode, rtx x, bool in_p)
+ enum machine_mode mode, rtx x, bool)
{
int regno;