projects
/
libfirm
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Do not mark the transformed as visited. It makes no sense at all.
[libfirm]
/
ir
/
opt
/
convopt.c
diff --git
a/ir/opt/convopt.c
b/ir/opt/convopt.c
index
ca983f0
..
09b5cd7
100644
(file)
--- a/
ir/opt/convopt.c
+++ b/
ir/opt/convopt.c
@@
-87,7
+87,7
@@
int is_downconv(ir_mode *src_mode, ir_mode *dest_mode)
return
mode_is_int(src_mode) &&
mode_is_int(dest_mode) &&
return
mode_is_int(src_mode) &&
mode_is_int(dest_mode) &&
- get_mode_size_bits(dest_mode) < get_mode_size_bits(src_mode);
+ get_mode_size_bits(dest_mode) <
=
get_mode_size_bits(src_mode);
}
static
}
static
@@
-107,7
+107,7
@@
int get_conv_costs(const ir_node *node, ir_mode *dest_mode)
}
if (is_Conv(node) &&
}
if (is_Conv(node) &&
- is_downconv(
get_irn_mode(node)
, dest_mode) &&
+ is_downconv(
mode
, dest_mode) &&
get_irn_mode(get_Conv_op(node)) == dest_mode) {
return -1;
}
get_irn_mode(get_Conv_op(node)) == dest_mode) {
return -1;
}
@@
-117,7
+117,7
@@
int get_conv_costs(const ir_node *node, ir_mode *dest_mode)
return 1;
}
return 1;
}
- if (is_Conv(node) && is_downconv(
get_irn_mode(node)
, dest_mode)) {
+ if (is_Conv(node) && is_downconv(
mode
, dest_mode)) {
return get_conv_costs(get_Conv_op(node), dest_mode) - 1;
}
return get_conv_costs(get_Conv_op(node), dest_mode) - 1;
}
@@
-140,7
+140,7
@@
int get_conv_costs(const ir_node *node, ir_mode *dest_mode)
}
#endif
}
#endif
- if (!is_optimizable_node(node)) {
+ if (!
mode_is_int(mode) || !
is_optimizable_node(node)) {
return 1;
}
return 1;
}
@@
-165,10
+165,11
@@
static ir_node *place_conv(ir_node *node, ir_mode *dest_mode)
static
ir_node *conv_transform(ir_node *node, ir_mode *dest_mode)
{
static
ir_node *conv_transform(ir_node *node, ir_mode *dest_mode)
{
- size_t arity;
- size_t i;
+ ir_mode *mode = get_irn_mode(node);
+ size_t arity;
+ size_t i;
- if (
get_irn_mode(node)
== dest_mode)
+ if (
mode
== dest_mode)
return node;
if (is_Const(node)) {
return node;
if (is_Const(node)) {
@@
-182,7
+183,7
@@
ir_node *conv_transform(ir_node *node, ir_mode *dest_mode)
}
if (is_Conv(node) &&
}
if (is_Conv(node) &&
- is_downconv(
get_irn_mode(node)
, dest_mode) &&
+ is_downconv(
mode
, dest_mode) &&
get_irn_mode(get_Conv_op(node)) == dest_mode) {
return get_Conv_op(node);
}
get_irn_mode(get_Conv_op(node)) == dest_mode) {
return get_Conv_op(node);
}
@@
-191,11
+192,11
@@
ir_node *conv_transform(ir_node *node, ir_mode *dest_mode)
return place_conv(node, dest_mode);
}
return place_conv(node, dest_mode);
}
- if (is_Conv(node) && is_downconv(
get_irn_mode(node)
, dest_mode)) {
+ if (is_Conv(node) && is_downconv(
mode
, dest_mode)) {
return conv_transform(get_Conv_op(node), dest_mode);
}
return conv_transform(get_Conv_op(node), dest_mode);
}
- if (!is_optimizable_node(node)) {
+ if (!
mode_is_int(mode) || !
is_optimizable_node(node)) {
return place_conv(node, dest_mode);
}
return place_conv(node, dest_mode);
}
@@
-255,6
+256,9
@@
void conv_opt_walker(ir_node *node, void *data)
mode = get_irn_mode(node);
pred_mode = get_irn_mode(pred);
mode = get_irn_mode(node);
pred_mode = get_irn_mode(pred);
+ if (mode_is_reference(mode) || mode_is_reference(pred_mode))
+ return;
+
if (!is_Phi(pred) && !is_downconv(pred_mode, mode))
return;
if (!is_Phi(pred) && !is_downconv(pred_mode, mode))
return;
@@
-270,7
+274,7
@@
void conv_opt_walker(ir_node *node, void *data)
}
}
}
}
-
void
conv_opt(ir_graph *irg)
+
int
conv_opt(ir_graph *irg)
{
char invalidate = 0;
FIRM_DBG_REGISTER(dbg, "firm.opt.conv");
{
char invalidate = 0;
FIRM_DBG_REGISTER(dbg, "firm.opt.conv");
@@
-288,4
+292,5
@@
void conv_opt(ir_graph *irg)
if (invalidate) {
set_irg_outs_inconsistent(irg);
}
if (invalidate) {
set_irg_outs_inconsistent(irg);
}
+ return invalidate;
}
}