diff options
| author | rguenth <rguenth@138bc75d-0d04-0410-961f-82ee72b054a4> | 2007-10-12 14:21:43 +0000 |
|---|---|---|
| committer | rguenth <rguenth@138bc75d-0d04-0410-961f-82ee72b054a4> | 2007-10-12 14:21:43 +0000 |
| commit | f4628d450c11089c2a55501397229982630e1a9a (patch) | |
| tree | 636e64fecd4d0a4db7898531177e24b66f90b990 /gcc/tree-ssa-forwprop.c | |
| parent | 41d0a435a921921ce771fec383a7facd0441cab6 (diff) | |
| download | gcc-f4628d450c11089c2a55501397229982630e1a9a.tar.gz | |
2007-10-12 Richard Guenther <rguenther@suse.de>
* tree-ssa-forwprop.c (forward_propagate_into_cond): For
combining both operands require either both have single uses
or combining to a constant.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@129261 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'gcc/tree-ssa-forwprop.c')
| -rw-r--r-- | gcc/tree-ssa-forwprop.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/gcc/tree-ssa-forwprop.c b/gcc/tree-ssa-forwprop.c index b92a9f13f48..047d057bfc9 100644 --- a/gcc/tree-ssa-forwprop.c +++ b/gcc/tree-ssa-forwprop.c @@ -360,7 +360,7 @@ forward_propagate_into_cond (tree cond_expr, tree stmt) tree tmp = NULL_TREE; tree cond = COND_EXPR_COND (cond_expr); tree name, def_stmt, rhs0 = NULL_TREE, rhs1 = NULL_TREE; - bool single_use_p; + bool single_use0_p = false, single_use1_p = false; /* We can do tree combining on SSA_NAME and comparison expressions. */ if (COMPARISON_CLASS_P (cond) @@ -369,7 +369,7 @@ forward_propagate_into_cond (tree cond_expr, tree stmt) /* For comparisons use the first operand, that is likely to simplify comparisons against constants. */ name = TREE_OPERAND (cond, 0); - def_stmt = get_prop_source_stmt (name, false, &single_use_p); + def_stmt = get_prop_source_stmt (name, false, &single_use0_p); if (def_stmt != NULL_TREE && can_propagate_from (def_stmt)) { @@ -377,7 +377,7 @@ forward_propagate_into_cond (tree cond_expr, tree stmt) rhs0 = GIMPLE_STMT_OPERAND (def_stmt, 1); tmp = combine_cond_expr_cond (TREE_CODE (cond), boolean_type_node, fold_convert (TREE_TYPE (op1), rhs0), - op1, !single_use_p); + op1, !single_use0_p); } /* If that wasn't successful, try the second operand. */ if (tmp == NULL_TREE @@ -385,7 +385,7 @@ forward_propagate_into_cond (tree cond_expr, tree stmt) { tree op0 = TREE_OPERAND (cond, 0); name = TREE_OPERAND (cond, 1); - def_stmt = get_prop_source_stmt (name, false, &single_use_p); + def_stmt = get_prop_source_stmt (name, false, &single_use1_p); if (def_stmt == NULL_TREE || !can_propagate_from (def_stmt)) return did_something; @@ -394,7 +394,7 @@ forward_propagate_into_cond (tree cond_expr, tree stmt) tmp = combine_cond_expr_cond (TREE_CODE (cond), boolean_type_node, op0, fold_convert (TREE_TYPE (op0), rhs1), - !single_use_p); + !single_use1_p); } /* If that wasn't successful either, try both operands. */ if (tmp == NULL_TREE @@ -403,7 +403,7 @@ forward_propagate_into_cond (tree cond_expr, tree stmt) tmp = combine_cond_expr_cond (TREE_CODE (cond), boolean_type_node, rhs0, fold_convert (TREE_TYPE (rhs0), rhs1), - !single_use_p); + !(single_use0_p && single_use1_p)); } else if (TREE_CODE (cond) == SSA_NAME) { |