From: Zdenek Dvorak <dvorakz@suse.cz>
Date: Mon, 13 Jun 2005 14:59:40 +0000 (+0200)
Subject: re PR middle-end/21985 (miscompiled or wrong code snippet?)
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=adacecf105b889e4d7f9b48dea7895724f850d66;p=gcc.git

re PR middle-end/21985 (miscompiled or wrong code snippet?)

	PR middle-end/21985
	* fold-const.c (split_address_to_core_and_offset): Always return
	the address of the base object.

	* gcc.dg/tree-ssa/pr21985.c: New test.

From-SVN: r100877
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 037a2050c09..f32577815f2 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2005-06-13  Zdenek Dvorak  <dvorakz@suse.cz>
+
+	PR middle-end/21985
+	* fold-const.c (split_address_to_core_and_offset): Always return
+	the address of the base object.
+
 2005-06-13  Bernd Schmidt  <bernd.schmidt@analog.com>
 
 	* doc/invoke.texi (Blackfin Options): Document -mlong-calls.
diff --git a/gcc/fold-const.c b/gcc/fold-const.c
index af3f01c38df..2c1bd15b365 100644
--- a/gcc/fold-const.c
+++ b/gcc/fold-const.c
@@ -11801,9 +11801,7 @@ split_address_to_core_and_offset (tree exp,
       core = get_inner_reference (TREE_OPERAND (exp, 0), &bitsize, pbitpos,
 				  poffset, &mode, &unsignedp, &volatilep,
 				  false);
-
-      if (TREE_CODE (core) == INDIRECT_REF)
-	core = TREE_OPERAND (core, 0);
+      core = build_fold_addr_expr (core);
     }
   else
     {
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0baaf8a6e86..42536ea780e 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2005-06-13  Zdenek Dvorak  <dvorakz@suse.cz>
+
+	PR middle-end/21985
+	* gcc.dg/tree-ssa/pr21985.c: New test.
+
 2005-06-12  James A. Morrison  <phython@gcc.gnu.org>
 
 	* ggcc.dg/pr14796-1.c: Add tests for (X << C) >> C.
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr21985.c b/gcc/testsuite/gcc.dg/tree-ssa/pr21985.c
new file mode 100644
index 00000000000..e94031045b4
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/tree-ssa/pr21985.c
@@ -0,0 +1,32 @@
+/* { dg-do compile } */
+/* { dg-options "-O1 -fdump-tree-vars" } */
+
+struct
+{
+  struct
+    {
+      int a;
+      int b;
+    } a[100];
+} a;
+
+void foo(unsigned);
+unsigned *bar (void);
+
+int
+main (void)
+{
+  unsigned *b = bar ();
+
+  /* This should be folded.  */
+  foo (&a.a[50].a - &a.a[33].b);
+  foo ((unsigned) &a.a[50].b - (unsigned) &a);
+
+  /* And this should not.  */
+  foo ((unsigned) &b - (unsigned) b);
+  return 0;
+}
+
+/* Two of the calls to foo should be folded to just foo(constant).  */
+
+/* { dg-final { scan-tree-dump-times "foo \\(\[0-9\]*\\)" 2 "vars" } } */