bespill: Remove redundant tests for ignore-only register classes.

Such register classes are flagged manual_ra already.

diff --git a/ir/be/bechordal_main.c b/ir/be/bechordal_main.c
index 90f0354..59912d0 100644 (file)
--- a/ir/be/bechordal_main.c
+++ b/ir/be/bechordal_main.c
@@ -234,85 +234,77 @@ static void pre_spill(be_chordal_env_t *const chordal_env, arch_register_class_t
  */
 static void post_spill(be_chordal_env_t *const chordal_env, ir_graph *const irg)
 {
-       /* some special classes contain only ignore regs, no work to be done */
-       int const allocatable_regs = be_get_n_allocatable_regs(irg, chordal_env->cls);
-       if (allocatable_regs > 0) {
-               /*
-                       If we have a backend provided spiller, post spill is
-                       called in a loop after spilling for each register class.
-                       But we only need to fix stack nodes once in this case.
-               */
-               be_timer_push(T_RA_SPILL_APPLY);
-               check_for_memory_operands(irg);
-               be_abi_fix_stack_nodes(irg);
-               be_timer_pop(T_RA_SPILL_APPLY);
-
-
-               /* verify schedule and register pressure */
-               be_timer_push(T_VERIFY);
-               if (chordal_env->opts->vrfy_option == BE_CH_VRFY_WARN) {
-                       be_verify_schedule(irg);
-                       be_verify_register_pressure(irg, chordal_env->cls);
-               } else if (chordal_env->opts->vrfy_option == BE_CH_VRFY_ASSERT) {
-                       assert(be_verify_schedule(irg) && "Schedule verification failed");
-                       assert(be_verify_register_pressure(irg, chordal_env->cls)
-                               && "Register pressure verification failed");
-               }
-               be_timer_pop(T_VERIFY);
-
-               /* Color the graph. */
-               be_timer_push(T_RA_COLOR);
-               be_ra_chordal_coloring(chordal_env);
-               be_timer_pop(T_RA_COLOR);
+       /*
+               If we have a backend provided spiller, post spill is
+               called in a loop after spilling for each register class.
+               But we only need to fix stack nodes once in this case.
+       */
+       be_timer_push(T_RA_SPILL_APPLY);
+       check_for_memory_operands(irg);
+       be_abi_fix_stack_nodes(irg);
+       be_timer_pop(T_RA_SPILL_APPLY);
+
+       /* verify schedule and register pressure */
+       be_timer_push(T_VERIFY);
+       if (chordal_env->opts->vrfy_option == BE_CH_VRFY_WARN) {
+               be_verify_schedule(irg);
+               be_verify_register_pressure(irg, chordal_env->cls);
+       } else if (chordal_env->opts->vrfy_option == BE_CH_VRFY_ASSERT) {
+               assert(be_verify_schedule(irg) && "Schedule verification failed");
+               assert(be_verify_register_pressure(irg, chordal_env->cls) && "Register pressure verification failed");
+       }
+       be_timer_pop(T_VERIFY);
 
-               dump(BE_CH_DUMP_CONSTR, irg, chordal_env->cls, "color");
+       /* Color the graph. */
+       be_timer_push(T_RA_COLOR);
+       be_ra_chordal_coloring(chordal_env);
+       be_timer_pop(T_RA_COLOR);
 
-               /* Create the ifg with the selected flavor */
-               be_timer_push(T_RA_IFG);
-               chordal_env->ifg = be_create_ifg(chordal_env);
-               be_timer_pop(T_RA_IFG);
+       dump(BE_CH_DUMP_CONSTR, irg, chordal_env->cls, "color");
 
-               if (stat_ev_enabled) {
-                       be_ifg_stat_t   stat;
-                       be_node_stats_t node_stats;
+       /* Create the ifg with the selected flavor */
+       be_timer_push(T_RA_IFG);
+       chordal_env->ifg = be_create_ifg(chordal_env);
+       be_timer_pop(T_RA_IFG);
 
-                       be_ifg_stat(irg, chordal_env->ifg, &stat);
-                       stat_ev_dbl("bechordal_ifg_nodes", stat.n_nodes);
-                       stat_ev_dbl("bechordal_ifg_edges", stat.n_edges);
-                       stat_ev_dbl("bechordal_ifg_comps", stat.n_comps);
+       if (stat_ev_enabled) {
+               be_ifg_stat_t   stat;
+               be_node_stats_t node_stats;
 
-                       be_collect_node_stats(&node_stats, irg);
-                       be_subtract_node_stats(&node_stats, &last_node_stats);
+               be_ifg_stat(irg, chordal_env->ifg, &stat);
+               stat_ev_dbl("bechordal_ifg_nodes", stat.n_nodes);
+               stat_ev_dbl("bechordal_ifg_edges", stat.n_edges);
+               stat_ev_dbl("bechordal_ifg_comps", stat.n_comps);
 
-                       stat_ev_dbl("bechordal_perms_before_coal",
-                                       node_stats[BE_STAT_PERMS]);
-                       stat_ev_dbl("bechordal_copies_before_coal",
-                                       node_stats[BE_STAT_COPIES]);
-               }
+               be_collect_node_stats(&node_stats, irg);
+               be_subtract_node_stats(&node_stats, &last_node_stats);
 
-               be_timer_push(T_RA_COPYMIN);
-               co_driver(chordal_env);
-               be_timer_pop(T_RA_COPYMIN);
+               stat_ev_dbl("bechordal_perms_before_coal",  node_stats[BE_STAT_PERMS]);
+               stat_ev_dbl("bechordal_copies_before_coal", node_stats[BE_STAT_COPIES]);
+       }
 
-               dump(BE_CH_DUMP_COPYMIN, irg, chordal_env->cls, "copymin");
+       be_timer_push(T_RA_COPYMIN);
+       co_driver(chordal_env);
+       be_timer_pop(T_RA_COPYMIN);
 
-               /* ssa destruction */
-               be_timer_push(T_RA_SSA);
-               be_ssa_destruction(chordal_env);
-               be_timer_pop(T_RA_SSA);
+       dump(BE_CH_DUMP_COPYMIN, irg, chordal_env->cls, "copymin");
 
-               dump(BE_CH_DUMP_SSADESTR, irg, chordal_env->cls, "ssadestr");
+       /* ssa destruction */
+       be_timer_push(T_RA_SSA);
+       be_ssa_destruction(chordal_env);
+       be_timer_pop(T_RA_SSA);
 
-               if (chordal_env->opts->vrfy_option != BE_CH_VRFY_OFF) {
-                       be_timer_push(T_VERIFY);
-                       be_ssa_destruction_check(chordal_env);
-                       be_timer_pop(T_VERIFY);
-               }
+       dump(BE_CH_DUMP_SSADESTR, irg, chordal_env->cls, "ssadestr");
 
-               /* the ifg exists only if there are allocatable regs */
-               be_ifg_free(chordal_env->ifg);
+       if (chordal_env->opts->vrfy_option != BE_CH_VRFY_OFF) {
+               be_timer_push(T_VERIFY);
+               be_ssa_destruction_check(chordal_env);
+               be_timer_pop(T_VERIFY);
        }
 
+       /* the ifg exists only if there are allocatable regs */
+       be_ifg_free(chordal_env->ifg);
+
        /* free some always allocated data structures */
        pmap_destroy(chordal_env->border_heads);
        bitset_free(chordal_env->allocatable_regs);

diff --git a/ir/be/bespilldaemel.c b/ir/be/bespilldaemel.c
index 69be6a1..08a395f 100644 (file)
--- a/ir/be/bespilldaemel.c
+++ b/ir/be/bespilldaemel.c
@@ -321,8 +321,6 @@ static void spill_block(ir_node *block, void *data)
 static void be_spill_daemel(ir_graph *irg, const arch_register_class_t *new_cls)
 {
        n_regs = be_get_n_allocatable_regs(irg, new_cls);
-       if (n_regs == 0)
-               return;
 
        be_assure_live_sets(irg);