remove has_compound_ret_parameter, we already had cc_compound_ret
[libfirm] / ir / be / bespillutil.c
index f341314..ccef797 100644 (file)
@@ -106,12 +106,10 @@ struct spill_env_t {
        ir_nodeset_t      mem_phis;       /**< set of all spilled phis. */
        ir_exec_freq     *exec_freq;
 
-#ifdef FIRM_STATISTICS
        unsigned          spill_count;
        unsigned          reload_count;
        unsigned          remat_count;
        unsigned          spilled_phi_count;
-#endif
 };
 
 /**
@@ -162,12 +160,10 @@ spill_env_t *be_new_spill_env(ir_graph *irg)
        env->exec_freq      = be_get_irg_exec_freq(irg);
        obstack_init(&env->obst);
 
-#ifdef FIRM_STATISTICS
        env->spill_count       = 0;
        env->reload_count      = 0;
        env->remat_count       = 0;
        env->spilled_phi_count = 0;
-#endif
 
        return env;
 }
@@ -232,29 +228,6 @@ void be_add_spill(spill_env_t *env, ir_node *to_spill, ir_node *after)
        spill_info->spills = spill;
 }
 
-void be_add_remat(spill_env_t *env, ir_node *to_spill, ir_node *before,
-                  ir_node *rematted_node)
-{
-       spill_info_t *spill_info;
-       reloader_t *reloader;
-
-       spill_info = get_spillinfo(env, to_spill);
-
-       /* add the remat information */
-       reloader                   = OALLOC(&env->obst, reloader_t);
-       reloader->next             = spill_info->reloaders;
-       reloader->reloader         = before;
-       reloader->rematted_node    = rematted_node;
-       reloader->remat_cost_delta = 0; /* We will never have a cost win over a
-                                          reload since we're not even allowed to
-                                          create a reload */
-
-       spill_info->reloaders  = reloader;
-
-       DBG((dbg, LEVEL_1, "creating spillinfo for %+F, will be rematerialized before %+F\n",
-               to_spill, before));
-}
-
 void be_add_reload2(spill_env_t *env, ir_node *to_spill, ir_node *before,
                ir_node *can_spill_after, const arch_register_class_t *reload_cls,
                int allow_remat)
@@ -317,8 +290,13 @@ ir_node *be_get_end_of_block_insertion_point(const ir_node *block)
        return last;
 }
 
-static ir_node *skip_keeps_phis(ir_node *node)
+/**
+ * determine final spill position: it should be after all phis, keep nodes
+ * and behind nodes marked as prolog
+ */
+static ir_node *determine_spill_point(ir_node *node)
 {
+       node = skip_Proj(node);
        while (true) {
                ir_node *next = sched_next(node);
                if (!is_Phi(next) && !be_is_Keep(next) && !be_is_CopyKeep(next))
@@ -388,7 +366,7 @@ void be_spill_phi(spill_env_t *env, ir_node *node)
                        insert = be_get_end_of_block_insertion_point(pred_block);
                        insert = sched_prev(insert);
                } else {
-                       insert = skip_keeps_phis(arg);
+                       insert = determine_spill_point(arg);
                }
 
                be_add_spill(env, arg, insert);
@@ -429,7 +407,7 @@ static void spill_irn(spill_env_t *env, spill_info_t *spillinfo)
        if (!sched_is_scheduled(insn)) {
                /* override spillinfos or create a new one */
                ir_graph *irg = get_irn_irg(to_spill);
-               spillinfo->spills->spill = new_r_NoMem(irg);
+               spillinfo->spills->spill = get_irg_no_mem(irg);
                DB((dbg, LEVEL_1, "don't spill %+F use NoMem\n", to_spill));
                return;
        }
@@ -438,16 +416,11 @@ static void spill_irn(spill_env_t *env, spill_info_t *spillinfo)
        spill = spillinfo->spills;
        for ( ; spill != NULL; spill = spill->next) {
                ir_node *after = spill->after;
-               ir_node *block = get_block(after);
-
-               after = skip_keeps_phis(after);
+               after = determine_spill_point(after);
 
-               spill->spill = be_spill(block, to_spill);
-               sched_add_after(skip_Proj(after), spill->spill);
+               spill->spill = arch_env_new_spill(env->arch_env, to_spill, after);
                DB((dbg, LEVEL_1, "\t%+F after %+F\n", spill->spill, after));
-#ifdef FIRM_STATISTICS
                env->spill_count++;
-#endif
        }
        DBG((dbg, LEVEL_1, "\n"));
 }
@@ -489,15 +462,13 @@ static void spill_phi(spill_env_t *env, spill_info_t *spillinfo)
 
        /* override or replace spills list... */
        spill         = OALLOC(&env->obst, spill_t);
-       spill->after  = skip_keeps_phis(phi);
+       spill->after  = determine_spill_point(phi);
        spill->spill  = be_new_Phi(block, arity, ins, mode_M, NULL);
        spill->next   = NULL;
        sched_add_after(block, spill->spill);
 
        spillinfo->spills = spill;
-#ifdef FIRM_STATISTICS
        env->spilled_phi_count++;
-#endif
 
        for (i = 0; i < arity; ++i) {
                ir_node      *arg      = get_irn_n(phi, i);
@@ -606,7 +577,7 @@ static int check_remat_conditions_costs(spill_env_t *env,
                return REMAT_COST_INFINITE;
        }
        /* never rematerialize a node which modifies the flags.
-        * (would be better to test wether the flags are actually live at point
+        * (would be better to test whether the flags are actually live at point
         * reloader...)
         */
        if (arch_irn_is(insn, modify_flags)) {
@@ -640,7 +611,7 @@ static int check_remat_conditions_costs(spill_env_t *env,
 /**
  * Re-materialize a node.
  *
- * @param senv      the spill environment
+ * @param env       the spill environment
  * @param spilled   the node that was spilled
  * @param reloader  a irn that requires a reload
  */
@@ -665,10 +636,8 @@ static ir_node *do_remat(spill_env_t *env, ir_node *spilled, ir_node *reloader)
                        ins[i] = arg;
                } else {
                        ins[i] = do_remat(env, arg, reloader);
-#ifdef FIRM_STATISTICS
-                       /* don't count the recursive call as remat */
-                       env->remat_count--;
-#endif
+                       /* don't count the argument rematerialization as an extra remat */
+                       --env->remat_count;
                }
        }
 
@@ -685,9 +654,7 @@ static ir_node *do_remat(spill_env_t *env, ir_node *spilled, ir_node *reloader)
                /* insert in schedule */
                sched_reset(res);
                sched_add_before(reloader, res);
-#ifdef FIRM_STATISTICS
-               env->remat_count++;
-#endif
+               ++env->remat_count;
        }
 
        return res;
@@ -743,6 +710,39 @@ double be_get_reload_costs_on_edge(spill_env_t *env, ir_node *to_spill,
        return be_get_reload_costs(env, to_spill, before);
 }
 
+ir_node *be_new_spill(ir_node *value, ir_node *after)
+{
+       ir_graph                    *irg       = get_irn_irg(value);
+       ir_node                     *frame     = get_irg_frame(irg);
+       const arch_register_class_t *cls       = arch_get_irn_reg_class(value);
+       const arch_register_class_t *cls_frame = arch_get_irn_reg_class(frame);
+       ir_node                     *block     = get_block(after);
+       ir_node                     *spill
+               = be_new_Spill(cls, cls_frame, block, frame, value);
+
+       sched_add_after(after, spill);
+       return spill;
+}
+
+ir_node *be_new_reload(ir_node *value, ir_node *spill, ir_node *before)
+{
+       ir_graph *irg   = get_irn_irg(value);
+       ir_node  *frame = get_irg_frame(irg);
+       ir_node  *block = get_block(before);
+       const arch_register_class_t *cls       = arch_get_irn_reg_class(value);
+       const arch_register_class_t *cls_frame = arch_get_irn_reg_class(frame);
+       ir_mode                     *mode      = get_irn_mode(value);
+       ir_node  *reload;
+
+       assert(be_is_Spill(spill) || is_Phi(spill));
+       assert(get_irn_mode(spill) == mode_M);
+
+       reload = be_new_Reload(cls, cls_frame, block, frame, spill, mode);
+       sched_add_before(before, reload);
+
+       return reload;
+}
+
 /*
  *  ___                     _     ____      _                 _
  * |_ _|_ __  ___  ___ _ __| |_  |  _ \ ___| | ___   __ _  __| |___
@@ -781,7 +781,7 @@ static void determine_spill_costs(spill_env_t *env, spill_info_t *spillinfo)
                spill_t *spill = OALLOC(&env->obst, spill_t);
                spill->after = NULL;
                spill->next  = NULL;
-               spill->spill = new_r_NoMem(irg);
+               spill->spill = get_irg_no_mem(irg);
 
                spillinfo->spills      = spill;
                spillinfo->spill_costs = 0;
@@ -828,7 +828,7 @@ static void determine_spill_costs(spill_env_t *env, spill_info_t *spillinfo)
 
        /* override spillinfos or create a new one */
        spill        = OALLOC(&env->obst, spill_t);
-       spill->after = skip_keeps_phis(to_spill);
+       spill->after = determine_spill_point(to_spill);
        spill->next  = NULL;
        spill->spill = NULL;
 
@@ -889,12 +889,11 @@ void be_insert_spills_reloads(spill_env_t *env)
 
        /* process each spilled node */
        foreach_set(env->spills, spill_info_t*, si) {
-               reloader_t *rld;
                ir_node  *to_spill        = si->to_spill;
-               ir_mode  *mode            = get_irn_mode(to_spill);
                ir_node **copies          = NEW_ARR_F(ir_node*, 0);
                double    all_remat_costs = 0; /** costs when we would remat all nodes */
-               int       force_remat     = 0;
+               bool      force_remat     = false;
+               reloader_t *rld;
 
                DBG((dbg, LEVEL_1, "\nhandling all reloaders of %+F:\n", to_spill));
 
@@ -954,7 +953,7 @@ void be_insert_spills_reloads(spill_env_t *env)
                        if (all_remat_costs < 0) {
                                DBG((dbg, LEVEL_1, "\nforcing remats of all reloaders (%f)\n",
                                     all_remat_costs));
-                               force_remat = 1;
+                               force_remat = true;
                        }
                }
 
@@ -975,11 +974,9 @@ void be_insert_spills_reloads(spill_env_t *env)
                                /* create a reload, use the first spill for now SSA
                                 * reconstruction for memory comes below */
                                assert(si->spills != NULL);
-                               copy = be_reload(si->reload_cls, rld->reloader, mode,
-                                                si->spills->spill);
-#ifdef FIRM_STATISTICS
+                               copy = arch_env_new_reload(env->arch_env, si->to_spill,
+                                                          si->spills->spill, rld->reloader);
                                env->reload_count++;
-#endif
                        }
 
                        DBG((dbg, LEVEL_1, " %+F of %+F before %+F\n",
@@ -1052,7 +1049,7 @@ void be_insert_spills_reloads(spill_env_t *env)
        be_timer_pop(T_RA_SPILL_APPLY);
 }
 
-BE_REGISTER_MODULE_CONSTRUCTOR(be_init_spill);
+BE_REGISTER_MODULE_CONSTRUCTOR(be_init_spill)
 void be_init_spill(void)
 {
        FIRM_DBG_REGISTER(dbg, "firm.be.spill");