BugFix: turn_into_tuple() now works if edges are activated
[libfirm] / ir / ana / analyze_irg_args.c
index dbe28c1..f0a8462 100644 (file)
@@ -72,30 +72,59 @@ static unsigned analyze_arg(ir_node *arg, unsigned bits)
        "Call" isn't computed else we analyze that graph. If our
        reference is the address of this
        Call node that mean the reference will be read.*/
-    if (get_irn_op(succ) == op_Call) {
-      ir_node *Call_ptr  = get_Call_ptr(succ);
+    switch (get_irn_opcode(succ)) {
 
-      if (Call_ptr == arg) {
+    case iro_Call: {
+      ir_node *ptr  = get_Call_ptr(succ);
+
+      if (ptr == arg) {
         /* Hmm: not sure what this is, most likely a read */
         bits |= ptr_access_read;
       }
-      else if (op_SymConst == get_irn_op(Call_ptr) &&
-             get_SymConst_kind(Call_ptr) == symconst_addr_ent) {
-        entity *meth_ent = get_SymConst_entity(Call_ptr);
-
-             for (p = get_Call_n_params(succ) - 1; p >= 0; --p){
-               if (get_Call_param(succ, p) == arg) {
-            /* an arg can be used more than once ! */
-            bits |= get_method_param_access(meth_ent, p);
-               }
-             }
-      } else /* can do anything */
-        bits |= ptr_access_all;
+      else {
+        ir_op *op = get_irn_op(ptr);
+        entity *meth_ent;
+
+        if (op == op_SymConst && get_SymConst_kind(ptr) == symconst_addr_ent) {
+          meth_ent = get_SymConst_entity(ptr);
+
+          for (p = get_Call_n_params(succ) - 1; p >= 0; --p) {
+            if (get_Call_param(succ, p) == arg) {
+              /* an arg can be used more than once ! */
+              bits |= get_method_param_access(meth_ent, p);
+            }
+          }
+        }
+        else if (op == op_Sel && get_irp_callee_info_state() == irg_callee_info_consistent) {
+          /* is be a polymorphic call but callee information is available */
+          int i, n_params = get_Call_n_params(succ);
+
+          /* simply look into ALL possible callees */
+          for (i = get_Call_n_callees(succ) - 1; i >= 0; --i) {
+            meth_ent = get_Call_callee(succ, i);
+
+            /* unknown_entity is used to signal that we don't know what is called */
+            if (meth_ent == unknown_entity) {
+              bits |= ptr_access_all;
+              break;
+            }
+
+            for (p = n_params - 1; p >= 0; --p) {
+              if (get_Call_param(succ, p) == arg) {
+                /* an arg can be used more than once ! */
+                bits |= get_method_param_access(meth_ent, p);
+              }
+            }
+          }
+        }
+        else /* can do anything */
+          bits |= ptr_access_all;
+      }
 
       /* search stops here anyway */
       continue;
     }
-    else if (get_irn_op(succ) == op_Store) {
+    case iro_Store:
       /* We have reached a Store node => the reference is written or stored. */
       if (get_Store_ptr(succ) == arg) {
         /* written to */
@@ -108,18 +137,21 @@ static unsigned analyze_arg(ir_node *arg, unsigned bits)
 
       /* search stops here anyway */
       continue;
-   }
-    else if (get_irn_op(succ) == op_Load) {
+
+    case iro_Load:
       /* We have reached a Load node => the reference is read. */
       bits |= ptr_access_read;
 
       /* search stops here anyway */
       continue;
-    }
-    else if (get_irn_op(succ) == op_Conv) {
+
+    case iro_Conv:
       /* our address is casted into something unknown. Break our search. */
       bits = ptr_access_all;
       break;
+
+    default:
+      break;
     }
 
     /* If we know that, the argument will be read, write and stored, we
@@ -157,7 +189,7 @@ static void analyze_ent_args(entity *ent)
   ir_mode *arg_mode;
   int nparams, i;
   long proj_nr;
-  type *mtp;
+  ir_type *mtp;
   ptr_access_kind *rw_info;
 
   mtp     = get_entity_type(ent);
@@ -185,7 +217,7 @@ static void analyze_ent_args(entity *ent)
 
   /* Call algorithm that computes the out edges */
   if (get_irg_outs_state(irg) != outs_consistent)
-    compute_outs(irg);
+    compute_irg_outs(irg);
 
   irg_args = get_irg_args(irg);
 
@@ -211,6 +243,7 @@ static void analyze_ent_args(entity *ent)
   /* copy the temporary info */
   memcpy(ent->param_access, rw_info, nparams * sizeof(ent->param_access[0]));
 
+#if 0
   printf("\n%s:\n", get_entity_name(ent));
   for (i = 0; i < nparams; ++i) {
     if (is_Pointer_type(get_method_param_type(mtp, i)))
@@ -225,6 +258,7 @@ static void analyze_ent_args(entity *ent)
         printf("\n");
       }
   }
+#endif
 }
 
 /**
@@ -254,8 +288,8 @@ void analyze_irg_args(ir_graph *irg)
  */
 ptr_access_kind get_method_param_access(entity *ent, int pos)
 {
-  type *mtp = get_entity_type(ent);
-  int  is_variadic = get_method_variadicity(mtp) == variadicity_variadic;
+  ir_type *mtp = get_entity_type(ent);
+  int    is_variadic = get_method_variadicity(mtp) == variadicity_variadic;
 
   assert(0 <= pos && (is_variadic || pos < get_method_n_params(mtp)));
 
@@ -275,11 +309,11 @@ ptr_access_kind get_method_param_access(entity *ent, int pos)
 }
 
 enum args_weight {
-  null_weight        = 0,  /* If can't be anything optimized. */
-  binop_weight       = 1,  /* If the argument have mode_weight and take part in binop. */
-  const_binop_weight = 1,  /* If the argument have mode_weight and take part in binop with a constant.*/
-  cmp_weight         = 4,  /* If the argument take part in cmp. */
-  const_cmp_weight   = 10  /* If the argument take part in cmp with a constant. */
+  null_weight        = 0,  /**< If can't be anything optimized. */
+  binop_weight       = 1,  /**< If the argument have mode_weight and take part in binop. */
+  const_binop_weight = 1,  /**< If the argument have mode_weight and take part in binop with a constant.*/
+  cmp_weight         = 4,  /**< If the argument take part in cmp. */
+  const_cmp_weight   = 10  /**< If the argument take part in cmp with a constant. */
 };
 
 /**
@@ -358,7 +392,7 @@ static float calc_method_param_weight(ir_node *arg)
  */
 static void analyze_method_params_weight(entity *ent)
 {
-  type *mtp;
+  ir_type *mtp;
   ir_graph *irg;
   int nparams, i, proj_nr;
   ir_node *irg_args, *arg;
@@ -366,14 +400,16 @@ static void analyze_method_params_weight(entity *ent)
   mtp      = get_entity_type(ent);
   nparams  = get_method_n_params(mtp);
 
+  /* allocate a new array. currently used as 'analysed' flag */
+  ent->param_weight = NEW_ARR_F(float, nparams);
+
   /* If the method haven't parameters we have
    * nothing to do.
    */
   if (nparams <= 0)
     return;
 
-  ent->param_weight = NEW_ARR_F(float, nparams);
-  irg               = get_entity_irg(ent);
+  irg = get_entity_irg(ent);
 
   /* First we initialize the parameter weight with 0. */
   for (i = nparams - 1; i >= 0; i--)
@@ -386,7 +422,7 @@ static void analyze_method_params_weight(entity *ent)
 
   /* Call algorithm that computes the out edges */
   if (get_irg_outs_state(irg) != outs_consistent)
-    compute_outs(irg);
+    compute_irg_outs(irg);
 
   irg_args = get_irg_args(irg);
 
@@ -409,8 +445,8 @@ static void analyze_method_params_weight(entity *ent)
  */
 float get_method_param_weight(entity *ent, int pos)
 {
-  type *mtp = get_entity_type(ent);
-  int  is_variadic = get_method_variadicity(mtp) == variadicity_variadic;
+  ir_type *mtp = get_entity_type(ent);
+  int     is_variadic = get_method_variadicity(mtp) == variadicity_variadic;
 
   assert(0 <= pos && (is_variadic || pos < get_method_n_params(mtp)));
 
@@ -441,9 +477,9 @@ void analyze_irg_args_weight(ir_graph *irg)
   entity *ent;
 
   ent = get_irg_entity(irg);
-  if(! ent)
+  if (! ent)
     return;
 
-  if(! ent->param_weight)
+  if (! ent->param_weight)
     analyze_method_params_weight(ent);
 }