another similar bugfix
[libfirm] / ir / opt / ldstopt.c
index d792260..6dc6ba1 100644 (file)
 # include "irflag_t.h"
 # include "array.h"
 # include "irhooks.h"
+# include "opt_polymorphy.h"
+
+#ifdef DO_CACHEOPT
+#include "cacheopt/cachesim.h"
+#endif
 
 #undef IMAX
 #define IMAX(a,b)      ((a) > (b) ? (a) : (b))
@@ -234,7 +239,7 @@ static void collect_nodes(ir_node *node, void *env)
 }
 
 /**
- * returns a entity if the address ptr points to a constant one.
+ * Returns an entity if the address ptr points to a constant one.
  */
 static entity *find_constant_entity(ir_node *ptr)
 {
@@ -248,6 +253,15 @@ static entity *find_constant_entity(ir_node *ptr)
       entity *ent = get_Sel_entity(ptr);
       type *tp    = get_entity_owner(ent);
 
+      /* Do not fiddle about polymorphy. */
+      if (is_Class_type(get_entity_owner(ent)) &&
+         ((get_entity_n_overwrites(ent)    != 0) ||
+          (get_entity_n_overwrittenby(ent) != 0)   ) )
+       return NULL;
+
+      if (variability_constant == get_entity_variability(ent))
+       return ent;
+
       if (is_Array_type(tp)) {
         /* check bounds */
         int i, n;
@@ -287,6 +301,44 @@ static entity *find_constant_entity(ir_node *ptr)
   }
 }
 
+static long get_Sel_array_index_long(ir_node *n, int dim) {
+  ir_node *index = get_Sel_index(n, dim);
+  assert(get_irn_op(index) == op_Const);
+  return get_tarval_long(get_Const_tarval(index));
+}
+
+static compound_graph_path *rec_get_accessed_path(ir_node *ptr, int depth) {
+  compound_graph_path *res = NULL;
+  entity              *root, *field;
+  int                 path_len, pos;
+
+  if (get_irn_op(ptr) == op_SymConst) {
+    assert(get_SymConst_kind(ptr) == symconst_addr_ent);
+    root = get_SymConst_entity(ptr);
+    res = (depth == 0) ? NULL : new_compound_graph_path(get_entity_type(root), depth);
+  }
+  else {
+    assert(get_irn_op(ptr) == op_Sel);
+    res = rec_get_accessed_path(get_Sel_ptr(ptr), depth+1);
+    field = get_Sel_entity(ptr);
+    path_len = get_compound_graph_path_length(res);
+    pos = path_len - depth - 1;
+    set_compound_graph_path_node(res, pos, field);
+    if (is_Array_type(get_entity_owner(field))) {
+      assert(get_Sel_n_indexs(ptr) == 1 && "multi dim arrays not implemented");
+      set_compound_graph_path_array_index(res, pos, get_Sel_array_index_long(ptr, 0));
+    }
+  }
+  return res;
+}
+
+
+/** Returns an access path or NULL.  The access path is only
+ *  valid, if the graph is in phase_high and _no_ address computation is used. */
+static compound_graph_path *get_accessed_path(ir_node *ptr) {
+  return rec_get_accessed_path(ptr, 0);
+}
+
 /**
  * optimize a Load
  */
@@ -294,7 +346,7 @@ static int optimize_load(ir_node *load)
 {
   ldst_info_t *info = get_irn_link(load);
   ir_mode *load_mode = get_Load_mode(load);
-  ir_node *pred, *mem, *ptr;
+  ir_node *pred, *mem, *ptr, *new_node;
   entity *ent;
   int res = 0;
 
@@ -313,7 +365,7 @@ static int optimize_load(ir_node *load)
     if (get_irn_op(ptr) == op_Sel) {
       ir_node *mem = get_Sel_mem(ptr);
 
-      if (get_irn_op(mem) == op_Alloc) {
+      if (get_irn_op(skip_Proj(mem)) == op_Alloc) {
         /* ok, check the types */
         entity *ent  = get_Sel_entity(ptr);
         type *s_type = get_entity_type(ent);
@@ -328,7 +380,8 @@ static int optimize_load(ir_node *load)
         }
       }
     }
-    else if (get_irn_op(ptr) == op_Alloc) {
+    else if ((get_irn_op(skip_Proj(ptr)) == op_Alloc) ||
+            ((get_irn_op(ptr) == op_Cast) && (get_irn_op(skip_Proj(get_Cast_op(ptr))) == op_Alloc))) {
       /* simple case: a direct load after an Alloc. Firm Alloc throw
        * an exception in case of out-of-memory. So, there is no way for an
        * exception in this load.
@@ -354,6 +407,23 @@ static int optimize_load(ir_node *load)
   /* the mem of the Load. Must still be returned after optimization */
   mem  = get_Load_mem(load);
 
+  /* Load from a constant polymorphic field, where we can resolve
+     polymorphy. */
+  new_node = transform_node_Load(load);
+  if (new_node != load) {
+    if (info->projs[pn_Load_M]) {
+      exchange(info->projs[pn_Load_M], mem);
+      info->projs[pn_Load_M] = NULL;
+    }
+    if (info->projs[pn_Load_X_except]) {
+      exchange(info->projs[pn_Load_X_except], new_Bad());
+      info->projs[pn_Load_X_except] = NULL;
+    }
+    if (info->projs[pn_Load_res])
+      exchange(info->projs[pn_Load_res], new_node);
+    return 1;
+  }
+
   /* check if we can determine the entity that will be loaded */
   ent = find_constant_entity(ptr);
   if (ent) {
@@ -369,10 +439,11 @@ static int optimize_load(ir_node *load)
       }
 
       if (variability_constant == get_entity_variability(ent)
-         && is_atomic_entity(ent)) {   /* Might not be atomic after
-                                        lowering of Sels.  In this
-                                        case we could also load, but
-                                        it's more complicated. */
+               && is_atomic_entity(ent)) {
+        /* Might not be atomic after
+           lowering of Sels.  In this
+           case we could also load, but
+           it's more complicated. */
         /* more simpler case: we load the content of a constant value:
          * replace it by the constant itself
          */
@@ -394,8 +465,51 @@ static int optimize_load(ir_node *load)
         }
       }
       else if (variability_constant == get_entity_variability(ent)) {
+        compound_graph_path *path;
+       /*
         printf(">>>>>>>>>>>>> Found access to constant entity %s in function %s\n", get_entity_name(ent),
-            get_entity_name(get_irg_entity(current_ir_graph)));
+        get_entity_name(get_irg_entity(current_ir_graph)));
+        printf("  load: "); DDMN(load);
+        printf("  ptr:  "); DDMN(ptr);
+       */
+        path = get_accessed_path(ptr);
+        if (path) {
+          ir_node *c;
+
+          assert(is_proper_compound_graph_path(path, get_compound_graph_path_length(path)-1));
+          c = get_compound_ent_value_by_path(ent, path);
+
+          /* printf("  cons: "); DDMN(c); */
+
+          if (info->projs[pn_Load_M])
+            exchange(info->projs[pn_Load_M], mem);
+          if (info->projs[pn_Load_res])
+            exchange(info->projs[pn_Load_res], copy_const_value(c));
+          /*
+          {
+            int j;
+            for (j = 0; j < get_compound_graph_path_length(path); ++j) {
+              entity *node = get_compound_graph_path_node(path, j);
+              fprintf(stdout, ".%s", get_entity_name(node));
+              if (is_Array_type(get_entity_owner(node)))
+                      fprintf(stdout, "[%d]", get_compound_graph_path_array_index(path, j));
+            }
+            printf("\n");
+          }
+         */
+        } else {
+         /*  We can not determine a correct access path.  E.g., in jack, we load
+             a byte from an object to generate an exception.   Happens in test program
+             Reflectiontest.
+         printf(">>>>>>>>>>>>> Found access to constant entity %s in function %s\n", get_entity_name(ent),
+                get_entity_name(get_irg_entity(current_ir_graph)));
+         printf("  load: "); DDMN(load);
+         printf("  ptr:  "); DDMN(ptr);
+         if (get_irn_op(ptr) == op_SymConst &&
+               get_SymConst_kind(ptr) == symconst_addr_ent) { printf("        "); DDMEO(get_SymConst_entity(ptr)); }
+          printf("cannot optimize.\n");
+         */
+       }
       }
 
       /* we changed the irg, but try further */
@@ -587,7 +701,7 @@ static int optimize_phi(ir_node *phi, void *env)
 {
   walk_env_t *wenv = env;
   int i, n;
-  ir_node *store, *ptr, *block, *phiM, *phiD, *exc, *projM;
+  ir_node *store, *old_store, *ptr, *block, *phiM, *phiD, *exc, *projM;
   ir_mode *mode;
   ir_node **inM, **inD;
   int *idx;
@@ -604,6 +718,7 @@ static int optimize_phi(ir_node *phi, void *env)
     return 0;
 
   store = skip_Proj(get_Phi_pred(phi, 0));
+  old_store = store;
   if (get_irn_op(store) != op_Store)
     return 0;
 
@@ -687,6 +802,10 @@ static int optimize_phi(ir_node *phi, void *env)
 
   /* fourth step: create the Store */
   store = new_rd_Store(db, current_ir_graph, block, phiM, ptr, phiD);
+#ifdef DO_CACHEOPT
+  co_set_irn_name(store, co_get_irn_ident(old_store));
+#endif
+
   projM = new_rd_Proj(NULL, current_ir_graph, block, store, mode_M, pn_Store_M);
 
   info = get_ldst_info(store, wenv);