From: Richard Henderson <rth@redhat.com>
Date: Mon, 13 Sep 2004 02:28:19 +0000 (-0700)
Subject: re PR tree-optimization/10528 (Unexpected loop un-optimization when removing a field... 
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=84cce55d0e1acc39d7b5bb44403eb0e1e61fc7b6;p=gcc.git

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
---

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  <rth@redhat.com>
+
+	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  <jsm@polyomino.org.uk>
 
 	* 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.  */