combos constant dataflow analysis has to be consistent with the localopt; this should...
[libfirm] / ir / ana / vrp.c
index 5733680..d541609 100644 (file)
 #include "irop.h"
 #include "pdeq.h"
 #include "irphase_t.h"
-#include "irprintf.h"
+#include "debug.h"
 
 static char v;
 static void *VISITED = &v;
 
 struct vrp_env_t {
        waitq *workqueue;
+       DEBUG_ONLY(firm_dbg_module_t *dbg);
 };
 
 static vrp_attr *get_vrp_attr(const ir_node *node)
@@ -65,7 +66,6 @@ static int vrp_update_node(ir_node *node)
                return 0; /* we don't optimize for non-int-nodes*/
        }
 
-       ir_printf("update_vrp for %d called\n", get_irn_node_nr(node));
        vrp = get_vrp_attr(node);
 
        /* TODO: Check if all predecessors have valid VRP information*/
@@ -81,8 +81,8 @@ static int vrp_update_node(ir_node *node)
                break;
        }
        case iro_And: {
-               vrp_attr *vrp_left, *vrp_right;
-               ir_node *left, *right;
+               const vrp_attr *vrp_left, *vrp_right;
+               const ir_node *left, *right;
 
                left = get_And_left(node);
                right = get_And_right(node);
@@ -97,7 +97,7 @@ static int vrp_update_node(ir_node *node)
        case iro_Add: {
                int overflow_top, overflow_bottom;
                tarval *new_top, *new_bottom;
-               vrp_attr *vrp_left, *vrp_right;
+               const vrp_attr *vrp_left, *vrp_right;
                vrp_left = get_vrp_attr(get_Add_left(node));
                vrp_right = get_vrp_attr(get_Add_right(node));
 
@@ -129,7 +129,7 @@ static int vrp_update_node(ir_node *node)
        case iro_Sub: {
                int overflow_top, overflow_bottom;
                tarval *new_top, *new_bottom;
-               vrp_attr *vrp_left, *vrp_right;
+               const vrp_attr *vrp_left, *vrp_right;
                vrp_left = get_vrp_attr(get_Sub_left(node));
                vrp_right = get_vrp_attr(get_Sub_right(node));
 
@@ -158,7 +158,7 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Or: {
-               vrp_attr *vrp_left, *vrp_right;
+               const vrp_attr *vrp_left, *vrp_right;
 
                vrp_left = get_vrp_attr(get_Or_left(node));
                vrp_right = get_vrp_attr(get_Or_right(node));
@@ -170,8 +170,8 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Rotl: {
-               vrp_attr *vrp_left, *vrp_right;
-               ir_node *right = get_Rotl_right(node);
+               const vrp_attr *vrp_left, *vrp_right;
+               const ir_node *right = get_Rotl_right(node);
 
                vrp_left = get_vrp_attr(get_Rotl_left(node));
                vrp_right = get_vrp_attr(get_Rotl_right(node));
@@ -186,8 +186,8 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Shl: {
-               vrp_attr *vrp_left, *vrp_right;
-               ir_node *right = get_Shl_right(node);
+               const vrp_attr *vrp_left, *vrp_right;
+               const ir_node *right = get_Shl_right(node);
                vrp_left = get_vrp_attr(get_Shl_left(node));
                vrp_right = get_vrp_attr(get_Shl_right(node));
 
@@ -200,8 +200,8 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Shr: {
-               vrp_attr *vrp_left, *vrp_right;
-               ir_node *right = get_Shr_right(node);
+               const vrp_attr *vrp_left, *vrp_right;
+               const ir_node *right = get_Shr_right(node);
 
                vrp_left = get_vrp_attr(get_Shr_left(node));
                vrp_right = get_vrp_attr(get_Shr_right(node));
@@ -215,8 +215,8 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Shrs: {
-               vrp_attr *vrp_left, *vrp_right;
-               ir_node *right = get_Shrs_right(node);
+               const vrp_attr *vrp_left, *vrp_right;
+               const ir_node *right = get_Shrs_right(node);
 
                vrp_left = get_vrp_attr(get_Shrs_left(node));
                vrp_right = get_vrp_attr(get_Shrs_right(node));
@@ -230,8 +230,7 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Eor: {
-               tarval *bits_set, *bits_not_set;
-               vrp_attr *vrp_left, *vrp_right;
+               const vrp_attr *vrp_left, *vrp_right;
 
                vrp_left = get_vrp_attr(get_Eor_left(node));
                vrp_right = get_vrp_attr(get_Eor_right(node));
@@ -249,7 +248,7 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Id: {
-               vrp_attr *vrp_pred = get_vrp_attr(get_Id_pred(node));
+               const vrp_attr *vrp_pred = get_vrp_attr(get_Id_pred(node));
                new_bits_set = vrp_pred->bits_set;
                new_bits_not_set = vrp_pred->bits_not_set;
                new_range_top = vrp_pred->range_top;
@@ -259,16 +258,16 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Not: {
-               vrp_attr *vrp_pred = get_vrp_attr(get_Not_op(node));
+               const vrp_attr *vrp_pred = get_vrp_attr(get_Not_op(node));
                new_bits_set = tarval_not(vrp_pred->bits_not_set);
                new_bits_not_set = tarval_not(vrp_pred->bits_set);
                break;
        }
 
        case iro_Conv: {
-               ir_node *pred = get_Conv_op(node);
+               const ir_node *pred = get_Conv_op(node);
                ir_mode *old_mode = get_irn_mode(pred);
-               vrp_attr *vrp_pred = get_vrp_attr(pred);
+               const vrp_attr *vrp_pred = get_vrp_attr(pred);
 
                ir_mode *new_mode;
 
@@ -294,8 +293,8 @@ static int vrp_update_node(ir_node *node)
        }
 
        case iro_Confirm: {
-               pn_Cmp cmp = get_Confirm_cmp(node);
-               ir_node *bound = get_Confirm_bound(node);
+               const pn_Cmp cmp = get_Confirm_cmp(node);
+               const ir_node *bound = get_Confirm_bound(node);
 
 
                if (cmp == pn_Cmp_Lg) {
@@ -322,8 +321,8 @@ static int vrp_update_node(ir_node *node)
                pn_Cmp cmp;
                int i;
 
-               ir_node *pred = get_Phi_pred(node,0);
-               vrp_attr *vrp_pred = get_vrp_attr(pred);
+               const ir_node *pred = get_Phi_pred(node,0);
+               const vrp_attr *vrp_pred = get_vrp_attr(pred);
                new_range_top = vrp_pred->range_top;
                new_range_bottom = vrp_pred->range_bottom;
                new_range_type = vrp_pred->range_type;
@@ -375,6 +374,23 @@ static int vrp_update_node(ir_node *node)
        is_SymConst(node) is_Sync(node) is_Tuple(node)
        */
 
+       /* @todo: At this place, we check if the mode of the variable changed. A
+        * better place for this might be in the convopt.c file
+        */
+
+       if (new_bits_set != tarval_bad && get_tarval_mode(new_bits_set) != get_tarval_mode(vrp->bits_set)) {
+               vrp->bits_set = tarval_convert_to(vrp->bits_set, get_irn_mode(node));
+       }
+       if (new_bits_not_set != tarval_bad && get_tarval_mode(new_bits_not_set) != get_tarval_mode(vrp->bits_not_set)) {
+               vrp->bits_not_set = tarval_convert_to(vrp->bits_not_set, get_irn_mode(node));
+       }
+
+       if (vrp->range_type != VRP_UNDEFINED && new_range_type != VRP_UNDEFINED && get_tarval_mode(new_range_top) != get_tarval_mode(vrp->range_top)) {
+               /* @todo: We might be able to preserve this range information if it
+                * fits in */
+               vrp->range_type = VRP_VARYING;
+       }
+
        /* Merge the newly calculated values with those that might already exist*/
        if (new_bits_set != tarval_bad) {
                new_bits_set = tarval_or(new_bits_set, vrp->bits_set);
@@ -485,11 +501,10 @@ static void *vrp_init_node(ir_phase *phase, const ir_node *n, void *old)
 {
        ir_mode *mode;
        vrp_attr *vrp;
+       struct vrp_env_t *env = phase->priv;
 
-       ir_printf("initialized node nr: %d\n", get_irn_node_nr(n));
-       if (old) {
-               assert(1==0 && "init called for node already initialized");
-       }
+       DBG((env->dbg, LEVEL_2, "initialized node nr: %d\n", get_irn_node_nr(n)));
+       assert(old==NULL && "init called for node already initialized");
        vrp = phase_alloc(phase, sizeof(vrp_attr));
 
        memset(vrp, 0, sizeof(vrp_attr));
@@ -540,6 +555,7 @@ void set_vrp_data(ir_graph *irg)
                phase = new_phase(irg, vrp_init_node);
                irg_register_phase(irg, PHASE_VRP, phase);
                env = phase_alloc(phase, sizeof(*env));
+               FIRM_DBG_REGISTER(env->dbg, "ir.ana.vrp");
                phase->priv = env;
        } else {
                env = phase->priv;
@@ -603,14 +619,14 @@ vrp_attr *vrp_get_info(const ir_node *node)
 {
        const ir_graph *irg   = get_irn_irg(node);
        const ir_phase *phase = irg_get_phase(irg, PHASE_VRP);
+       vrp_attr       *vrp;
 
        if (phase == NULL) {
                /* phase has not yet been initialized */
                return NULL;
        }
 
-
-       vrp_attr *vrp = phase_get_irn_data(phase, node);
+       vrp = phase_get_irn_data(phase, node);
        if (vrp && vrp->valid) {
                return vrp;
        }