* File name: ir/opt/scalar_replace.c
* Purpose: scalar replacement of arrays and compounds
* Author: Beyhan Veliev
+ * Modified by: Michael Beck
* Created:
* CVS-ID: $Id$
* Copyright: (c) 1998-2005 Universität Karlsruhe
#include <malloc.h>
#endif
+#ifdef HAVE_STRING_H
+#include <string.h>
+#endif
+
#include "scalar_replace.h"
+#include "irflag_t.h"
+#include "irouts.h"
+#include "set.h"
+#include "pset.h"
+#include "array.h"
+#include "tv.h"
+#include "ircons_t.h"
+#include "hashptr.h"
+#include "irgwalk.h"
+#include "irgmod.h"
+#include "irnode_t.h"
+
+#define SET_VNUM(node, vnum) set_irn_link(node, (void *)vnum)
+#define GET_VNUM(node) (unsigned)get_irn_link(node)
/**
* A path element entry: it is either an entity
- * or a tarval, because the evaluate only constant array
+ * or a tarval, because we evaluate only constant array
* accesses like a.b.c[8].d
*/
typedef union {
/**
* An access path, used to assign value numbers
- * to variables that will be skalar replaced
+ * to variables that will be scalar replaced
*/
typedef struct _path_t {
unsigned vnum; /**< the value number */
path_elem_t path[1]; /**< the path */
} path_t;
-typedef struct {
- ir_node *irn; /**< Phi or unknown node from the graph to be repairing. */
- ir_node **link; /**< Array of Stores's volue, that have been scalar replaced. */
-} repairs_t;
-
typedef struct _scalars_t {
entity *ent; /**< A entity for scalar replacement. */
type *ent_owner; /**< The owner of this entity. */
const path_t *p1 = elt;
const path_t *p2 = key;
+ /* we can use memcmp here, because identical tarvals should have identical addresses */
return memcmp(p1->path, p2->path, p1->path_len * sizeof(p1->path[0]));
}
-/**
- * Compare two elements of the repairs_t set.
- *
- * @return 0 if they are identically
- */
-static int set_cmp(const void *elt, const void *key, size_t size)
-{
- const repairs_t *c1 = elt;
- const repairs_t *c2 = key;
-
- return c1->irn != c2->irn;
-}
-
/**
* Compare two elements of the scalars_t set.
*
}
/**
- * Returns non-zero, if the address of a Sel node (or it's succsessor Sels) is taken.
+ * Returns non-zero, if the address of an entity
+ * represented by a Sel node (or it's successor Sels) is taken.
*
* @param sel the Sel node
*/
-static int is_address_taken(type *ent_type, ir_node *sel)
+static int is_address_taken(ir_node *sel)
{
int i, n;
switch (get_irn_opcode(succ)) {
case iro_Load:
+ /* ok, we just load from that entity */
break;
case iro_Store:
- /* check that Sel is not the store's value */
+ /* check that Sel is not the Store's value */
if (get_Store_value(succ) == sel)
- return 1;
+ return 1;
break;
case iro_Sel: {
/* Check the Sel successor of Sel */
- int res = is_address_taken(ent_type, succ);
+ int res = is_address_taken(succ);
if (res)
return 1;
break;
}
+
+ case iro_Call:
+ /* The address of an entity is given as a parameter.
+ * As long as we do not have analyses that can tell what
+ * is done with parameters, think is taken.
+ */
+ return 1;
+
default:
/* another op, the address is taken */
return 1;
flag = 0;
}
}
+
if (flag) {
+ /* if Sel nodes with memory inputs are used, a entity can be
+ * visited more than once causing a ring here, so we use the
+ * node flag to mark linked nodes
+ */
+ if (irn_visited(sel))
+ return;
+
/* we know we are at a leave, because this function is only
* called if the address is NOT taken, so succ must be a Load
* or a Store node
*/
set_irn_link(sel, get_entity_link(ent));
set_entity_link(ent, sel);
+
+ mark_irn_visited(sel);
}
}
* This function finds variables on the (members of the) frame type
* that can be scalar replaced, because their address is never taken.
* If such a variable is found, it's entity link will hold a list of all
- * Sel nodes, that selects the atomar fields of this entity.
- * Otherwise, the link will be NULL.
+ * Sel nodes, that selects the atomic fields of this entity.
+ * Otherwise, the link will be ADDRESS_TAKEN or NULL.
+ *
+ * @return non-zero if at least one entity could be replaced
+ * potentially
*/
-static void do_find_scalar_replacements(ir_graph *irg)
+static int find_possible_replacements(ir_graph *irg)
{
ir_node *irg_frame = get_irg_frame(irg);
int i, n;
+ int res = 0;
+
+ inc_irg_visited(irg);
n = get_irn_n_outs(irg_frame);
- /* first, clear the link field of all interesting entities */
- for (i = 0 ; i < n; i++) {
+ /*
+ * First, clear the link field of all interesting entities.
+ * Note that we did not rely on the fact that there is only
+ * one Sel node per entity, so we might access one entity
+ * more than once here.
+ * That's why we have need two loops.
+ */
+ for (i = 0; i < n; ++i) {
ir_node *succ = get_irn_out(irg_frame, i);
if (get_irn_op(succ) == op_Sel) {
}
}
- /* Check the ir_graph for Sel nodes. If the entity of Sel isn't a scalar replacement
- set the link of this entity equal NULL. */
- for (i = 0 ; i < n; i++) {
+ /*
+ * Check the ir_graph for Sel nodes. If the entity of Sel
+ * isn't a scalar replacement set the link of this entity
+ * equal ADDRESS_TAKEN.
+ */
+ for (i = 0; i < n; ++i) {
ir_node *succ = get_irn_out(irg_frame, i);
if (get_irn_op(succ) == op_Sel) {
entity *ent = get_Sel_entity(succ);
- type *ent_type = get_entity_type(ent);
+ type *ent_type;
+
+ if (get_entity_link(ent) == ADDRESS_TAKEN)
+ continue;
+
+ ent_type = get_entity_type(ent);
/* we can handle arrays, structs and atomic types yet */
if (is_Array_type(ent_type) || is_Struct_type(ent_type) || is_atomic_type(ent_type)) {
- if (is_address_taken(ent_type, succ)) {
+ if (is_address_taken(succ)) {
+ if (get_entity_link(ent)) /* killing one */
+ --res;
set_entity_link(ent, ADDRESS_TAKEN);
}
- else {
- if (get_entity_link(ent) != ADDRESS_TAKEN)
- link_all_leave_sels(ent, succ);
+ else {
+ /* possible found one */
+ if (get_entity_link(ent) == NULL)
+ ++res;
+ link_all_leave_sels(ent, succ);
}
}
}
}
+
+ return res;
}
/**
* Return a path from the Sel node sel to it's root.
+ *
+ * @param sel the Sel node
+ * @param len the length of the path so far
*/
static path_t *find_path(ir_node *sel, unsigned len)
{
int pos, i, n;
path_t *res;
ir_node *pred = get_Sel_ptr(sel);
+
/* the current Sel node will add some path elements */
n = get_Sel_n_indexs(sel);
len += n + 1;
* Allocate value numbers for the leaves
* in our found entities.
*
+ * @param sels a set that will contain all Sels that have a value number
* @param ent the entity that will be scalar replaced
* @param vnum the first value number we can assign
* @param modes a flexible array, containing all the modes of
*
* @return the next free value number
*/
-static unsigned allocate_value_numbers(entity *ent, unsigned vnum, ir_mode ***modes)
+static unsigned allocate_value_numbers(pset *sels, entity *ent, unsigned vnum, ir_mode ***modes)
{
ir_node *sel, *next;
path_t *key, *path;
for (sel = get_entity_link(ent); sel; sel = next) {
next = get_irn_link(sel);
+ /* we must mark this sel for later */
+ pset_insert_ptr(sels, sel);
+
key = find_path(sel, 0);
path = set_find(pathes, key, sizeof(*key) + sizeof(key->path[0]) * key->path_len, path_hash(key));
if (path)
- set_irn_link(sel, (void *)path->vnum);
+ SET_VNUM(sel, path->vnum);
else {
unsigned i;
set_insert(pathes, key, sizeof(*key) + sizeof(key->path[0]) * key->path_len, path_hash(key));
- set_irn_link(sel, (void *)key->vnum);
+ SET_VNUM(sel, key->vnum);
ARR_EXTO(ir_mode *, *modes, (key->vnum + 15) & ~15);
(*modes)[key->vnum] = get_type_mode(get_entity_type(get_Sel_entity(sel)));
- printf(" %s", get_entity_name(ent));
- for (i = 1; i < key->path_len; ++i) {
- if (is_entity(key->path[i].ent))
- printf(".%s", get_entity_name(key->path[i].ent));
- else
- printf("[%ld]", get_tarval_long(key->path[i].tv));
+ assert((*modes)[key->vnum] && "Value is not atomic");
+
+#ifdef DEBUG_libfirm
+ /* Debug output */
+ if (get_opt_scalar_replacement_verbose() && get_firm_verbosity() > 1) {
+ printf(" %s", get_entity_name(ent));
+ for (i = 1; i < key->path_len; ++i) {
+ if (is_entity(key->path[i].ent))
+ printf(".%s", get_entity_name(key->path[i].ent));
+ else
+ printf("[%ld]", get_tarval_long(key->path[i].tv));
+ }
+ printf(" = %u (%s)\n", (int)get_irn_link(sel), get_mode_name((*modes)[key->vnum]));
}
- printf(" = %u (%s)\n", (int)get_irn_link(sel), get_mode_name((*modes)[key->vnum]));
+#endif /* DEBUG_libfirm */
}
free(key);
}
return vnum;
}
-static char p;
-static void *NODE_VISITED = &p;
-static char t;
-static void *LOOP_WALK = &t;
-static char s;
-static void *LOOP_HEAD_PHI = &s;
+/**
+ * A list entry for the fixing lists
+ */
+typedef struct _list_entry_t {
+ ir_node *node; /**< the node that must be fixed */
+ unsigned vnum; /**< the value number of this node */
+} list_entry_t;
/**
* environment for memory walker
*/
typedef struct _env_t {
- struct obstack obst; /**< a obstack for the value blocks */
- int nvals; /**< number of values */
- ir_mode **modes; /**< the modes of the values */
+ struct obstack obst; /**< a obstack for the value blocks */
+ int nvals; /**< number of values */
+ ir_mode **modes; /**< the modes of the values */
+ list_entry_t *fix_phis; /**< list of all Phi nodes that must be fixed */
+ list_entry_t *fix_loads; /**< list of all Load nodes that must be fixed */
+ pset *sels; /**< A set of all Sel nodes that have a value number */
} env_t;
-static char q;
-static void *PRED_SEARCH = &q;
-
/**
- * Recursive walk over the blocks, that are predecessors of "node".
- *
- * @param node A Load or Phi node. The predecessor of this node muss be faund.
- * @param value A struct pointer, that contains the block of node and their link.
- * @param repairs A set, that contains all blocks, that have a array as link, and all Phis, that
- * have copies to repair.
- * @param pos The position, that muss contain the predecessor of node, in the array, that have got the bloks
- * from the repairs set.
- * @param phi_pred If node is a Phi, phi_pred is the predecessor number of this Phi, that muss be repair.
+ * Walker
*/
-static void pred_search(ir_node *node, repairs_t *value, set *repairs, int pos, int phi_pred)
+static void handle_first(ir_node *node, void *ctx)
{
- ir_node *nodes_block;
- repairs_t key, *value_pred;
- int i, n;
+ env_t *env = ctx;
+ ir_op *op = get_irn_op(node);
+ ir_node *adr, *block, *mem, *unk, **value_arr, **in;
+ unsigned vnum;
+ int i, j, n;
+ list_entry_t *l;
+
+ if (op == op_Load) {
+ /* a load, check if we can resolve it */
+ adr = get_Load_ptr(node);
+
+ if (get_irn_op(adr) != op_Sel)
+ return;
- DDMN(node);
- nodes_block = get_nodes_block(node);
+ if (! pset_find_ptr(env->sels, adr))
+ return;
- /* If the predecessor is found. */
- if ((pos == -2 && value->link[pos] != NULL))
- return;
+ /* ok, we have a Load that will be replaced */
+ vnum = GET_VNUM(adr);
- n = get_Block_n_cfgpreds(nodes_block);
+ assert(vnum < (unsigned)env->nvals);
- for (i = n - 1; i >= 0; --i) {
- ir_node *pred = get_Block_cfgpred(nodes_block, i);
+ block = get_nodes_block(node);
+ value_arr = get_irn_link(block);
- key.irn = nodes_block;
- value_pred = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- /* If nodes_block don't have the necessary information and the predecessor of it isn't
- visited "pred_search" was called recursive. Else the necessary information is found
- and the recursion stops. */
- if (value_pred == NULL || value_pred->link[pos] == NULL) {
- if (get_irn_link(pred) != PRED_SEARCH) {
- set_irn_link(node, PRED_SEARCH);
- pred_search(pred, value, repairs, pos, phi_pred);
- }
+ /* check, if we can replace this Load */
+ if (value_arr[vnum]) {
+ mem = get_Load_mem(node);
+
+ turn_into_tuple(node, pn_Load_max);
+ set_Tuple_pred(node, pn_Load_M, mem);
+ set_Tuple_pred(node, pn_Load_res, value_arr[vnum]);
+ set_Tuple_pred(node, pn_Load_X_except, new_Bad());
}
else {
- if (value->link[pos] == NULL && pos != -2) {
- if (get_Block_dom_depth(value->irn) >=
- get_Block_dom_depth(get_nodes_block(value_pred->link[pos]))) {
- value->link[pos] = value_pred->link[pos];
- pos = -2;
- break;
- }
- }
- else {
- if (get_irn_op(value->link[pos]) == op_Phi && pos != -2 &&
- value->link[pos] != value_pred->link[pos]) {
- set_Phi_pred(value->link[pos], phi_pred, value_pred->link[pos]);
- pos = -2;
- break;
- }
- }
- }
- }
+ l = obstack_alloc(&env->obst, sizeof(*l));
+ l->node = node;
+ l->vnum = vnum;
- if (get_irn_link(node) == PRED_SEARCH)
- set_irn_link(node, NULL);
-}
-
-/**
- * Create a link for blocks, that have Phi, Load or Store nodes for scalar_replacement.
- *
- * @param node A Phi, Load or Store node, that block muss get a link.
- * @param env Contains information about scalars number and mode.
- * @param repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
- */
-static void block_link(ir_node *node, env_t *env, set *repairs)
-{
- repairs_t key, *value;
- ir_node *nods_block ;
- int i;
- nods_block = get_nodes_block(node);
- /* If the block of the node have no link. */
- if (get_irn_link(nods_block) == NULL) {
- DDMN(nods_block);
- key.irn = nods_block;
- value = set_insert(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- value->link = obstack_alloc(&env->obst, sizeof(ir_node *) * env->nvals);
- /* The block of the node be inserted in the set repairs, because it have yet a link.
- The link is a array with size equal to number of scalars.*/
- for (i = 0; i < env->nvals; i++)
- /* First all links member be set to NULL. */
- value->link[i] = NULL;
- set_irn_link(nods_block, value->link);
+ set_irn_link(node, env->fix_loads);
+ env->fix_loads = l;
+ }
}
-}
+ else if (op == op_Store) {
+ /* a Store always can be replaced */
+ adr = get_Store_ptr(node);
-/**
- * Handle Phis, that get the memory edge of Loads or Stors, that were been scalar replaced or
- * will be scalar replaced.
- *
- * @param phi A Phi node, that muss have mode_M.
- * @param env Contains information about scalars number and mode.
- * @param repairs A set that contains all blocks having a link and all Phis that
- * have copies to repair.
- */
-static void phi_handling(ir_node *phi, env_t *env, set *repairs)
-{
- ir_node *phi_block, **link;
- repairs_t key, *value;
- int rem;
- int p = 0, i, phi_preds;
- ir_node **in;
+ if (get_irn_op(adr) != op_Sel)
+ return;
- key.irn = phi;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
+ if (! pset_find_ptr(env->sels, adr))
+ return;
- /* Test if the phi have been handled. */
- if (value != NULL)
- return;
+ vnum = GET_VNUM(adr);
- rem = get_optimize();
+ assert(vnum < (unsigned)env->nvals);
- phi_block = get_nodes_block(phi);
- phi_preds = get_Phi_n_preds(phi);
+ block = get_nodes_block(node);
+ value_arr = get_irn_link(block);
- /* Test if the Phi have predecessors, that were been or will be scalar replaced. */
- for (i = 0; i < phi_preds; i++){
- ir_node *pred = get_Phi_pred(phi, i);
- ir_node *pred_block = get_nodes_block(pred);
+ value_arr[vnum] = get_Store_value(node);
- key.irn = pred_block;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
+ mem = get_Store_mem(node);
- if (value != NULL)
- p++;
- }
- /* Phis from the loop head muss be handled.*/
- if (get_irn_link(phi) == LOOP_HEAD_PHI)
- p++;
-
- /* If the Phi node have such predecessor(s), be inserted in the repairs set, else is nothing to do
- and exit "phi_handling".*/
- if (p) {
- key.irn = phi;
- value = set_insert(repairs, &key, sizeof(key), HASH_PTR(key.irn));
+ turn_into_tuple(node, pn_Store_max);
+ set_Tuple_pred(node, pn_Store_M, mem);
+ set_Tuple_pred(node, pn_Store_X_except, new_Bad());
}
- else
- return;
-
- //DDMN(phi);
- if (get_irn_link(phi_block) == NULL)
- block_link(phi, env, repairs);
+ else if (op == op_Phi && get_irn_mode(node) == mode_M) {
+ /*
+ * found a memory Phi: Here, we must create new Phi nodes
+ */
+ block = get_nodes_block(node);
+ value_arr = get_irn_link(block);
- link = get_irn_link(phi_block);
+ n = get_Block_n_cfgpreds(block);
- key.irn = phi_block;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(phi_block));
+ in = alloca(sizeof(*in) * n);
- in = alloca(phi_preds * sizeof(*in));
+ for (i = env->nvals - 1; i >= 0; --i) {
+ unk = new_Unknown(env->modes[i]);
+ for (j = n - 1; j >= 0; --j)
+ in[j] = unk;
- /* We will build some Phi nodes. Beware, as they have all Unknown predecessors, CSE
- * would combine them into one. To prevent this, we must deactivate Optimizations here
- */
- for (p = 0; p < env->nvals; ++p) {
+ value_arr[i] = new_r_Phi(current_ir_graph, block, n, in, env->modes[i]);
- for (i = 0; i < phi_preds; ++i)
- in[i] = new_Unknown(env->modes[p]);
+ l = obstack_alloc(&env->obst, sizeof(*l));
+ l->node = value_arr[i];
+ l->vnum = i;
- set_optimize(0);
- value->link[p] = new_r_Phi(current_ir_graph, get_nodes_block(phi), phi_preds, in, env->modes[p]);
- set_optimize(rem);
+ set_irn_link(value_arr[i], env->fix_phis);
+ env->fix_phis = l;
+ }
}
}
/**
- * Handle Loads, that were been scalar replaced or
- * will be scalar replaced.
- *
- * @param load A load node.
- * @param env Contains information about scalars number and mode.
- * @param repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
+ * Walker: allocate the value array for every block.
*/
-static void load_handling(ir_node *load, env_t *env, set *repairs)
+static void alloc_value_arr(ir_node *block, void *ctx)
{
- repairs_t key, *value;
- ir_node *load_ptr, *load_mem, *nods_block;
- unsigned i;
+ env_t *env = ctx;
+ ir_node **var_arr = obstack_alloc(&env->obst, sizeof(*var_arr) * env->nvals);
- nods_block = get_nodes_block(load);
- load_ptr = get_Load_ptr(load);
- load_mem = get_Load_mem(load);
- // DDMN(load);
- /* The pointer predecessor of Load muss be a Sel node. */
- if (get_irn_op(load_ptr) == op_Sel) {
- /* If the link field of sel's entity is set to "ADDRESS_TAKEN", that means this value
- can't be scalar replaced. It is nothing to do and load_handling() must be exited. */
- if (get_entity_link(get_Sel_entity(load_ptr)) == ADDRESS_TAKEN)
- return;
-
- key.irn = nods_block;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(nods_block));
-
- /* Load's pointer predecessor's link field contains the position in the block's link, where
- must be searched the predecessor of this Load. */
- i = (unsigned)get_irn_link(load_ptr);
-
- /* If the link of Load's block doesn't contains at position "i" a node or isn't calculated,
- than pred_search() must be called .*/
- if (value == NULL) {
- block_link(load, env, repairs);
- key.irn = nods_block;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(nods_block));
- pred_search(load, value, repairs, i, 0);
- } else if (value->link[i] == NULL)
- pred_search(load, value, repairs, i, 0);
-
- /* If after the pred_search() call the link of Load's block at position "i" is equal to NULL,
- that means the loaded value wasn't initialized and the load predecessor is set to Unknown */
- if (value->link[i] == NULL)
- value->link[i] = new_Unknown(env->modes[i]);
-
- /* The Load node can be turned into a tuple now. This tuple will be optimized later. */
- turn_into_tuple(load, pn_Load_max);
- set_Tuple_pred(load, pn_Load_M, load_mem);
- set_Tuple_pred(load, pn_Load_res, value->link[i]);
- set_Tuple_pred(load, pn_Load_X_except, new_Bad());
- }
+ /* the value array is empty at start */
+ memset(var_arr, 0, sizeof(*var_arr) * env->nvals);
+ set_irn_link(block, var_arr);
}
/**
- * A walker along the memory edge. Load and Phi nodes muss be found for optimization
- *
- * @param node A node from the graph.
- * @param env Contains information about scalars number and mode.
- * @param repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
-*/
-static void memory_edge_walk2(ir_node *node, env_t *env, set *repairs)
+ * searches through blocks beginning from block for value
+ * vnum and return it.
+ */
+static ir_node *find_value(ir_node *block, unsigned vnum)
{
- int i, p, n = get_irn_arity(node);
- repairs_t key, *value, *value_block;
- ir_node *phi_pred;
- DDMN(node);
-
- for (i = 0; i < n; i++) {
- ir_node *pred = get_irn_n(node, i);
-
- if((get_irn_op(pred) == op_Proj &&
- get_irn_mode(pred) == mode_M) ||
- get_irn_mode(pred) == mode_T ||
- is_memop(pred) ||
- get_irn_op(pred) == op_Call ||
- get_irn_op(pred) == op_Alloc)
- memory_edge_walk2(pred, env, repairs);
-
- if(get_irn_op(pred) == op_Phi &&
- get_irn_mode(pred) == mode_M &&
- get_irn_link(pred) != NODE_VISITED){
- set_irn_link(pred, NODE_VISITED);
- memory_edge_walk2(pred, env, repairs);
- }
- }
+ ir_node **value_arr;
+ int i;
+ ir_node *res;
- if (get_irn_op(node) == op_Load)
- load_handling(node, env, repairs);
-
- if (get_irn_op(node) == op_Phi && get_irn_mode(node) == mode_M){
- key.irn = node;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- /* If the phi is in the set " repairs ", then must be handled.*/
- if (value != NULL){
- // DDMN(node);
- key.irn = get_nodes_block(node);
- value_block = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- n = get_irn_arity(node);
- /* All predecessors of a Phi node muss be found.*/
- for(i = 0; i < env->nvals; i ++)
- for(p = 0; p < n; p ++){
- phi_pred = get_Phi_pred(value->irn, p);
- pred_search(phi_pred, value_block, repairs, i, p);
- }
- }
- }
- /* Reset the links, that have been used by the walk.*/
- if (get_irn_link(node) == NODE_VISITED)
- set_irn_link(node, NULL);
-}
+ if (Block_not_block_visited(block)) {
+ mark_Block_block_visited(block);
-/**
- * A walker along the memory edge in a loop.The walker walk from the node to the loop head.
- * Load and Phi nodes muss be found for optimisation
- *
- * @param *node A node from the graph.
- * @param *env Contains information about scalars number and mode.
- * @param *repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
-*/
-static void loop_walk(ir_node *node, env_t *env, set *repairs)
-{
- int i, p, n = get_irn_arity(node);
- repairs_t key, *value, *value_block;
- ir_node *phi_pred;
- DDMN(node);
+ value_arr = get_irn_link(block);
- /* Test if the loop head have been achieved. */
- if (has_backedges(get_nodes_block(node)))
- return;
+ if (value_arr[vnum])
+ return value_arr[vnum];
- for (i = 0; i < n; i++) {
- ir_node *pred = get_irn_n(node, i);
-
- if((get_irn_op(pred) == op_Proj &&
- get_irn_mode(pred) == mode_M) ||
- get_irn_mode(pred) == mode_T ||
- is_memop(pred) ||
- get_irn_op(pred) == op_Call ||
- get_irn_op(pred) == op_Alloc)
- loop_walk(pred, env, repairs);
-
- if(get_irn_op(pred) == op_Phi &&
- get_irn_mode(pred) == mode_M &&
- get_irn_link(pred) != LOOP_WALK) {
- set_irn_link(pred, LOOP_WALK);
- loop_walk(pred, env, repairs);
- }
- }
+ for (i = get_Block_n_cfgpreds(block) - 1; i >= 0; --i) {
+ ir_node *pred = get_Block_cfgpred(block, i);
- if (get_irn_op(node) == op_Load)
- load_handling(node, env, repairs);
-
- if (get_irn_op(node) == op_Phi && get_irn_mode(node) == mode_M){
- key.irn = node;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- /* If the phi is in the set " repairs ", then muss be handled.*/
- if(value != NULL){
- DDMN(node);
- key.irn = get_nodes_block(node);
- value_block = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- n = get_irn_arity(node);
- /* All predecessors of a Phi node muss be found.*/
- for(i = 0; i < env->nvals; i ++)
- for(p = 0; p < n; p ++){
- phi_pred = get_Phi_pred(value->irn, p);
- pred_search(phi_pred, value_block, repairs, i, p);
- }
+ res = find_value(get_nodes_block(pred), vnum);
+ if (res)
+ return res;
}
}
- /* Reset the links, that have beeb used by the walk.*/
- if(get_irn_link(node) == LOOP_WALK)
- set_irn_link(node, NULL);
+ return NULL;
}
/**
- * Handle Stores that were been scalar replaced or
- * will be scalar replaced.
- *
- * @param load A store node.
- * @param env Contains information about scalars number and mode.
- * @param repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
+ * fix the Phi list
*/
-static void store_handling(ir_node *store, env_t *env, set *repairs)
+static void fix_phis(env_t *env)
{
- repairs_t key, *value;
- ir_node *nods_block, *store_mem, *store_ptr, *store_value, *phi;
- ir_loop *store_l;
-
- store_ptr = get_Store_ptr(store);
-
- /* The pointer predecessor of Store must be a Sel node.*/
- if (get_irn_op(store_ptr) == op_Sel) {
- /* If the link field of a Store's is set to "ADDRESS_TAKEN", that mean this value
- can't be scalar replaced. It's nothing to do and store_handling() must be exit.*/
- if (get_entity_link(get_Sel_entity(store_ptr)) == ADDRESS_TAKEN)
- return;
-
- /* If the Store node is in a loop, than the loop head of the Store
- * must be handled, if this was not already done. */
- store_l = get_irn_loop(store);
- if (store_l != NULL) {
- phi = get_loop_node(store_l, 0);
- if (get_irn_op(phi) != op_Block) {
- key.irn = get_nodes_block(phi);
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(key.irn));
- if (value == NULL) {
- set_irn_link(phi, LOOP_HEAD_PHI);
- block_link(phi, env, repairs);
- phi_handling(phi, env, repairs);
- }
- }
- }
-
- DDMN(store);
- nods_block = get_nodes_block(store);
+ list_entry_t *l;
+ ir_node *phi, *block, *pred, *val;
+ int i;
- key.irn = nods_block;
- value = set_find(repairs, &key, sizeof(key), HASH_PTR(nods_block));
+ for (l = env->fix_phis; l; l = get_irn_link(phi)) {
+ phi = l->node;
- store_mem = get_Store_mem(store);
- store_value = get_Store_value(store);
+ block = get_nodes_block(phi);
+ for (i = get_irn_arity(phi) - 1; i >= 0; --i) {
+ pred = get_Block_cfgpred(block, i);
+ pred = get_nodes_block(pred);
- if (store_l != NULL)
- loop_walk(store, env, repairs);
- else
- memory_edge_walk2(store, env, repairs);
- /* Store's pointer predecessor's link field contains the position in the block's link, where
- must be saved the value predecessor of this store. */
- value->link[(unsigned)get_irn_link(store_ptr)] = store_value;
+ inc_irg_block_visited(current_ir_graph);
+ val = find_value(pred, l->vnum);
- /* The store node can be turned into a tuple now. This tuple will be optimized later. */
- turn_into_tuple(store, pn_Store_max);
- set_Tuple_pred(store, pn_Store_M, store_mem);
- set_Tuple_pred(store, pn_Store_X_except, new_Bad());
+ if (val)
+ set_irn_n(phi, i, val);
+ }
}
}
/**
- * A walker along the memory edge. Stores, Phis and their blocks as well Load's blocks must be found for optimization
- *
- * @param node A node from the graph.
- * @param env Contains information about scalars number and mode.
- * @param repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
-*/
-static void memory_edge_walk(ir_node *node, env_t *env, set *repairs)
+ * fix the Load list
+ */
+static void fix_loads(env_t *env)
{
- int i, n = get_irn_arity(node);
- ir_op *op;
-
- DDMN(node);
-
- set_irn_link(get_nodes_block(node), NULL);
-
- for (i = n - 1; i >= 0; --i) {
- ir_node *pred = get_irn_n(node, i);
- ir_op *op = get_irn_op(pred);
-
- if ((op == op_Proj &&
- get_irn_mode(pred) == mode_M) ||
- op == op_Load ||
- op == op_Store ||
- op == op_Call ||
- op == op_Alloc)
- memory_edge_walk(pred, env, repairs);
-
- if (op == op_Phi &&
- get_irn_mode(pred) == mode_M &&
- get_irn_link(pred) != NODE_VISITED) {
- set_irn_link(pred, NODE_VISITED);
- memory_edge_walk(pred, env, repairs);
- }
- }
+ list_entry_t *l;
+ ir_node *load, *block, *pred, *val, *mem;
+ int i;
+
+ for (l = env->fix_loads; l; l = get_irn_link(load)) {
+ load = l->node;
+
+ block = get_nodes_block(load);
+ for (i = get_Block_n_cfgpreds(block) - 1; i >= 0; --i) {
+ pred = get_Block_cfgpred(block, i);
+ pred = get_nodes_block(pred);
- op = get_irn_op(node);
- if (op == op_Store || op == op_Load)
- block_link(node, env, repairs);
+ inc_irg_block_visited(current_ir_graph);
+ val = find_value(pred, l->vnum);
- if (op == op_Phi && get_irn_mode(node) == mode_M)
- phi_handling(node, env, repairs);
+ if (val)
+ break;
+ }
+
+ if (! val) {
+ /* access of an uninitialized value */
+ val = new_Unknown(env->modes[l->vnum]);
+ }
- if (op == op_Store)
- store_handling(node, env, repairs);
+ mem = get_Load_mem(load);
- /* Reset the links, that have been used by the walk.*/
- if (get_irn_link(node) == NODE_VISITED)
- set_irn_link(node, NULL);
+ turn_into_tuple(load, pn_Load_max);
+ set_Tuple_pred(load, pn_Load_M, mem);
+ set_Tuple_pred(load, pn_Load_res, val);
+ set_Tuple_pred(load, pn_Load_X_except, new_Bad());
+ }
}
/**
* Make scalar replacement.
*
- * @param envals The number of scalars.
- * @param repairs A set, that contains all blocks, that have a link, and all Phis, that
- * have copies to repair.
+ * @param sels A set containing all Sel nodes that have a value number
+ * @param nvals The number of scalars.
* @param modes A flexible array, containing all the modes of
* the value numbers.
*/
-static void do_scalar_replacements(int nvals, set * repairs, ir_mode **modes)
+static void do_scalar_replacements(pset *sels, int nvals, ir_mode **modes)
{
- ir_node *end_block = get_irg_end_block(current_ir_graph);
- int i;
- int n = get_Block_n_cfgpreds(end_block);
env_t env;
obstack_init(&env.obst);
- env.nvals = nvals;
- env.modes = modes;
+ env.nvals = nvals;
+ env.modes = modes;
+ env.fix_phis = NULL;
+ env.fix_loads = NULL;
+ env.sels = sels;
+
+ /* first step: allocate the value arrays for every block */
+ irg_block_walk_graph(current_ir_graph, NULL, alloc_value_arr, &env);
+
+ /*
+ * second step: walk over the graph blockwise in topological order
+ * and fill the array as much as possible.
+ */
+ irg_walk_blkwise_graph(current_ir_graph, NULL, handle_first, &env);
- /* Search for scalars.*/
- for (i = 0; i < n; ++i) {
- ir_node *pred = get_Block_cfgpred(end_block, i);
- if (get_irn_op(pred) != op_Block)
- memory_edge_walk(pred, &env, repairs);
- }
+ /* third, fix the list of Phis, then the list of Loads */
+ fix_phis(&env);
+ fix_loads(&env);
- /* Search for predecessors of scalars.*/
- for (i = 0; i < n; ++i) {
- ir_node *pred = get_Block_cfgpred(end_block, i);
- if (get_irn_op(pred) != op_Block)
- memory_edge_walk2(pred, &env, repairs);
- }
obstack_free(&env.obst, NULL);
}
*
* @param irg The current ir graph.
*/
-void find_scalar_replacements(ir_graph *irg)
+void scalar_replacement_opt(ir_graph *irg)
{
- unsigned nvals;
- int i;
+ unsigned nvals;
+ int i;
scalars_t key, *value;
- ir_node *irg_frame;
- ir_mode **modes;
- set *set_ent, *repairs;
- type *ent_type;
-
- /* Call algorithm that computes the out edges */
- compute_irg_outs(irg);
-
- /* Call algorithm that computes the loop information */
- construct_backedges(irg);
+ ir_node *irg_frame;
+ ir_mode **modes;
+ set *set_ent;
+ pset *sels;
+ type *ent_type;
+ ir_graph *rem;
+
+ if (! get_opt_scalar_replacement())
+ return;
- /* Call algorithm that computes the backedges */
- construct_cf_backedges(irg);
+ rem = current_ir_graph;
- /* Call algorithm that computes the dominator trees. */
- compute_doms(irg);
+ /* Call algorithm that computes the out edges */
+ if (get_irg_outs_state(irg) != outs_consistent)
+ compute_irg_outs(irg);
/* Find possible scalar replacements */
- do_find_scalar_replacements(irg);
-
- printf("Called for %s\n", get_entity_name(get_irg_entity(irg)));
+ if (find_possible_replacements(irg)) {
- /* Insert in set the scalar replacemnts. */
- irg_frame = get_irg_frame(irg);
- nvals = 0;
- modes = NEW_ARR_F(ir_mode *, 16);
- set_ent = new_set(ent_cmp, 8);
-
- for (i = 0 ; i < get_irn_n_outs(irg_frame); i++) {
- ir_node *succ = get_irn_out(irg_frame, i);
+ if (get_opt_scalar_replacement_verbose()) {
+ printf("Scalar Replacement: %s\n", get_entity_name(get_irg_entity(irg)));
+ }
- if (get_irn_op(succ) == op_Sel) {
- entity *ent = get_Sel_entity(succ);
+ /* Insert in set the scalar replacements. */
+ irg_frame = get_irg_frame(irg);
+ nvals = 0;
+ modes = NEW_ARR_F(ir_mode *, 16);
+ set_ent = new_set(ent_cmp, 8);
+ sels = pset_new_ptr(8);
+
+ for (i = 0 ; i < get_irn_n_outs(irg_frame); i++) {
+ ir_node *succ = get_irn_out(irg_frame, i);
+
+ if (get_irn_op(succ) == op_Sel) {
+ entity *ent = get_Sel_entity(succ);
+
+ if (get_entity_link(ent) == NULL || get_entity_link(ent) == ADDRESS_TAKEN)
+ continue;
+
+ ent_type = get_entity_type(ent);
+
+ key.ent = ent;
+ key.ent_owner = get_entity_owner(ent);
+ set_insert(set_ent, &key, sizeof(key), HASH_PTR(key.ent));
+
+ if (get_opt_scalar_replacement_verbose()) {
+ if (is_Array_type(ent_type)) {
+ printf(" found array %s\n", get_entity_name(ent));
+ }
+ else if (is_Struct_type(ent_type)) {
+ printf(" found struct %s\n", get_entity_name(ent));
+ }
+ else if (is_atomic_type(ent_type))
+ printf(" found atomic value %s\n", get_entity_name(ent));
+ else {
+ assert(0 && "Neither an array nor a struct or atomic value");
+ }
+ }
- if (get_entity_link(ent) == NULL || get_entity_link(ent) == ADDRESS_TAKEN)
- continue;
+ nvals = allocate_value_numbers(sels, ent, nvals, &modes);
+ }
+ }
- ent_type = get_entity_type(ent);
+ if (get_opt_scalar_replacement_verbose()) {
+ printf(" %u values will be needed\n", nvals);
+ }
- key.ent = ent;
- key.ent_owner = get_entity_owner (ent);
- set_insert(set_ent, &key, sizeof(key), HASH_PTR(key.ent));
+ /* If scalars were found. */
+ if (nvals) {
+ do_scalar_replacements(sels, nvals, modes);
- if (is_Array_type(ent_type)) {
- printf("<<<<<<<found array %s\n", get_entity_name(ent));
- }
- else if (is_Struct_type(ent_type)) {
- printf("<<<<<<<found struct %s\n", get_entity_name(ent));
- }
- else if(is_atomic_type(ent_type))
- printf("<<<<<<<found atomic value %s\n", get_entity_name(ent));
- else {
- assert(0 && "Neither an array nor a struct or atomic value");
+ for (value = set_first(set_ent); value; value = set_next(set_ent)) {
+ remove_class_member(value->ent_owner, value->ent);
}
+ }
- nvals = allocate_value_numbers(ent, nvals, &modes);
+ del_pset(sels);
+ del_set(set_ent);
+ DEL_ARR_F(modes);
+
+ if (nvals) {
+ /*
+ * We changed the graph, but did NOT introduce new blocks
+ * neither changed control flow, cf-backedges should be still
+ * consistent.
+ */
+ set_irg_outs_inconsistent(irg);
+ set_irg_loopinfo_inconsistent(irg);
}
}
- printf(" %u values will be needed\n", nvals);
-
- /* A set of nodes that must be repaired. These are Phi or Unknown nodes. */
- repairs = new_set(set_cmp, 8);
-
- /* If scalars ware found. */
- if (nvals)
- do_scalar_replacements(nvals, repairs, modes);
-
- value = set_first(set_ent);
- for (; value != NULL; value = set_next(set_ent))
- remove_class_member(value->ent_owner, value->ent);
-
- del_set(set_ent);
- del_set(repairs);
- DEL_ARR_F(modes);
+ current_ir_graph = rem;
}