aboutsummaryrefslogtreecommitdiff
path: root/gcc/tree-ssa-forwprop.c
diff options
context:
space:
mode:
authorJakub Jelinek <jakub@redhat.com>2008-11-21 22:59:33 +0100
committerJakub Jelinek <jakub@gcc.gnu.org>2008-11-21 22:59:33 +0100
commit88e091c3372048f6c3b3d41cbfedf422b5496808 (patch)
treec0dad29bcfb209c5ed0c46b8e8b2cc20e4bdba85 /gcc/tree-ssa-forwprop.c
parent0e51e769b52cb4c91339a0eef5fa6a70c0ee2dde (diff)
downloadgcc-88e091c3372048f6c3b3d41cbfedf422b5496808.zip
gcc-88e091c3372048f6c3b3d41cbfedf422b5496808.tar.gz
gcc-88e091c3372048f6c3b3d41cbfedf422b5496808.tar.bz2
re PR middle-end/38200 (internal compiler error: in find_func_aliases, at tree-ssa-structalias.c:3905)
PR middle-end/38200 * tree-ssa-forwprop.c (forward_propagate_addr_expr_1): Only propagate x = &a into *x = b if conversion from b to a's type is useless. * gcc.dg/pr38200.c: New test. From-SVN: r142100
Diffstat (limited to 'gcc/tree-ssa-forwprop.c')
-rw-r--r--gcc/tree-ssa-forwprop.c32
1 files changed, 10 insertions, 22 deletions
diff --git a/gcc/tree-ssa-forwprop.c b/gcc/tree-ssa-forwprop.c
index 333ef82..b19879f 100644
--- a/gcc/tree-ssa-forwprop.c
+++ b/gcc/tree-ssa-forwprop.c
@@ -719,30 +719,18 @@ forward_propagate_addr_expr_1 (tree name, tree def_rhs,
propagate the ADDR_EXPR into the use of NAME and fold the result. */
if (TREE_CODE (lhs) == INDIRECT_REF
&& TREE_OPERAND (lhs, 0) == name
- && may_propagate_address_into_dereference (def_rhs, lhs))
+ && may_propagate_address_into_dereference (def_rhs, lhs)
+ && (lhsp != gimple_assign_lhs_ptr (use_stmt)
+ || useless_type_conversion_p (TREE_TYPE (TREE_OPERAND (def_rhs, 0)),
+ TREE_TYPE (rhs))))
{
- bool valid = true;
- if (lhsp == gimple_assign_lhs_ptr (use_stmt)
- && !useless_type_conversion_p (TREE_TYPE (TREE_OPERAND (def_rhs, 0)),
- TREE_TYPE (rhs))
- && !CONVERT_EXPR_CODE_P (rhs_code))
- {
- if (rhs_code == SSA_NAME)
- gimple_assign_set_rhs_code (use_stmt, NOP_EXPR);
- else
- valid = false;
- }
- if (valid)
- {
- *lhsp = unshare_expr (TREE_OPERAND (def_rhs, 0));
- fold_stmt_inplace (use_stmt);
- tidy_after_forward_propagate_addr (use_stmt);
+ *lhsp = unshare_expr (TREE_OPERAND (def_rhs, 0));
+ fold_stmt_inplace (use_stmt);
+ tidy_after_forward_propagate_addr (use_stmt);
- /* Continue propagating into the RHS if this was not the only
- use. */
- if (single_use_p)
- return true;
- }
+ /* Continue propagating into the RHS if this was not the only use. */
+ if (single_use_p)
+ return true;
}
/* Strip away any outer COMPONENT_REF, ARRAY_REF or ADDR_EXPR