From 84cce55d0e1acc39d7b5bb44403eb0e1e61fc7b6 Mon Sep 17 00:00:00 2001 From: Richard Henderson Date: Sun, 12 Sep 2004 19:28:19 -0700 Subject: [PATCH] re PR tree-optimization/10528 (Unexpected loop un-optimization when removing a field of a class) PR tree-opt/10528 * tree-inline.c (copy_body_r): Recompute bits for ADDR_EXPR, after copying its argument. From-SVN: r87423 --- gcc/ChangeLog | 6 ++++++ gcc/tree-inline.c | 10 ++++++++++ 2 files changed, 16 insertions(+) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 5a704a16dbe..ce26d86ad26 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2004-09-13 Richard Henderson + + PR tree-opt/10528 + * tree-inline.c (copy_body_r): Recompute bits for ADDR_EXPR, + after copying its argument. + 2004-09-13 Joseph S. Myers * c-common.c, c-decl.c, c-lex.c, c-objc-common.c, c-opts.c, diff --git a/gcc/tree-inline.c b/gcc/tree-inline.c index 6286f8ab380..82b7857cb1b 100644 --- a/gcc/tree-inline.c +++ b/gcc/tree-inline.c @@ -619,6 +619,16 @@ copy_body_r (tree *tp, int *walk_subtrees, void *data) TREE_OPERAND (*tp, 1) = TREE_OPERAND (*tp, 3); TREE_OPERAND (*tp, 3) = NULL_TREE; } + + /* Variable substitution need not be simple. In particular, the + INDIRECT_REF substitution above. Make sure that TREE_CONSTANT + and friends are up-to-date. */ + else if (TREE_CODE (*tp) == ADDR_EXPR) + { + walk_tree (&TREE_OPERAND (*tp, 0), copy_body_r, id, NULL); + recompute_tree_invarant_for_addr_expr (*tp); + *walk_subtrees = 0; + } } /* Keep iterating. */ -- 2.30.2