re PR ada/46192 (wrong code for renaming of volatile packed array with address clause)

	PR ada/46192
	* gcc-interface/decl.c (gnat_to_gnu_entity) <object>: In the case of a
	renaming, preserve the volatileness through the indirection, if any.

From-SVN: r183367

diff --git a/gcc/ada/ChangeLog b/gcc/ada/ChangeLog
index c9bca727e143940198180f771f00a99fe46eb549..d80ac672a29087ceb783d65df0d0e0f1ffdaeb91 100644 (file)
--- a/gcc/ada/ChangeLog
+++ b/gcc/ada/ChangeLog
@@ -1,3 +1,9 @@
+2012-01-21  Eric Botcazou  <ebotcazou@adacore.com>
+
+       PR ada/46192
+       * gcc-interface/decl.c (gnat_to_gnu_entity) <object>: In the case of a
+       renaming, preserve the volatileness through the indirection, if any.
+
 2012-01-09  Eric Botcazou  <ebotcazou@adacore.com>
 
        * gcc-interface/trans.c (addressable_p) <COMPONENT_REF>: Fix thinko.

diff --git a/gcc/ada/gcc-interface/decl.c b/gcc/ada/gcc-interface/decl.c
index 6d7fa13fde1b866d1f13d7a30aa6b78a3ad6ca7c..b9971248cb0f89656f65438f2a5da23c575a8217 100644 (file)
--- a/gcc/ada/gcc-interface/decl.c
+++ b/gcc/ada/gcc-interface/decl.c
@@ -942,6 +942,14 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                   entity is always accessed indirectly through it.  */
                else
                  {
+                   /* We need to preserve the volatileness of the renamed
+                      object through the indirection.  */
+                   if (TREE_THIS_VOLATILE (gnu_expr)
+                       && !TYPE_VOLATILE (gnu_type))
+                     gnu_type
+                       = build_qualified_type (gnu_type,
+                                               (TYPE_QUALS (gnu_type)
+                                                | TYPE_QUAL_VOLATILE));
                    gnu_type = build_reference_type (gnu_type);
                    inner_const_flag = TREE_READONLY (gnu_expr);
                    const_flag = true;

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index b7e0e69e9511402e037f41e8cc9fc88cb9477aba..b606a2604b00ce6aea236c306bfc226c2720f475 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2012-01-21  Eric Botcazou  <ebotcazou@adacore.com>
+
+       * gnat.dg/renaming5.ad[sb]: New test.
+
 2012-01-20  Kai Tietz  <ktietz@redhat.com>
 
        * g++.dg/torture/pr51344.C: Fix typo.

diff --git a/gcc/testsuite/gnat.dg/renaming5.adb b/gcc/testsuite/gnat.dg/renaming5.adb
new file mode 100644 (file)
index 0000000..25374fe
--- /dev/null
+++ b/gcc/testsuite/gnat.dg/renaming5.adb
@@ -0,0 +1,30 @@
+-- PR ada/46192
+-- Testcase by Rolf Ebert <rolf.ebert.gcc@gmx.de>
+
+-- { dg-do compile }
+-- { dg-options "-O2 -fdump-tree-optimized" }
+
+with System; use System;
+
+package body Renaming5 is
+
+   type Bits_In_Byte is array (0 .. 7) of Boolean;
+   pragma Pack (Bits_In_Byte);
+
+   A : Bits_In_Byte;
+   for A'Address use System'To_Address(16#c0#);
+   pragma Volatile (A);
+
+   B : Bits_In_Byte renames A;
+
+   procedure Proc is
+   begin
+      while B (0) = False loop
+         null;
+      end loop;
+   end;
+
+end Renaming5;
+
+-- { dg-final { scan-tree-dump-times "goto" 2 "optimized" } }
+-- { dg-final { cleanup-tree-dump "optimized" } }

diff --git a/gcc/testsuite/gnat.dg/renaming5.ads b/gcc/testsuite/gnat.dg/renaming5.ads
new file mode 100644 (file)
index 0000000..2b39663
--- /dev/null
+++ b/gcc/testsuite/gnat.dg/renaming5.ads
@@ -0,0 +1,5 @@
+package Renaming5 is
+
+   procedure Proc;
+
+end Renaming5;