X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fbe%2Fbeverify.c;h=282dd311da97ed06b4d17f42bbf437bf994e932e;hb=8a5ac70896b18a522fa39aad29b3c7cf36d5941e;hp=ec2af73efc2786efa72a1e0e931fa2c39fc57c84;hpb=20eeb0eec8a216005fce59f005fc301779e42cdf;p=libfirm diff --git a/ir/be/beverify.c b/ir/be/beverify.c index ec2af73ef..282dd311d 100644 --- a/ir/be/beverify.c +++ b/ir/be/beverify.c @@ -1,5 +1,5 @@ /* - * Copyright (C) 1995-2008 University of Karlsruhe. All right reserved. + * Copyright (C) 1995-2010 University of Karlsruhe. All right reserved. * * This file is part of libFirm. * @@ -22,13 +22,11 @@ * @brief Various verify routines that check a scheduled graph for correctness. * @author Matthias Braun * @date 05.05.2006 - * @version $Id$ */ -#ifdef HAVE_CONFIG_H #include "config.h" -#endif #include +#include #include "bitset.h" #include "set.h" @@ -43,17 +41,17 @@ #include "beverify.h" #include "belive.h" -#include "besched_t.h" -#include "benode_t.h" -#include "beirg_t.h" +#include "besched.h" +#include "benode.h" +#include "beirg.h" #include "beintlive_t.h" +#include "belistsched.h" static int my_values_interfere(const ir_node *a, const ir_node *b); typedef struct be_verify_register_pressure_env_t_ { ir_graph *irg; /**< the irg to verify */ be_lv_t *lv; /**< Liveness information. */ - const arch_env_t *arch_env; /**< an architecture environment */ const arch_register_class_t *cls; /**< the register class to check for */ int registers_available; /**< number of available registers */ int problem_found; /**< flag indicating if a problem was found */ @@ -62,7 +60,8 @@ typedef struct be_verify_register_pressure_env_t_ { /** * Print all nodes of a pset into a file. */ -static void print_living_values(FILE *F, const ir_nodeset_t *live_nodes) { +static void print_living_values(FILE *F, const ir_nodeset_t *live_nodes) +{ ir_nodeset_iterator_t iter; ir_node *node; @@ -73,10 +72,17 @@ static void print_living_values(FILE *F, const ir_nodeset_t *live_nodes) { ir_fprintf(F, "\n"); } +static const char *get_irg_name(ir_graph *irg) +{ + ir_entity *entity = get_irg_entity(irg); + return get_entity_ld_name(entity); +} + /** * Check if number of live nodes never exceeds the number of available registers. */ -static void verify_liveness_walker(ir_node *block, void *data) { +static void verify_liveness_walker(ir_node *block, void *data) +{ be_verify_register_pressure_env_t *env = (be_verify_register_pressure_env_t *)data; ir_nodeset_t live_nodes; ir_node *irn; @@ -85,14 +91,14 @@ static void verify_liveness_walker(ir_node *block, void *data) { /* collect register pressure info, start with end of a block */ // ir_fprintf(stderr, "liveness check %+F\n", block); ir_nodeset_init(&live_nodes); - be_liveness_end_of_block(env->lv, env->arch_env, env->cls, block, + be_liveness_end_of_block(env->lv, env->cls, block, &live_nodes); // print_living_values(stderr, &live_nodes); pressure = ir_nodeset_size(&live_nodes); - if(pressure > env->registers_available) { + if (pressure > env->registers_available) { ir_fprintf(stderr, "Verify Warning: Register pressure too high at end of block %+F(%s) (%d/%d):\n", - block, get_irg_dump_name(env->irg), pressure, env->registers_available); + block, get_irg_name(env->irg), pressure, env->registers_available); print_living_values(stderr, &live_nodes); env->problem_found = 1; } @@ -102,13 +108,13 @@ static void verify_liveness_walker(ir_node *block, void *data) { break; // print_living_values(stderr, &live_nodes); - be_liveness_transfer(env->arch_env, env->cls, irn, &live_nodes); + be_liveness_transfer(env->cls, irn, &live_nodes); pressure = ir_nodeset_size(&live_nodes); - if(pressure > env->registers_available) { + if (pressure > env->registers_available) { ir_fprintf(stderr, "Verify Warning: Register pressure too high before node %+F in %+F(%s) (%d/%d):\n", - irn, block, get_irg_dump_name(env->irg), pressure, env->registers_available); + irn, block, get_irg_name(env->irg), pressure, env->registers_available); print_living_values(stderr, &live_nodes); env->problem_found = 1; assert(0); @@ -120,78 +126,69 @@ static void verify_liveness_walker(ir_node *block, void *data) { /** * Start a walk over the irg and check the register pressure. */ -int be_verify_register_pressure(const be_irg_t *birg, - const arch_register_class_t *cls, - ir_graph *irg) { +int be_verify_register_pressure(ir_graph *irg, const arch_register_class_t *cls) +{ be_verify_register_pressure_env_t env; - env.lv = be_liveness(birg); + env.lv = be_liveness_new(irg); env.irg = irg; - env.arch_env = birg->main_env->arch_env; env.cls = cls; - env.registers_available = env.cls->n_regs - be_put_ignore_regs(birg, env.cls, NULL); + env.registers_available = be_get_n_allocatable_regs(irg, cls); env.problem_found = 0; - be_liveness_assure_sets(env.lv); + be_liveness_compute_sets(env.lv); irg_block_walk_graph(irg, verify_liveness_walker, NULL, &env); be_liveness_free(env.lv); return ! env.problem_found; } - - /*--------------------------------------------------------------------------- */ - - typedef struct be_verify_schedule_env_t_ { - int problem_found; /**< flags indicating if there was a problem */ - bitset_t *scheduled; /**< bitset of scheduled nodes */ - ir_graph *irg; /**< the irg to check */ - const arch_env_t *arch_env; /**< the arch_env */ + int problem_found; /**< flags indicating a problem */ + bitset_t *scheduled; /**< bitset of scheduled nodes */ + ir_graph *irg; /**< the irg to check */ } be_verify_schedule_env_t; /** * Simple schedule checker. */ -static void verify_schedule_walker(ir_node *block, void *data) { +static void verify_schedule_walker(ir_node *block, void *data) +{ be_verify_schedule_env_t *env = (be_verify_schedule_env_t*) data; ir_node *node; - int non_phi_found = 0; - int cfchange_found = 0; - /* TODO ask arch about delay branches */ - int delay_branches = 0; + ir_node *non_phi_found = NULL; + ir_node *cfchange_found = NULL; int last_timestep = INT_MIN; /* * Tests for the following things: - * 1. Make sure that all phi nodes are scheduled at the beginning of the block - * 2. There is 1 or no control flow changing node scheduled and exactly delay_branches operations after it. - * 3. No value is defined after it has been used - * 4. mode_T nodes have all projs scheduled behind them followed by Keeps + * 1. Make sure that all phi nodes are scheduled at the beginning of the + * block + * 2. No value is defined after it has been used + * 3. mode_T nodes have all projs scheduled behind them followed by Keeps * (except mode_X projs) */ sched_foreach(block, node) { - int i, arity; int timestep; /* this node is scheduled */ - if(bitset_is_set(env->scheduled, get_irn_idx(node))) { + if (bitset_is_set(env->scheduled, get_irn_idx(node))) { ir_fprintf(stderr, "Verify warning: %+F appears to be schedule twice\n"); env->problem_found = 1; } bitset_set(env->scheduled, get_irn_idx(node)); /* Check that scheduled nodes are in the correct block */ - if(get_nodes_block(node) != block) { + if (get_nodes_block(node) != block) { ir_fprintf(stderr, "Verify warning: %+F is in block %+F but scheduled in %+F\n", node, get_nodes_block(node), block); env->problem_found = 1; } /* Check that timesteps are increasing */ timestep = sched_get_time_step(node); - if(timestep <= last_timestep) { + if (timestep <= last_timestep) { ir_fprintf(stderr, "Verify warning: Schedule timestep did not increase at node %+F\n", node); env->problem_found = 1; @@ -200,146 +197,101 @@ static void verify_schedule_walker(ir_node *block, void *data) { /* Check that phis come before any other node */ if (is_Phi(node)) { - if (non_phi_found) { - ir_fprintf(stderr, "Verify Warning: Phi node %+F scheduled after non-Phi nodes in block %+F (%s)\n", - node, block, get_irg_dump_name(env->irg)); + if (non_phi_found != NULL) { + ir_fprintf(stderr, "Verify Warning: Phi node %+F scheduled after non-Phi nodes (for example %+F) in block %+F (%s)\n", + node, non_phi_found, block, get_irg_name(env->irg)); env->problem_found = 1; } } else { - non_phi_found = 1; + non_phi_found = node; } /* Check for control flow changing nodes */ - if (is_cfop(node) && get_irn_opcode(node) != iro_Start) { + if (is_cfop(node)) { /* check, that only one CF operation is scheduled */ - if (cfchange_found == 1) { - ir_fprintf(stderr, "Verify Warning: More than 1 control flow changing node (%+F) scheduled in block %+F (%s)\n", - node, block, get_irg_dump_name(env->irg)); + if (cfchange_found != NULL) { + ir_fprintf(stderr, "Verify Warning: Additional control flow changing node %+F scheduled after %+F in block %+F (%s)\n", + node, block, cfchange_found, get_irg_name(env->irg)); env->problem_found = 1; + } else { + cfchange_found = node; } - cfchange_found = 1; - } else if (cfchange_found) { + } else if (cfchange_found != NULL) { /* proj and keepany aren't real instructions... */ - if(!is_Proj(node) && !be_is_Keep(node)) { - /* check for delay branches */ - if (delay_branches == 0) { - ir_fprintf(stderr, "Verify Warning: Node %+F scheduled after control flow changing node (+delay branches) in block %+F (%s)\n", - node, block, get_irg_dump_name(env->irg)); - env->problem_found = 1; - } else { - delay_branches--; - } + if (!is_Proj(node) && !be_is_Keep(node)) { + ir_fprintf(stderr, "Verify Warning: Node %+F scheduled after control flow changing node in block %+F (%s)\n", + node, block, get_irg_name(env->irg)); + env->problem_found = 1; } } /* Check that all uses come before their definitions */ - if(!is_Phi(node)) { - int nodetime = sched_get_time_step(node); - for(i = 0, arity = get_irn_arity(node); i < arity; ++i) { + if (!is_Phi(node)) { + int i; + int arity; + sched_timestep_t nodetime = sched_get_time_step(node); + for (i = 0, arity = get_irn_arity(node); i < arity; ++i) { ir_node *arg = get_irn_n(node, i); - if(get_nodes_block(arg) != block + if (get_nodes_block(arg) != block || !sched_is_scheduled(arg)) continue; - if(sched_get_time_step(arg) >= nodetime) { + if (sched_get_time_step(arg) >= nodetime) { ir_fprintf(stderr, "Verify Warning: Value %+F used by %+F before it was defined in block %+F (%s)\n", - arg, node, block, get_irg_dump_name(env->irg)); + arg, node, block, get_irg_name(env->irg)); env->problem_found = 1; } } } /* Check that no dead nodes are scheduled */ - if(get_irn_n_edges(node) == 0) { + if (get_irn_n_edges(node) == 0) { ir_fprintf(stderr, "Verify warning: Node %+F is dead but scheduled in block %+F (%s)\n", - node, block, get_irg_dump_name(env->irg)); + node, block, get_irg_name(env->irg)); env->problem_found = 1; } - if(be_is_Keep(node)) { - /* at least 1 of the keep arguments has to be it schedule + if (be_is_Keep(node) || be_is_CopyKeep(node)) { + /* at least 1 of the keep arguments has to be its schedule * predecessor */ int arity = get_irn_arity(node); - int problem = 1; + bool found = false; ir_node *prev = sched_prev(node); - while(be_is_Keep(prev)) + while (be_is_Keep(prev) || be_is_CopyKeep(prev)) prev = sched_prev(prev); - for(i = 0; i < arity; ++i) { - ir_node *in = get_irn_n(node, i); - in = skip_Proj(in); - if(in == prev) - problem = 0; + while (true) { + int i; + for (i = 0; i < arity; ++i) { + ir_node *in = get_irn_n(node, i); + in = skip_Proj(in); + if (in == prev) + found = true; + } + if (found) + break; + prev = sched_prev(prev); + if (!is_Phi(prev)) + break; } - if(problem) { + if (!found) { ir_fprintf(stderr, "%+F not scheduled after its pred node in block %+F (%s)\n", - node, block, get_irg_dump_name(env->irg)); + node, block, get_irg_name(env->irg)); env->problem_found = 1; } } } - - /* check that all delay branches are filled (at least with NOPs) */ - if (cfchange_found && delay_branches != 0) { - ir_fprintf(stderr, "Verify warning: Not all delay slots filled after jump (%d/%d) in block %+F (%s)\n", - block, get_irg_dump_name(env->irg)); - env->problem_found = 1; - } } -static int should_be_scheduled(be_verify_schedule_env_t *env, ir_node *node) { - if(is_Block(node)) - return -1; - - if(get_irn_mode(node) == mode_M) { - if(is_Proj(node)) - return 0; - if(is_Phi(node) || is_Sync(node) || is_Pin(node)) - return 0; - } -#ifdef SCHEDULE_PROJS - if(is_Proj(node)) { - if(get_irn_mode(node) == mode_X) - return 0; - return should_be_scheduled(env, get_Proj_pred(node)); - } -#else - if(is_Proj(node)) - return 0; -#endif - if(be_is_Keep(node) && get_irn_opcode(get_nodes_block(node)) == iro_Bad) - return 0; - - switch(get_irn_opcode(node)) { - case iro_End: - case iro_NoMem: - case iro_Bad: - case iro_Unknown: - return 0; - default: - break; - } - - if(arch_irn_get_flags(env->arch_env, node) & arch_irn_flags_ignore) - return -1; - - return 1; -} - -static void check_schedule(ir_node *node, void *data) { - be_verify_schedule_env_t *env = data; - int should_be; - int scheduled; - - should_be = should_be_scheduled(env, node); - if(should_be == -1) - return; +static void check_schedule(ir_node *node, void *data) +{ + be_verify_schedule_env_t *env = (be_verify_schedule_env_t*)data; + bool should_be = !is_Proj(node) && !(arch_get_irn_flags(node) & arch_irn_flags_not_scheduled); + bool scheduled = bitset_is_set(env->scheduled, get_irn_idx(node)); - scheduled = bitset_is_set(env->scheduled, get_irn_idx(node)) ? 1 : 0; - should_be = should_be ? 1 : 0; - if(should_be != scheduled) { + if (should_be != scheduled) { ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) should%s be scheduled\n", - node, get_nodes_block(node), get_irg_dump_name(env->irg), should_be ? "" : " not"); + node, get_nodes_block(node), get_irg_name(env->irg), should_be ? "" : " not"); env->problem_found = 1; } } @@ -347,18 +299,17 @@ static void check_schedule(ir_node *node, void *data) { /** * Start a walk over the irg and check schedule. */ -int be_verify_schedule(const be_irg_t *birg) +int be_verify_schedule(ir_graph *irg) { be_verify_schedule_env_t env; env.problem_found = 0; - env.irg = be_get_birg_irg(birg); + env.irg = irg; env.scheduled = bitset_alloca(get_irg_last_idx(env.irg)); - env.arch_env = birg->main_env->arch_env; - irg_block_walk_graph(env.irg, verify_schedule_walker, NULL, &env); + irg_block_walk_graph(irg, verify_schedule_walker, NULL, &env); /* check if all nodes are scheduled */ - irg_walk_graph(env.irg, check_schedule, NULL, &env); + irg_walk_graph(irg, check_schedule, NULL, &env); return ! env.problem_found; } @@ -369,57 +320,60 @@ int be_verify_schedule(const be_irg_t *birg) -typedef struct _spill_t { +typedef struct spill_t { ir_node *spill; ir_entity *ent; } spill_t; typedef struct { - const arch_env_t *arch_env; - ir_graph *irg; - set *spills; - ir_node **reloads; - int problem_found; + ir_graph *irg; + set *spills; + ir_node **reloads; + int problem_found; } be_verify_spillslots_env_t; -static int cmp_spill(const void* d1, const void* d2, size_t size) { - const spill_t* s1 = d1; - const spill_t* s2 = d2; +static int cmp_spill(const void* d1, const void* d2, size_t size) +{ + const spill_t* s1 = (const spill_t*)d1; + const spill_t* s2 = (const spill_t*)d2; (void) size; return s1->spill != s2->spill; } -static spill_t *find_spill(be_verify_spillslots_env_t *env, ir_node *node) { +static spill_t *find_spill(be_verify_spillslots_env_t *env, ir_node *node) +{ spill_t spill; spill.spill = node; - return set_find(env->spills, &spill, sizeof(spill), HASH_PTR(node)); + return set_find(spill_t, env->spills, &spill, sizeof(spill), hash_ptr(node)); } -static spill_t *get_spill(be_verify_spillslots_env_t *env, ir_node *node, ir_entity *ent) { +static spill_t *get_spill(be_verify_spillslots_env_t *env, ir_node *node, ir_entity *ent) +{ spill_t spill, *res; - int hash = HASH_PTR(node); + int hash = hash_ptr(node); spill.spill = node; - res = set_find(env->spills, &spill, sizeof(spill), hash); + res = set_find(spill_t, env->spills, &spill, sizeof(spill), hash); - if(res == NULL) { + if (res == NULL) { spill.ent = ent; - res = set_insert(env->spills, &spill, sizeof(spill), hash); + res = set_insert(spill_t, env->spills, &spill, sizeof(spill), hash); } return res; } -static ir_node *get_memory_edge(const ir_node *node) { +static ir_node *get_memory_edge(const ir_node *node) +{ int i, arity; ir_node *result = NULL; arity = get_irn_arity(node); - for(i = arity - 1; i >= 0; --i) { + for (i = arity - 1; i >= 0; --i) { ir_node *arg = get_irn_n(node, i); - if(get_irn_mode(arg) == mode_M) { + if (get_irn_mode(arg) == mode_M) { assert(result == NULL); result = arg; } @@ -428,34 +382,34 @@ static ir_node *get_memory_edge(const ir_node *node) { return result; } -static -void collect(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent); +static void collect(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent); -static -void be_check_entity(be_verify_spillslots_env_t *env, ir_node *node, ir_entity *ent) { - if(ent == NULL) { +static void be_check_entity(be_verify_spillslots_env_t *env, ir_node *node, ir_entity *ent) +{ + if (ent == NULL) { ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) should have an entity assigned\n", - node, get_nodes_block(node), get_irg_dump_name(env->irg)); + node, get_nodes_block(node), get_irg_name(env->irg)); } } -static -void collect_spill(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent) { - ir_entity *spillent = arch_get_frame_entity(env->arch_env, node); +static void collect_spill(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent) +{ + ir_entity *spillent = arch_get_frame_entity(node); be_check_entity(env, node, spillent); get_spill(env, node, ent); - if(spillent != ent) { + if (spillent != ent) { ir_fprintf(stderr, "Verify warning: Spill %+F has different entity than reload %+F in block %+F(%s)\n", - node, reload, get_nodes_block(node), get_irg_dump_name(env->irg)); + node, reload, get_nodes_block(node), get_irg_name(env->irg)); env->problem_found = 1; } } -static void collect_memperm(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent) { +static void collect_memperm(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent) +{ int i, arity; spill_t spill, *res; - int hash = HASH_PTR(node); + int hash = hash_ptr(node); int out; ir_node* memperm; ir_entity *spillent; @@ -467,22 +421,22 @@ static void collect_memperm(be_verify_spillslots_env_t *env, ir_node *node, ir_n spillent = be_get_MemPerm_out_entity(memperm, out); be_check_entity(env, memperm, spillent); - if(spillent != ent) { + if (spillent != ent) { ir_fprintf(stderr, "Verify warning: MemPerm %+F has different entity than reload %+F in block %+F(%s)\n", - node, reload, get_nodes_block(node), get_irg_dump_name(env->irg)); + node, reload, get_nodes_block(node), get_irg_name(env->irg)); env->problem_found = 1; } spill.spill = node; - res = set_find(env->spills, &spill, sizeof(spill), hash); - if(res != NULL) { + res = set_find(spill_t, env->spills, &spill, sizeof(spill), hash); + if (res != NULL) { return; } spill.ent = spillent; - res = set_insert(env->spills, &spill, sizeof(spill), hash); + res = set_insert(spill_t, env->spills, &spill, sizeof(spill), hash); - for(i = 0, arity = be_get_MemPerm_entity_arity(memperm); i < arity; ++i) { + for (i = 0, arity = be_get_MemPerm_entity_arity(memperm); i < arity; ++i) { ir_node* arg = get_irn_n(memperm, i + 1); ir_entity* argent = be_get_MemPerm_in_entity(memperm, i); @@ -490,41 +444,43 @@ static void collect_memperm(be_verify_spillslots_env_t *env, ir_node *node, ir_n } } -static void collect_memphi(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity *ent) { +static void collect_memphi(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity *ent) +{ int i, arity; spill_t spill, *res; - int hash = HASH_PTR(node); + int hash = hash_ptr(node); assert(is_Phi(node)); spill.spill = node; - res = set_find(env->spills, &spill, sizeof(spill), hash); - if(res != NULL) { + res = set_find(spill_t, env->spills, &spill, sizeof(spill), hash); + if (res != NULL) { return; } spill.ent = ent; - res = set_insert(env->spills, &spill, sizeof(spill), hash); + res = set_insert(spill_t, env->spills, &spill, sizeof(spill), hash); /* is 1 of the arguments a spill? */ - for(i = 0, arity = get_irn_arity(node); i < arity; ++i) { + for (i = 0, arity = get_irn_arity(node); i < arity; ++i) { ir_node* arg = get_irn_n(node, i); collect(env, arg, reload, ent); } } -static void collect(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent) { - if(be_is_Spill(node)) { +static void collect(be_verify_spillslots_env_t *env, ir_node *node, ir_node *reload, ir_entity* ent) +{ + if (be_is_Spill(node)) { collect_spill(env, node, reload, ent); - } else if(is_Proj(node)) { + } else if (is_Proj(node)) { collect_memperm(env, node, reload, ent); - } else if(is_Phi(node) && get_irn_mode(node) == mode_M) { + } else if (is_Phi(node) && get_irn_mode(node) == mode_M) { collect_memphi(env, node, reload, ent); } else { /* Disabled for now, spills might get transformed by the backend */ #if 0 ir_fprintf(stderr, "Verify warning: No spill, memperm or memphi attached to node %+F found from node %+F in block %+F(%s)\n", - node, reload, get_nodes_block(node), get_irg_dump_name(env->irg)); + node, reload, get_nodes_block(node), get_irg_name(env->irg)); env->problem_found = 1; #endif } @@ -534,25 +490,21 @@ static void collect(be_verify_spillslots_env_t *env, ir_node *node, ir_node *rel * This walker function searches for reloads and collects all the spills * and memphis attached to them. */ -static void collect_spills_walker(ir_node *node, void *data) { - be_verify_spillslots_env_t *env = data; - const arch_env_t *arch_env = env->arch_env; - - /* @@@ ia32_classify returns classification of Proj_pred :-/ */ - if(is_Proj(node)) - return; +static void collect_spills_walker(ir_node *node, void *data) +{ + be_verify_spillslots_env_t *env = (be_verify_spillslots_env_t*)data; - if(arch_irn_class_is(arch_env, node, reload)) { + if (be_is_Reload(node)) { ir_node *spill = get_memory_edge(node); ir_entity *ent; - if(spill == NULL) { + if (spill == NULL) { ir_fprintf(stderr, "Verify warning: No spill attached to reload %+F in block %+F(%s)\n", - node, get_nodes_block(node), get_irg_dump_name(env->irg)); + node, get_nodes_block(node), get_irg_name(env->irg)); env->problem_found = 1; return; } - ent = arch_get_frame_entity(env->arch_env, node); + ent = arch_get_frame_entity(node); be_check_entity(env, node, ent); collect(env, spill, node, ent); @@ -560,30 +512,31 @@ static void collect_spills_walker(ir_node *node, void *data) { } } -static void check_spillslot_interference(be_verify_spillslots_env_t *env) { - int spillcount = set_count(env->spills); - spill_t **spills = alloca(spillcount * sizeof(spills[0])); - spill_t *spill; - int i; +static void check_spillslot_interference(be_verify_spillslots_env_t *env) +{ + int spillcount = set_count(env->spills); + spill_t **spills = ALLOCAN(spill_t*, spillcount); + int i; - for(spill = set_first(env->spills), i = 0; spill != NULL; spill = set_next(env->spills), ++i) { - spills[i] = spill; + i = 0; + foreach_set(env->spills, spill_t, spill) { + spills[i++] = spill; } - for(i = 0; i < spillcount; ++i) { + for (i = 0; i < spillcount; ++i) { spill_t *sp1 = spills[i]; int i2; - for(i2 = i+1; i2 < spillcount; ++i2) { + for (i2 = i+1; i2 < spillcount; ++i2) { spill_t *sp2 = spills[i2]; - if(sp1->ent != sp2->ent) + if (sp1->ent != sp2->ent) continue; - if(my_values_interfere(sp1->spill, sp2->spill)) { + if (my_values_interfere(sp1->spill, sp2->spill)) { ir_fprintf(stderr, "Verify warning: Spillslots for %+F in block %+F(%s) and %+F in block %+F(%s) interfere\n", - sp1->spill, get_nodes_block(sp1->spill), get_irg_dump_name(env->irg), - sp2->spill, get_nodes_block(sp2->spill), get_irg_dump_name(env->irg)); + sp1->spill, get_nodes_block(sp1->spill), get_irg_name(env->irg), + sp2->spill, get_nodes_block(sp2->spill), get_irg_name(env->irg)); env->problem_found = 1; my_values_interfere(sp1->spill, sp2->spill); } @@ -591,28 +544,28 @@ static void check_spillslot_interference(be_verify_spillslots_env_t *env) { } } -static void check_lonely_spills(ir_node *node, void *data) { - be_verify_spillslots_env_t *env = data; +static void check_lonely_spills(ir_node *node, void *data) +{ + be_verify_spillslots_env_t *env = (be_verify_spillslots_env_t*)data; - if(be_is_Spill(node) || (is_Proj(node) && be_is_MemPerm(get_Proj_pred(node)))) { + if (be_is_Spill(node) || (is_Proj(node) && be_is_MemPerm(get_Proj_pred(node)))) { spill_t *spill = find_spill(env, node); - if(be_is_Spill(node)) { - ir_entity *ent = arch_get_frame_entity(env->arch_env, node); + if (be_is_Spill(node)) { + ir_entity *ent = arch_get_frame_entity(node); be_check_entity(env, node, ent); } - if(spill == NULL) { - ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) not connected to a reaload\n", - node, get_nodes_block(node), get_irg_dump_name(env->irg)); + if (spill == NULL) { + ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) not connected to a reload\n", + node, get_nodes_block(node), get_irg_name(env->irg)); } } } -int be_verify_spillslots(const arch_env_t *arch_env, ir_graph *irg) +int be_verify_spillslots(ir_graph *irg) { be_verify_spillslots_env_t env; - env.arch_env = arch_env; env.irg = irg; env.spills = new_set(cmp_spill, 10); env.reloads = NEW_ARR_F(ir_node*, 0); @@ -641,21 +594,22 @@ int be_verify_spillslots(const arch_env_t *arch_env, ir_graph *irg) * @param b The second value. * @return 1, if a and b interfere, 0 if not. */ -static int my_values_interfere(const ir_node *a, const ir_node *b) { +static int my_values_interfere(const ir_node *a, const ir_node *b) +{ const ir_edge_t *edge; ir_node *bb; int a2b = value_dominates(a, b); int b2a = value_dominates(b, a); /* If there is no dominance relation, they do not interfere. */ - if(!a2b && !b2a) + if (!a2b && !b2a) return 0; /* * Adjust a and b so, that a dominates b if * a dominates b or vice versa. */ - if(b2a) { + if (b2a) { const ir_node *t = a; a = b; b = t; @@ -676,21 +630,21 @@ static int my_values_interfere(const ir_node *a, const ir_node *b) { */ foreach_out_edge(a, edge) { const ir_node *user = get_edge_src_irn(edge); - if(b == user) + if (b == user) continue; - if(get_irn_opcode(user) == iro_End) + if (get_irn_opcode(user) == iro_End) continue; /* in case of phi arguments we compare with the block the value comes from */ - if(is_Phi(user)) { + if (is_Phi(user)) { ir_node *phiblock = get_nodes_block(user); - if(phiblock == bb) + if (phiblock == bb) continue; user = get_irn_n(phiblock, get_edge_src_pos(edge)); } - if(value_dominates(b, user)) + if (value_dominates(b, user)) return 1; } @@ -701,212 +655,350 @@ static int my_values_interfere(const ir_node *a, const ir_node *b) { /*--------------------------------------------------------------------------- */ +static const arch_env_t *arch_env; +static ir_graph *irg; +static be_lv_t *lv; +static bool problem_found; +static const ir_node **registers; - -typedef struct _be_verify_register_allocation_env_t { - const arch_env_t *arch_env; - ir_graph *irg; - be_lv_t *lv; - int problem_found; -} be_verify_register_allocation_env_t; - -static void check_register_constraints(ir_node *node, - be_verify_register_allocation_env_t *env) +static void check_output_constraints(const ir_node *node) { - const arch_env_t *arch_env = env->arch_env; - const arch_register_t *reg; - int i, arity; + if (arch_get_irn_reg_class(node) == NULL) + return; /* verify output register */ - if (arch_get_irn_reg_class(arch_env, node, -1) != NULL) { - reg = arch_get_irn_register(arch_env, node); - if (reg == NULL) { - ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) should have a register assigned\n", - node, get_nodes_block(node), get_irg_dump_name(env->irg)); - env->problem_found = 1; - } - else if (! arch_register_type_is(reg, joker) && !arch_reg_is_allocatable(arch_env, node, -1, reg)) { - ir_fprintf(stderr, "Verify warning: Register %s assigned as output of %+F not allowed (register constraint) in block %+F(%s)\n", - reg->name, node, get_nodes_block(node), get_irg_dump_name(env->irg)); - env->problem_found = 1; - } + const arch_register_req_t *req = arch_get_irn_register_req(node); + const arch_register_t *reg = arch_get_irn_register(node); + if (reg == NULL) { + ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) should have a register assigned\n", + node, get_nodes_block(node), get_irg_name(irg)); + problem_found = true; + } else if (!arch_reg_is_allocatable(req, reg)) { + ir_fprintf(stderr, "Verify warning: Register %s assigned as output of %+F not allowed (register constraint) in block %+F(%s)\n", + reg->name, node, get_nodes_block(node), get_irg_name(irg)); + problem_found = true; } +} + +static void check_input_constraints(ir_node *node) +{ + const arch_register_t *reg; + int i, arity; /* verify input register */ arity = get_irn_arity(node); for (i = 0; i < arity; ++i) { - ir_node *pred = get_irn_n(node, i); - - if (is_Unknown(pred)) - continue; + const arch_register_req_t *req = arch_get_irn_register_req_in(node, i); + ir_node *pred = get_irn_n(node, i); + const arch_register_req_t *pred_req = arch_get_irn_register_req(pred); if (is_Bad(pred)) { ir_fprintf(stderr, "Verify warning: %+F in block %+F(%s) has Bad as input %d\n", - node, get_nodes_block(node), get_irg_dump_name(env->irg), i); - env->problem_found = 1; + node, get_nodes_block(node), get_irg_name(irg), i); + problem_found = 1; continue; } - - if (arch_get_irn_reg_class(arch_env, node, i) == NULL) + if (req->cls == NULL) continue; - reg = arch_get_irn_register(arch_env, pred); + if (req->width > pred_req->width) { + ir_fprintf(stderr, "Verify warning: %+F in block %+F(%s) register width of value at input %d too small\n", + node, get_nodes_block(node), get_irg_name(irg), i); + problem_found = 1; + } + + reg = arch_get_irn_register(pred); + if (req->type & arch_register_req_type_aligned) { + if (reg->index % req->width != 0) { + ir_fprintf(stderr, "Verify warning: %+F in block %+F(%s) register allignment not fulfilled\n", + node, get_nodes_block(node), get_irg_name(irg), i); + problem_found = 1; + } + } + if (reg == NULL) { ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) should have a register assigned (%+F input constraint)\n", - pred, get_nodes_block(pred), get_irg_dump_name(env->irg), node); - env->problem_found = 1; + pred, get_nodes_block(pred), get_irg_name(irg), node); + problem_found = 1; continue; - } - else if (! arch_register_type_is(reg, joker) && ! arch_reg_is_allocatable(arch_env, node, i, reg)) { + } else if (!arch_reg_is_allocatable(req, reg)) { ir_fprintf(stderr, "Verify warning: Register %s as input %d of %+F not allowed (register constraint) in block %+F(%s)\n", - reg->name, i, node, get_nodes_block(node), get_irg_dump_name(env->irg)); - env->problem_found = 1; + reg->name, i, node, get_nodes_block(node), get_irg_name(irg)); + problem_found = 1; + } + } + + /* phis should be NOPs at this point, which means all input regs + * must be the same as the output reg */ + if (is_Phi(node)) { + reg = arch_get_irn_register(node); + + arity = get_irn_arity(node); + for (i = 0; i < arity; ++i) { + ir_node *pred = get_Phi_pred(node, i); + const arch_register_t *pred_reg = arch_get_irn_register(pred); + + if (reg != pred_reg && !(pred_reg->type & arch_register_type_joker)) { + const char *pred_name = pred_reg != NULL ? pred_reg->name : "(null)"; + const char *reg_name = reg != NULL ? reg->name : "(null)"; + ir_fprintf(stderr, "Verify warning: Input %d of %+F in block %+F(%s) uses register %s instead of %s\n", + i, node, get_nodes_block(node), + get_irg_name(irg), pred_name, reg_name); + problem_found = 1; + } } } } -static void check_register_allocation(be_verify_register_allocation_env_t *env, - const arch_register_class_t *regclass, - ir_nodeset_t *nodes) +static void value_used(const ir_node *block, const ir_node *node) { - const arch_env_t *arch_env = env->arch_env; - const arch_register_t *reg = NULL; - int fail = 0; - bitset_t *registers = bitset_alloca(arch_register_class_n_regs(regclass)); - ir_node *node; - ir_nodeset_iterator_t iter; - - foreach_ir_nodeset(nodes, node, iter) { - if (arch_get_irn_reg_class(arch_env, node, -1) != regclass) - continue; + const arch_register_t *reg = arch_get_irn_register(node); + const arch_register_req_t *req; + unsigned i; + unsigned idx; - reg = arch_get_irn_register(arch_env, node); - - /* this problem is already reported in 'check_register_constraints' */ - if (! reg) - continue; + if (reg == NULL || reg->type & arch_register_type_virtual) + return; - if (bitset_is_set(registers, reg->index)) { - fail = 1; - break; + req = arch_get_irn_register_req(node); + assert(req->width > 0); + idx = reg->global_index; + for (i = 0; i < req->width; ++i) { + const ir_node *reg_node = registers[idx+i]; + if (reg_node != NULL && reg_node != node) { + const arch_register_t *realreg = &arch_env->registers[idx+i]; + ir_fprintf(stderr, "Verify warning: Register %s assigned more than once in block %+F(%s) (nodes %+F %+F)\n", + realreg->name, block, get_irg_name(irg), + node, reg_node); + problem_found = true; } - bitset_set(registers, reg->index); + registers[idx+i] = node; } +} - if (fail) { - ir_fprintf(stderr, "Verify warning: Register %s assigned more than once in block %+F(%s)\n", - reg->name, get_nodes_block(node), get_irg_dump_name(env->irg)); - env->problem_found = 1; +static void value_def(const ir_node *node) +{ + const arch_register_t *reg = arch_get_irn_register(node); + const arch_register_req_t *req; + unsigned idx; + unsigned i; - foreach_ir_nodeset(nodes, node, iter) { - if (arch_get_irn_register(arch_env, node) == reg) { - ir_fprintf(stderr, " at node %+F\n", node); - } + if (reg == NULL || reg->type & arch_register_type_virtual) + return; + + req = arch_get_irn_register_req(node); + assert(req->width > 0); + idx = reg->global_index; + for (i = 0; i < req->width; ++i) { + const ir_node *reg_node = registers[idx+i]; + + /* a little cheat, since its so hard to remove all outedges to dead code + * in the backend. This particular case should never be a problem. */ + if (reg_node == NULL && get_irn_n_edges(node) == 0) + return; + + if (reg_node != node) { + const arch_register_t *realreg = &arch_env->registers[idx+i]; + ir_fprintf(stderr, "Verify warning: Node %+F not registered as value for Register %s (but %+F) in block %+F(%s)\n", + node, realreg->name, reg_node, get_nodes_block(node), + get_irg_name(irg)); + problem_found = true; } + registers[idx+i] = NULL; } } -static void verify_block_register_allocation(ir_node *block, void *data) { - be_verify_register_allocation_env_t *env = data; - const arch_env_t *arch_env = env->arch_env; - int i, nregclasses; +static void verify_block_register_allocation(ir_node *block, void *data) +{ + unsigned i; + ir_node *node; + unsigned n_regs; + int idx; + + (void) data; - nregclasses = arch_env_get_n_reg_class(arch_env); - for (i = 0; i < nregclasses; ++i) { - const arch_register_class_t *regclass = arch_env_get_reg_class(arch_env, i); - ir_node *node; - ir_nodeset_t live_nodes; + assert(lv->sets_valid && "live sets must be computed"); - ir_nodeset_init(&live_nodes); + n_regs = arch_env->n_registers; + registers = ALLOCANZ(const ir_node*, n_regs); - be_liveness_end_of_block(env->lv, env->arch_env, regclass, block, - &live_nodes); - check_register_allocation(env, regclass, &live_nodes); + be_lv_foreach(lv, block, be_lv_state_end, idx) { + ir_node *lv_node = be_lv_get_irn(lv, block, idx); + value_used(block, lv_node); + } - sched_foreach_reverse(block, node) { - if (is_Phi(node)) - break; + sched_foreach_reverse(block, node) { + int arity; - be_liveness_transfer(env->arch_env, regclass, node, &live_nodes); - check_register_allocation(env, regclass, &live_nodes); - check_register_constraints(node, env); + if (get_irn_mode(node) == mode_T) { + const ir_edge_t *edge; + foreach_out_edge(node, edge) { + ir_node *def = get_edge_src_irn(edge); + value_def(def); + check_output_constraints(def); + } + } else { + value_def(node); + check_output_constraints(node); } - ir_nodeset_destroy(&live_nodes); + check_input_constraints(node); + + /* process uses. (Phi inputs are no real uses) */ + if (!is_Phi(node)) { + int in; + arity = get_irn_arity(node); + for (in = 0; in < arity; ++in) { + ir_node *use = get_irn_n(node, in); + value_used(block, use); + } + } } -} -int be_verify_register_allocation(const be_irg_t *birg) { - be_verify_register_allocation_env_t env; + be_lv_foreach(lv, block, be_lv_state_in, idx) { + ir_node *lv_node = be_lv_get_irn(lv, block, idx); + value_def(lv_node); + } - env.arch_env = be_get_birg_arch_env(birg); - env.irg = be_get_birg_irg(birg); - env.lv = be_liveness(birg); - env.problem_found = 0; + /* set must be empty now */ + for (i = 0; i < n_regs; ++i) { + if (registers[i] == NULL) + continue; - be_liveness_assure_sets(env.lv); - irg_block_walk_graph(env.irg, verify_block_register_allocation, NULL, &env); + ir_fprintf(stderr, "Verify warning: Node %+F not live-in and no def found in block %+F(%s)\n", + registers[i], block, get_irg_name(irg)); + problem_found = true; + } +} - be_liveness_free(env.lv); +bool be_verify_register_allocation(ir_graph *new_irg) +{ + irg = new_irg; + arch_env = be_get_irg_arch_env(irg); + lv = be_liveness_new(irg); + problem_found = false; - return !env.problem_found; + be_liveness_compute_sets(lv); + irg_block_walk_graph(irg, verify_block_register_allocation, NULL, NULL); + be_liveness_free(lv); + + return !problem_found; } +/*--------------------------------------------------------------------------- */ +/** + * Walker: checks that every predecessors of a node dominates the node. + */ +static void dom_check(ir_node *irn, void *data) +{ + bool *problem_found = (bool*)data; -/*--------------------------------------------------------------------------- */ + if (!is_Block(irn) && irn != get_irg_end(get_irn_irg(irn))) { + int i, n; + ir_node *bl = get_nodes_block(irn); + for (i = 0, n = get_irn_arity(irn); i < n; ++i) { + ir_node *op = get_irn_n(irn, i); + ir_node *def_bl = get_nodes_block(op); + ir_node *use_bl = bl; + if (is_Phi(irn)) + use_bl = get_Block_cfgpred_block(bl, i); -typedef struct _verify_out_dead_nodes_env { - ir_graph *irg; - bitset_t *reachable; - int problem_found; -} verify_out_dead_nodes_env; + if (get_irn_opcode(use_bl) != iro_Bad + && get_irn_opcode(def_bl) != iro_Bad + && !block_dominates(def_bl, use_bl)) { + ir_fprintf(stderr, "Verify warning: %+F in %+F must dominate %+F for user %+F (%s)\n", op, def_bl, use_bl, irn, get_irg_name(get_irn_irg(op))); + *problem_found = true; + } + } + } +} -static void check_out_edges(ir_node *node, verify_out_dead_nodes_env *env) { - ir_graph *irg = env->irg; - const ir_edge_t* edge; +/* Check, if the SSA dominance property is fulfilled. */ +bool be_check_dominance(ir_graph *irg) +{ + bool problem_found = false; - if(irn_visited(node)) - return; - mark_irn_visited(node); + assure_doms(irg); + irg_walk_graph(irg, dom_check, NULL, &problem_found); - /* we find too many (uncritical) dead nodes in block out edges */ - if(is_Block(node)) - return; + return !problem_found; +} - foreach_out_edge(node, edge) { - ir_node* src = get_edge_src_irn(edge); +/*--------------------------------------------------------------------------- */ - if(!bitset_is_set(env->reachable, get_irn_idx(src)) && !is_Block(src)) { - ir_fprintf(stderr, "Verify warning: Node %+F in block %+F(%s) only reachable through out edges from %+F\n", - src, get_nodes_block(src), get_irg_dump_name(irg), node); - env->problem_found = 1; - continue; - } +typedef struct lv_walker_t { + be_lv_t *lv; + void *data; +} lv_walker_t; - check_out_edges(src, env); - } +static const char *lv_flags_to_str(unsigned flags) +{ + static const char *states[] = { + "---", + "i--", + "-e-", + "ie-", + "--o", + "i-o", + "-eo", + "ieo" + }; + + return states[flags & 7]; } -static void set_reachable(ir_node *node, void* data) +static void lv_check_walker(ir_node *bl, void *data) { - bitset_t* reachable = data; - bitset_set(reachable, get_irn_idx(node)); -} + lv_walker_t *w = (lv_walker_t*)data; + be_lv_t *lv = w->lv; + be_lv_t *fresh = (be_lv_t*)w->data; -int be_verify_out_edges(ir_graph *irg) { - verify_out_dead_nodes_env env; + be_lv_info_t *curr = (be_lv_info_t*)ir_nodehashmap_get(&fresh->map, bl); + be_lv_info_t *fr = (be_lv_info_t*)ir_nodehashmap_get(&fresh->map, bl); - env.irg = irg; - env.reachable = bitset_alloca(get_irg_last_idx(irg)); - env.problem_found = edges_verify(irg); + if (!fr && curr && curr[0].head.n_members > 0) { + unsigned i; + + ir_fprintf(stderr, "%+F liveness should be empty but current liveness contains:\n", bl); + for (i = 0; i < curr[0].head.n_members; ++i) { + ir_fprintf(stderr, "\t%+F\n", get_idx_irn(lv->irg, curr[1 + i].node.idx)); + } + } - irg_walk_in_or_dep_graph(irg, set_reachable, NULL, env.reachable); - irg_walk_anchors(irg, set_reachable, NULL, env.reachable); - inc_irg_visited(irg); - check_out_edges(get_irg_start(irg), &env); + else if (curr) { + unsigned n_curr = curr[0].head.n_members; + unsigned n_fresh = fr[0].head.n_members; - return ! env.problem_found; + unsigned i; + + if (n_curr != n_fresh) { + ir_fprintf(stderr, "%+F: liveness set sizes differ. curr %d, correct %d\n", bl, n_curr, n_fresh); + + ir_fprintf(stderr, "current:\n"); + for (i = 0; i < n_curr; ++i) { + be_lv_info_node_t *n = &curr[1 + i].node; + ir_fprintf(stderr, "%+F %u %+F %s\n", bl, i, get_idx_irn(lv->irg, n->idx), lv_flags_to_str(n->flags)); + } + + ir_fprintf(stderr, "correct:\n"); + for (i = 0; i < n_fresh; ++i) { + be_lv_info_node_t *n = &fr[1 + i].node; + ir_fprintf(stderr, "%+F %u %+F %s\n", bl, i, get_idx_irn(lv->irg, n->idx), lv_flags_to_str(n->flags)); + } + } + } +} + +void be_liveness_check(be_lv_t *lv) +{ + lv_walker_t w; + be_lv_t *fresh = be_liveness_new(lv->irg); + + w.lv = lv; + w.data = fresh; + irg_block_walk_graph(lv->irg, lv_check_walker, NULL, &w); + be_liveness_free(fresh); }