This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
FRV cleanup (remove frv_option_optimization)
- From: "Joseph S. Myers" <joseph at codesourcery dot com>
- To: gcc-patches at gcc dot gnu dot org, nickc at redhat dot com, aoliva at redhat dot com
- Date: Fri, 15 Oct 2010 00:23:46 +0000 (UTC)
- Subject: FRV cleanup (remove frv_option_optimization)
The FRV port of GCC contains some code in frv_option_optimization that
appears to have no actual use in the FSF GCC sources.
The function sets flag_rcsp if ENABLE_RCSP is defined. There is
nothing that defines that macro, or that variable. A macro
RCSP_SOFTWARE_PIPELINING is defined in frv.h, but is also unused. So
these are removed by this patch.
The function also clears flag_schedule_insns_after_reload if
DISABLE_SCHED2 is defined. But nothing ever defines DISABLE_SCHED2.
So this patch also removes that code.
With both of these dead, the function actually does nothing at all.
So this patch completely removes the hook from this port.
Tested by building cc1 for a cross to frv-elf. OK to commit?
(Given this code it seems quite plausible this port could have other
dead macro definitions or tests, but I haven't checked for them.)
2010-10-14 Joseph Myers <joseph@codesourcery.com>
* config/frv/frv.c (frv_option_optimization,
TARGET_OPTION_OPTIMIZATION): Remove.
* config/frv/frv.h (RCSP_SOFTWARE_PIPELINING): Remove.
Index: gcc/config/frv/frv.h
===================================================================
--- gcc/config/frv/frv.h (revision 165460)
+++ gcc/config/frv/frv.h (working copy)
@@ -2460,11 +2460,6 @@ frv_ifcvt_modify_multiple_tests (CE_INFO
/* If the following macro is defined and nonzero and deterministic
- finite state automata are used for pipeline hazard recognition, the
- code making resource-constrained software pipelining is on. */
-#define RCSP_SOFTWARE_PIPELINING 1
-
-/* If the following macro is defined and nonzero and deterministic
finite state automata are used for pipeline hazard recognition, we
will try to exchange insns in queue ready to improve the schedule.
The more macro value, the more tries will be made. */
Index: gcc/config/frv/frv.c
===================================================================
--- gcc/config/frv/frv.c (revision 165460)
+++ gcc/config/frv/frv.c (working copy)
@@ -265,7 +265,6 @@ frv_cpu_t frv_cpu_type = CPU_TYPE; /* va
static bool frv_handle_option (size_t, const char *, int);
static void frv_option_override (void);
-static void frv_option_optimization (int, int);
static bool frv_legitimate_address_p (enum machine_mode, rtx, bool);
static int frv_default_flags_for_cpu (void);
static int frv_string_begins_with (const_tree, const char *);
@@ -432,8 +431,6 @@ static bool frv_class_likely_spilled_p
#define TARGET_HANDLE_OPTION frv_handle_option
#undef TARGET_OPTION_OVERRIDE
#define TARGET_OPTION_OVERRIDE frv_option_override
-#undef TARGET_OPTION_OPTIMIZATION
-#define TARGET_OPTION_OPTIMIZATION frv_option_optimization
#undef TARGET_INIT_BUILTINS
#define TARGET_INIT_BUILTINS frv_init_builtins
#undef TARGET_EXPAND_BUILTIN
@@ -847,25 +844,6 @@ frv_option_override (void)
}
-/* Implement TARGET_OPTION_OPTIMIZATION.
-
- On the FRV, possibly disable VLIW packing which is done by the 2nd
- scheduling pass at the current time. */
-static void
-frv_option_optimization (int level, int size ATTRIBUTE_UNUSED)
-{
- if (level >= 2)
- {
-#ifdef DISABLE_SCHED2
- flag_schedule_insns_after_reload = 0;
-#endif
-#ifdef ENABLE_RCSP
- flag_rcsp = 1;
-#endif
- }
-}
-
-
/* Return true if NAME (a STRING_CST node) begins with PREFIX. */
static int
--
Joseph S. Myers
joseph@codesourcery.com