Commit 93b64caf authored by Michael Beck's avatar Michael Beck
Browse files

Improved implementation:

 - removed commented out code
 - removed some unnecessary memory copies
 - used new functions
 - use unknown_entity instead of "MARK"
 - fixed some comments
 - improve doxygen comments

[r17122]
parent df81b97b
/* /*
* Copyright (C) 1995-2007 University of Karlsruhe. All right reserved. * Copyright (C) 1995-2008 University of Karlsruhe. All right reserved.
* *
* This file is part of libFirm. * This file is part of libFirm.
* *
...@@ -58,12 +58,12 @@ ...@@ -58,12 +58,12 @@
* Stores the result in the field 'callees' of the Call node. If the * Stores the result in the field 'callees' of the Call node. If the
* address can not be analysed, e.g. because it is loaded from a * address can not be analysed, e.g. because it is loaded from a
* variable, the array contains the unknown_entity. (See * variable, the array contains the unknown_entity. (See
* "set_Call_callee"). cgana returns the set of 'free' methods, i.e., * set_Call_callee()). cgana() returns the set of 'free' methods, i.e.,
* the methods that can be called from external or via function * the methods that can be called from external or via function
* pointers. This datastructure must be freed with 'free()' by the * pointers. This datastructure must be freed with 'xfree()' by the
* caller of cgana. * caller of cgana().
* *
* cgana sets the callee_info_state of each graph and the program to * cgana() sets the callee_info_state of each graph and the program to
* consistent. * consistent.
* *
* The algorithm implements roughly Static Class Hierarchy Analysis * The algorithm implements roughly Static Class Hierarchy Analysis
...@@ -74,13 +74,7 @@ ...@@ -74,13 +74,7 @@
* Performs some optimizations possible by the analysed information: * Performs some optimizations possible by the analysed information:
* - Replace SymConst-name nodes by SymConst-entity nodes if possible. * - Replace SymConst-name nodes by SymConst-entity nodes if possible.
* - Replace (Sel-method(Alloc)) by SymConst-entity. * - Replace (Sel-method(Alloc)) by SymConst-entity.
* - Replaces Sel nodes by Bad if there is no implementation for the
* selected entity.
* - Replaces Sel-method by SymConst-entity if the method is never overwritten. * - Replaces Sel-method by SymConst-entity if the method is never overwritten.
* - Replaces Calls by Tuple containing Bads if callee array is empty
* (there is no implementation to call)
*
* Leaves Bad control predecessors in the graph!
*/ */
void cgana(int *len, ir_entity ***free_methods); void cgana(int *len, ir_entity ***free_methods);
...@@ -97,5 +91,4 @@ void free_irp_callee_info(void); ...@@ -97,5 +91,4 @@ void free_irp_callee_info(void);
* Performs only the optimizations done by cgana. */ * Performs only the optimizations done by cgana. */
void opt_call_addrs(void); void opt_call_addrs(void);
#endif #endif
/* /*
* Copyright (C) 1995-2007 University of Karlsruhe. All right reserved. * Copyright (C) 1995-2008 University of Karlsruhe. All right reserved.
* *
* This file is part of libFirm. * This file is part of libFirm.
* *
...@@ -183,7 +183,7 @@ static ir_entity ** get_impl_methods(ir_entity * method) { ...@@ -183,7 +183,7 @@ static ir_entity ** get_impl_methods(ir_entity * method) {
* @param node The node to analyze * @param node The node to analyze
* @param env A map that maps names of entities to the entities. * @param env A map that maps names of entities to the entities.
*/ */
static void sel_methods_walker(ir_node * node, void *env) { static void sel_methods_walker(ir_node *node, void *env) {
pmap *ldname_map = env; pmap *ldname_map = env;
ir_entity **arr; ir_entity **arr;
...@@ -195,18 +195,16 @@ static void sel_methods_walker(ir_node * node, void *env) { ...@@ -195,18 +195,16 @@ static void sel_methods_walker(ir_node * node, void *env) {
} }
/* replace SymConst(name)-operations by SymConst(ent) */ /* replace SymConst(name)-operations by SymConst(ent) */
if (get_irn_op(node) == op_SymConst) { if (is_SymConst(node)) {
if (get_SymConst_kind(node) == symconst_addr_name) { if (get_SymConst_kind(node) == symconst_addr_name) {
pmap_entry * entry = pmap_find(ldname_map, (void *) get_SymConst_name(node)); pmap_entry *entry = pmap_find(ldname_map, get_SymConst_name(node));
if (entry != NULL) { /* Method is declared in the compiled code */ if (entry != NULL) { /* Method is declared in the compiled code */
assert(0 && "There should not be a SymConst[addr_name] addressing a method with an implementation" assert(!"There should not be a SymConst[addr_name] addressing a method with an implementation"
"in this compilation unit. Use a SymConst[addr_ent]."); "in this compilation unit. Use a SymConst[addr_ent].");
} }
} }
} } else if (is_Sel(node) && is_Method_type(get_entity_type(get_Sel_entity(node)))) {
else if (get_irn_op(node) == op_Sel && ir_entity *ent = get_SymConst_entity(get_atomic_ent_value(get_Sel_entity(node)));
is_Method_type(get_entity_type(get_Sel_entity(node)))) {
ir_entity * ent = get_SymConst_entity(get_atomic_ent_value(get_Sel_entity(node)));
assert(get_entity_peculiarity(ent) != peculiarity_inherited); assert(get_entity_peculiarity(ent) != peculiarity_inherited);
if (!eset_contains(entities, ent)) { if (!eset_contains(entities, ent)) {
...@@ -226,7 +224,7 @@ static void sel_methods_walker(ir_node * node, void *env) { ...@@ -226,7 +224,7 @@ static void sel_methods_walker(ir_node * node, void *env) {
* We could not call it, but it may be description: * We could not call it, but it may be description:
* We call a method in a dead part of the program. * We call a method in a dead part of the program.
*/ */
assert (get_entity_peculiarity(ent) == peculiarity_description); assert(get_entity_peculiarity(ent) == peculiarity_description);
} }
else if (get_opt_closed_world() && get_opt_dyn_meth_dispatch() && else if (get_opt_closed_world() && get_opt_dyn_meth_dispatch() &&
(ARR_LEN(arr) == 1 && arr[0] != NULL)) { (ARR_LEN(arr) == 1 && arr[0] != NULL)) {
...@@ -247,7 +245,8 @@ static void sel_methods_walker(ir_node * node, void *env) { ...@@ -247,7 +245,8 @@ static void sel_methods_walker(ir_node * node, void *env) {
} }
} }
/** Initialize auxiliary data structures. /**
* Initialize auxiliary data structures.
* *
* Computes a set of entities that overwrite an entity and contain * Computes a set of entities that overwrite an entity and contain
* an implementation. The set is stored in the entity's link field. * an implementation. The set is stored in the entity's link field.
...@@ -255,10 +254,11 @@ static void sel_methods_walker(ir_node * node, void *env) { ...@@ -255,10 +254,11 @@ static void sel_methods_walker(ir_node * node, void *env) {
* Further replaces Sel nodes where this set contains exactly one * Further replaces Sel nodes where this set contains exactly one
* method by SymConst nodes. * method by SymConst nodes.
* Finally asserts if there is a SymConst(name) if there could be a * Finally asserts if there is a SymConst(name) if there could be a
* SymConst(ent). */ * SymConst(ent).
*/
static void sel_methods_init(void) { static void sel_methods_init(void) {
int i; int i;
pmap * ldname_map = pmap_create(); /* Map entity names to entities: to replace pmap *ldname_map = pmap_create(); /* Map entity names to entities: to replace
SymConst(name) by SymConst(ent). */ SymConst(name) by SymConst(ent). */
assert(entities == NULL); assert(entities == NULL);
entities = eset_create(); entities = eset_create();
...@@ -266,7 +266,7 @@ static void sel_methods_init(void) { ...@@ -266,7 +266,7 @@ static void sel_methods_init(void) {
ir_entity * ent = get_irg_entity(get_irp_irg(i)); ir_entity * ent = get_irg_entity(get_irp_irg(i));
/* only external visible methods are allowed to call by a SymConst_ptr_name */ /* only external visible methods are allowed to call by a SymConst_ptr_name */
if (get_entity_visibility(ent) != visibility_local) { if (get_entity_visibility(ent) != visibility_local) {
pmap_insert(ldname_map, (void *) get_entity_ld_ident(ent), ent); pmap_insert(ldname_map, (void *)get_entity_ld_ident(ent), ent);
} }
} }
...@@ -328,6 +328,7 @@ static ir_entity * get_Sel_method(ir_node * sel, int pos) { ...@@ -328,6 +328,7 @@ static ir_entity * get_Sel_method(ir_node * sel, int pos) {
return arr[pos]; return arr[pos];
} }
/* forward */
static void free_mark(ir_node * node, eset * set); static void free_mark(ir_node * node, eset * set);
static void free_mark_proj(ir_node * node, long n, eset * set) { static void free_mark_proj(ir_node * node, long n, eset * set) {
...@@ -394,8 +395,8 @@ static void free_mark(ir_node *node, eset * set) { ...@@ -394,8 +395,8 @@ static void free_mark(ir_node *node, eset * set) {
switch (get_irn_opcode(node)) { switch (get_irn_opcode(node)) {
case iro_Sel: { case iro_Sel: {
ir_entity * ent = get_Sel_entity(node); ir_entity *ent = get_Sel_entity(node);
if (is_Method_type(get_entity_type(ent))) { if (is_method_entity(ent)) {
for (i = get_Sel_n_methods(node) - 1; i >= 0; --i) { for (i = get_Sel_n_methods(node) - 1; i >= 0; --i) {
eset_insert(set, get_Sel_method(node, i)); eset_insert(set, get_Sel_method(node, i));
} }
...@@ -404,8 +405,8 @@ static void free_mark(ir_node *node, eset * set) { ...@@ -404,8 +405,8 @@ static void free_mark(ir_node *node, eset * set) {
} }
case iro_SymConst: case iro_SymConst:
if (get_SymConst_kind(node) == symconst_addr_ent) { if (get_SymConst_kind(node) == symconst_addr_ent) {
ir_entity * ent = get_SymConst_entity(node); ir_entity *ent = get_SymConst_entity(node);
if (is_Method_type(get_entity_type(ent))) { if (is_method_entity(ent)) {
eset_insert(set, ent); eset_insert(set, ent);
} }
} else { } else {
...@@ -419,14 +420,11 @@ static void free_mark(ir_node *node, eset * set) { ...@@ -419,14 +420,11 @@ static void free_mark(ir_node *node, eset * set) {
free_mark(get_Phi_pred(node, i), set); free_mark(get_Phi_pred(node, i), set);
} }
break; break;
case iro_Id:
free_mark(get_Id_pred(node), set);
break;
case iro_Proj: case iro_Proj:
free_mark_proj(get_Proj_pred(node), get_Proj_proj(node), set); free_mark_proj(get_Proj_pred(node), get_Proj_proj(node), set);
break; break;
default: default:
/* nothing: Wird unten behandelt! */ /* nothing: */
break; break;
} }
set_irn_link(node, NULL); set_irn_link(node, NULL);
...@@ -454,9 +452,9 @@ static void free_ana_walker(ir_node *node, void *env) { ...@@ -454,9 +452,9 @@ static void free_ana_walker(ir_node *node, void *env) {
case iro_Tuple: case iro_Tuple:
/* nothing */ /* nothing */
break; break;
/* Sonderbehandlung, da der Funktionszeigereingang natürlich kein
* Verräter ist. */
case iro_Call: case iro_Call:
/* we must handle Call nodes specially, because their call address input
do not expose a method address. */
set_irn_link(node, MARK); set_irn_link(node, MARK);
for (i = get_Call_n_params(node) - 1; i >= 0; --i) { for (i = get_Call_n_params(node) - 1; i >= 0; --i) {
ir_node *pred = get_Call_param(node, i); ir_node *pred = get_Call_param(node, i);
...@@ -465,12 +463,12 @@ static void free_ana_walker(ir_node *node, void *env) { ...@@ -465,12 +463,12 @@ static void free_ana_walker(ir_node *node, void *env) {
} }
} }
break; break;
default:
/* other nodes: Alle anderen Knoten nehmen wir als Verräter an, bis /* other nodes: Alle anderen Knoten nehmen wir als Verräter an, bis
* jemand das Gegenteil implementiert. */ * jemand das Gegenteil implementiert. */
default:
set_irn_link(node, MARK); set_irn_link(node, MARK);
for (i = get_irn_arity(node) - 1; i >= 0; --i) { for (i = get_irn_arity(node) - 1; i >= 0; --i) {
ir_node * pred = get_irn_n(node, i); ir_node *pred = get_irn_n(node, i);
if (mode_is_reference(get_irn_mode(pred))) { if (mode_is_reference(get_irn_mode(pred))) {
free_mark(pred, set); free_mark(pred, set);
} }
...@@ -541,14 +539,14 @@ static void add_method_address(ir_entity *ent, eset *set) ...@@ -541,14 +539,14 @@ static void add_method_address(ir_entity *ent, eset *set)
* umgewandelt worden sein, d.h. SymConst-Operationen verweisen immer * umgewandelt worden sein, d.h. SymConst-Operationen verweisen immer
* auf eine echt externe Methode. * auf eine echt externe Methode.
*/ */
static ir_entity ** get_free_methods(void) static ir_entity **get_free_methods(int *length)
{ {
eset *free_set = eset_create(); eset *free_set = eset_create();
int i; int i;
ir_entity **arr = NEW_ARR_F(ir_entity *, 0); ir_entity **arr;
ir_entity *ent; ir_entity *ent;
ir_graph *irg; ir_graph *irg;
ir_type *glob; ir_type *tp;
for (i = get_irp_n_irgs() - 1; i >= 0; --i) { for (i = get_irp_n_irgs() - 1; i >= 0; --i) {
irg = get_irp_irg(i); irg = get_irp_irg(i);
...@@ -557,36 +555,38 @@ static ir_entity ** get_free_methods(void) ...@@ -557,36 +555,38 @@ static ir_entity ** get_free_methods(void)
if (get_entity_visibility(ent) != visibility_local) { if (get_entity_visibility(ent) != visibility_local) {
eset_insert(free_set, ent); eset_insert(free_set, ent);
} }
/* Finde alle Methoden die in dieser Methode extern sichtbar werden,
z.B. da die Adresse einer Methode abgespeichert wird. */
irg_walk_graph(irg, NULL, free_ana_walker, free_set);
}
/* insert sticky methods, too */
for (i = get_irp_n_irgs() - 1; i >= 0; --i) {
ent = get_irg_entity(get_irp_irg(i));
/* insert "sticky" methods. */ /* insert "sticky" methods. */
if (get_entity_stickyness (ent) == stickyness_sticky) { if (get_entity_stickyness(ent) == stickyness_sticky) {
eset_insert(free_set, ent); eset_insert(free_set, ent);
} }
/* Find all method entities that gets "visible" trough this graphs,
* for instance because their address is stored. */
irg_walk_graph(irg, NULL, free_ana_walker, free_set);
} }
/* insert all methods that are used in global variables initializers */ /* insert all methods that are used in global variables initializers */
glob = get_glob_type(); tp = get_glob_type();
for (i = get_class_n_members(glob) - 1; i >= 0; --i) { for (i = get_class_n_members(tp) - 1; i >= 0; --i) {
ent = get_class_member(glob, i); ent = get_class_member(tp, i);
add_method_address(ent, free_set);
}
tp = get_tls_type();
for (i = get_struct_n_members(tp) - 1; i >= 0; --i) {
ent = get_struct_member(tp, i);
add_method_address(ent, free_set); add_method_address(ent, free_set);
} }
/* the main program is even then "free", if it's not external visible. */ /* the main program is even then "free", if it's not external visible. */
irg = get_irp_main_irg(); irg = get_irp_main_irg();
if (irg) if (irg != NULL)
eset_insert(free_set, get_irg_entity(irg)); eset_insert(free_set, get_irg_entity(irg));
/* Finally, transform the set into an array. */ /* Finally, transform the set into an array. */
for (ent = eset_first(free_set); ent; ent = eset_next(free_set)) { *length = eset_count(free_set);
ARR_APP1(ir_entity *, arr, ent); arr = xmalloc(sizeof(ir_entity *) * (*length));
for (i = 0, ent = eset_first(free_set); ent; ent = eset_next(free_set)) {
arr[i++] = ent;
} }
eset_destroy(free_set); eset_destroy(free_set);
...@@ -599,7 +599,7 @@ static ir_entity ** get_free_methods(void) ...@@ -599,7 +599,7 @@ static ir_entity ** get_free_methods(void)
static void callee_ana_node(ir_node * node, eset * methods); static void callee_ana_node(ir_node * node, eset * methods);
static void callee_ana_proj(ir_node * node, long n, eset * methods) { static void callee_ana_proj(ir_node *node, long n, eset *methods) {
assert(get_irn_mode(node) == mode_T); assert(get_irn_mode(node) == mode_T);
if (get_irn_link(node) == MARK) { if (get_irn_link(node) == MARK) {
/* already visited */ /* already visited */
...@@ -612,12 +612,12 @@ static void callee_ana_proj(ir_node * node, long n, eset * methods) { ...@@ -612,12 +612,12 @@ static void callee_ana_proj(ir_node * node, long n, eset * methods) {
/* proj_proj: in einem "sinnvollen" Graphen kommt jetzt ein /* proj_proj: in einem "sinnvollen" Graphen kommt jetzt ein
* op_Tuple oder ein Knoten, der eine "freie Methode" * op_Tuple oder ein Knoten, der eine "freie Methode"
* zurückgibt. */ * zurückgibt. */
ir_node * pred = get_Proj_pred(node); ir_node *pred = get_Proj_pred(node);
if (get_irn_link(pred) != MARK) { if (get_irn_link(pred) != MARK) {
if (get_irn_op(pred) == op_Tuple) { if (is_Tuple(pred)) {
callee_ana_proj(get_Tuple_pred(pred, get_Proj_proj(node)), n, methods); callee_ana_proj(get_Tuple_pred(pred, get_Proj_proj(node)), n, methods);
} else { } else {
eset_insert(methods, MARK); /* free method -> unknown */ eset_insert(methods, unknown_entity); /* free method -> unknown */
} }
} }
break; break;
...@@ -627,12 +627,8 @@ static void callee_ana_proj(ir_node * node, long n, eset * methods) { ...@@ -627,12 +627,8 @@ static void callee_ana_proj(ir_node * node, long n, eset * methods) {
callee_ana_node(get_Tuple_pred(node, n), methods); callee_ana_node(get_Tuple_pred(node, n), methods);
break; break;
case iro_Id:
callee_ana_proj(get_Id_pred(node), n, methods);
break;
default: default:
eset_insert(methods, MARK); /* free method -> unknown */ eset_insert(methods, unknown_entity); /* free method -> unknown */
break; break;
} }
...@@ -660,27 +656,27 @@ static void callee_ana_node(ir_node *node, eset *methods) { ...@@ -660,27 +656,27 @@ static void callee_ana_node(ir_node *node, eset *methods) {
case iro_Const: case iro_Const:
/* A direct address call. We tread this as an external /* A direct address call. We tread this as an external
call and ignore it completely. */ call and ignore it completely. */
eset_insert(methods, MARK); /* free method -> unknown */ eset_insert(methods, unknown_entity); /* free method -> unknown */
break; break;
case iro_SymConst: case iro_SymConst:
if (get_SymConst_kind(node) == symconst_addr_ent) { if (get_SymConst_kind(node) == symconst_addr_ent) {
ir_entity *ent = get_SymConst_entity(node); ir_entity *ent = get_SymConst_entity(node);
assert(ent && is_Method_type(get_entity_type(ent))); assert(ent && is_method_entity(ent));
eset_insert(methods, ent); eset_insert(methods, ent);
} else { } else {
assert(get_SymConst_kind(node) == symconst_addr_name); assert(get_SymConst_kind(node) == symconst_addr_name);
/* externe Methode (wegen fix_symconst!) */ /* external method (because fix_symconst()!) */
eset_insert(methods, MARK); /* free method -> unknown */ eset_insert(methods, unknown_entity); /* free method -> unknown */
} }
break; break;
case iro_Sel: case iro_Sel:
/* polymorphe Methode */ /* polymorphic method */
for (i = get_Sel_n_methods(node) - 1; i >= 0; --i) { for (i = get_Sel_n_methods(node) - 1; i >= 0; --i) {
ir_entity * ent = get_Sel_method(node, i); ir_entity *ent = get_Sel_method(node, i);
if (ent) { if (ent != NULL) {
eset_insert(methods, ent); eset_insert(methods, ent);
} else { } else {
eset_insert(methods, MARK); eset_insert(methods, unknown_entity);
} }
} }
break; break;
...@@ -719,7 +715,7 @@ static void callee_ana_node(ir_node *node, eset *methods) { ...@@ -719,7 +715,7 @@ static void callee_ana_node(ir_node *node, eset *methods) {
case iro_Sub: case iro_Sub:
case iro_Conv: case iro_Conv:
/* extern */ /* extern */
eset_insert(methods, MARK); /* free method -> unknown */ eset_insert(methods, unknown_entity); /* free method -> unknown */
break; break;
default: default:
...@@ -731,48 +727,29 @@ static void callee_ana_node(ir_node *node, eset *methods) { ...@@ -731,48 +727,29 @@ static void callee_ana_node(ir_node *node, eset *methods) {
} }
/** /**
* Walker: Analyses every call node and calculates an array of possible * Walker: Analyses every Call node and calculates an array of possible
* callees for that call. * callees for that call.
*/ */
static void callee_walker(ir_node * call, void * env) { static void callee_walker(ir_node *call, void *env) {
(void) env; (void) env;
if (is_Call(call)) { if (is_Call(call)) {
eset * methods = eset_create(); eset *methods = eset_create();
ir_entity * ent; ir_entity *ent;
ir_entity ** arr = NEW_ARR_F(ir_entity *, 0); ir_entity **arr;
assert(get_irn_op(get_Call_ptr(call)) != op_Id); int i;
callee_ana_node(get_Call_ptr(call), methods); callee_ana_node(get_Call_ptr(call), methods);
if (eset_contains(methods, MARK)) { /* unknown method */ arr = NEW_ARR_F(ir_entity *, eset_count(methods));
ARR_APP1(ir_entity *, arr, unknown_entity); for (i = 0, ent = eset_first(methods); ent; ent = eset_next(methods)) {
} arr[i] = ent;
for (ent = eset_first(methods); ent; ent = eset_next(methods)) { /* we want the unknown_entity on the zero position for easy tests later */
if (ent != MARK) { if (ent == unknown_entity) {
ARR_APP1(ir_entity *, arr, ent); arr[i] = arr[0];
} arr[0] = unknown_entity;
} }
#if 0 /* This generates Bad nodes when we don't want it. ++i;
Call it with a check for valid cgana information in local_optimize. */
if (ARR_LEN(arr) == 0 && get_opt_optimize() && get_opt_closed_world() && get_opt_dyn_meth_dispatch()) {
/* Kann vorkommen, wenn der Vorgänger beispielsweise eine
* Sel-Operation war, die keine Methoden zurückgeben
* konnte. Wir ersetzen die Call-Operation ebenfalls durch
* eine Bad-Operation. Die Verlinkung muss wiederhergestellt
* werden! */
ir_node *mem = get_Call_mem(call);
ir_node *blk = get_nodes_block(call);
turn_into_tuple (call, pn_Call_max);
set_Tuple_pred(call, pn_Call_M_regular , mem);
set_Tuple_pred(call, pn_Call_T_result , new_Bad());
set_Tuple_pred(call, pn_Call_P_value_res_base, new_Bad());
set_Tuple_pred(call, pn_Call_X_regular , new_r_Jmp(current_ir_graph, blk));
set_Tuple_pred(call, pn_Call_X_except , new_Bad()); /* new_Jmp() ?? new_Raise() ?? */
set_Tuple_pred(call, pn_Call_M_except , new_Bad());
} else
#endif
{
set_Call_callee_arr(call, ARR_LEN(arr), arr);
} }
set_Call_callee_arr(call, ARR_LEN(arr), arr);
DEL_ARR_F(arr); DEL_ARR_F(arr);
eset_destroy(methods); eset_destroy(methods);
} }
...@@ -791,13 +768,14 @@ static void remove_Tuples(ir_node *proj, void *env) { ...@@ -791,13 +768,14 @@ static void remove_Tuples(ir_node *proj, void *env) {
} }
/* Bestimmt für jede Call-Operation die Menge der aufrufbaren Methode /**
* und speichert das Ergebnis in der Call-Operation. (siehe * Determine for every Call the set of possibly called methods and stores it
* "set_Call_callee"). "sel_methods" wird für den Aufbau benötigt und * inside the Call (@see set_Call_callee()).
* muss bereits aufgebaut sein. */ * Uses the sel_methods set with much be already calculated.
*/
static void callee_ana(void) { static void callee_ana(void) {
int i; int i;
/* Alle Graphen analysieren. */ /* analyse all graphs */
for (i = get_irp_n_irgs() - 1; i >= 0; --i) { for (i = get_irp_n_irgs() - 1; i >= 0; --i) {
ir_graph *irg = get_irp_irg(i); ir_graph *irg = get_irp_irg(i);
irg_walk_graph(irg, callee_walker, remove_Tuples, NULL); irg_walk_graph(irg, callee_walker, remove_Tuples, NULL);
...@@ -841,23 +819,11 @@ static void destruct_walker(ir_node * node, void * env) { ...@@ -841,23 +819,11 @@ static void destruct_walker(ir_node * node, void * env) {
/*--------------------------------------------------------------------------*/ /*--------------------------------------------------------------------------*/
void cgana(int *length, ir_entity ***free_methods) { void cgana(int *length, ir_entity ***free_methods) {
ir_entity ** free_meths, **p;
/* Optimize Sel/SymConst nodes and compute all methods that implement an entity. */ /* Optimize Sel/SymConst nodes and compute all methods that implement an entity. */
sel_methods_init(); sel_methods_init();
free_meths = get_free_methods(); *free_methods = get_free_methods(length);
callee_ana(); callee_ana();
sel_methods_dispose(); sel_methods_dispose();
/* Convert the flexible array to an array that can be handled
by standard C. */
p = xmalloc(sizeof(*p) * ARR_LEN(free_meths));
memcpy(p, free_meths, ARR_LEN(free_meths) * sizeof(*p));
*length = ARR_LEN(free_meths);
*free_methods = p;
DEL_ARR_F(free_meths);
} }
void free_callee_info(ir_graph *irg) { void free_callee_info(ir_graph *irg) {
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment