]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
ada: Fix internal error on bit-packed array type with Volatile_Full_Access
authorEric Botcazou <ebotcazou@adacore.com>
Wed, 11 Sep 2024 17:53:12 +0000 (19:53 +0200)
committerMarc Poulhiès <dkm@gcc.gnu.org>
Fri, 25 Oct 2024 09:09:03 +0000 (11:09 +0200)
The problem occurs when the component type is a record type with default
values for the initialization procedure of the (base) array type, because
the compiler is trying to generate a full access for a parameter of the
base array type, which does not make sense.

gcc/ada/ChangeLog:

PR ada/116551
* gcc-interface/trans.cc (node_is_atomic) <N_Identifier>: Return
false if the type of the entity is an unconstrained array type.
(node_is_volatile_full_access) <N_Identifier>: Likewise.

gcc/ada/gcc-interface/trans.cc

index 7728e60ccb6bb4d86798d90ae3aa7d4d5368186c..93f67404ddb44df9f771e9bf88410605952b60ad 100644 (file)
@@ -4263,7 +4263,10 @@ node_is_atomic (Node_Id gnat_node)
       if (!Is_Object (gnat_entity))
        break;
       return Is_Atomic (gnat_entity)
+            /* Disregard the flag on unconstrained arrays or simple constants
+               since we cannot or need not generate an atomic access.  */
             || (Is_Atomic (Etype (gnat_entity))
+                && Ekind (Etype (gnat_entity)) != E_Array_Type
                 && !simple_constant_p (gnat_entity));
 
     case N_Selected_Component:
@@ -4303,7 +4306,10 @@ node_is_volatile_full_access (Node_Id gnat_node)
       if (!Is_Object (gnat_entity))
        break;
       return Is_Volatile_Full_Access (gnat_entity)
+            /* Disregard the flag on unconstrained arrays or simple constants
+               since we cannot or need not generate a full access.  */
             || (Is_Volatile_Full_Access (Etype (gnat_entity))
+                && Ekind (Etype (gnat_entity)) != E_Array_Type
                 && !simple_constant_p (gnat_entity));
 
     case N_Selected_Component: