diff options
author | Thomas Quinot <quinot@adacore.com> | 2018-09-26 09:20:05 +0000 |
---|---|---|
committer | Pierre-Marie de Rodat <pmderodat@gcc.gnu.org> | 2018-09-26 09:20:05 +0000 |
commit | def15641da84fb82adfeb946c6d714683148e6e3 (patch) | |
tree | 03afa6c40302957d55a36ab4791afc3234c1e4ef | |
parent | 0ebf09ed046391377af708ab2471403d3affdddd (diff) | |
download | gcc-def15641da84fb82adfeb946c6d714683148e6e3.zip gcc-def15641da84fb82adfeb946c6d714683148e6e3.tar.gz gcc-def15641da84fb82adfeb946c6d714683148e6e3.tar.bz2 |
[Ada] Fix inheritance of representation items defined as aspects
When a representation item is defined by a pragma or attribute
definition clause, the entity it applies to is that of the Name of the
representation item. But when it is defined by an aspect definition, the
entity is directly denoted by the Entity attribute of the represenation
item. The circuitry that inherits representation items for derived types
or subtypes must account for these two possible cases.
2018-09-26 Thomas Quinot <quinot@adacore.com>
gcc/ada/
* sem_ch13.adb (Inherit_Aspects_At_Freeze_Point): For a
representation item that is an N_Aspect_Definition, retrieve the
entity it applies to using the Entity attribute.
gcc/testsuite/
* gnat.dg/sso13.adb: New testcase.
From-SVN: r264636
-rw-r--r-- | gcc/ada/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/ada/sem_ch13.adb | 24 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
-rw-r--r-- | gcc/testsuite/gnat.dg/sso13.adb | 24 |
4 files changed, 56 insertions, 2 deletions
diff --git a/gcc/ada/ChangeLog b/gcc/ada/ChangeLog index f460963..38354ec 100644 --- a/gcc/ada/ChangeLog +++ b/gcc/ada/ChangeLog @@ -1,3 +1,9 @@ +2018-09-26 Thomas Quinot <quinot@adacore.com> + + * sem_ch13.adb (Inherit_Aspects_At_Freeze_Point): For a + representation item that is an N_Aspect_Definition, retrieve the + entity it applies to using the Entity attribute. + 2018-09-26 Justin Squirek <squirek@adacore.com> * sem_ch8.adb (Analyze_Subprogram_Renaming): Add extra condition diff --git a/gcc/ada/sem_ch13.adb b/gcc/ada/sem_ch13.adb index 00854c9..b999be6 100644 --- a/gcc/ada/sem_ch13.adb +++ b/gcc/ada/sem_ch13.adb @@ -11446,6 +11446,26 @@ package body Sem_Ch13 is -- specification node whose correponding pragma (if any) is present in -- the Rep Item chain of the entity it has been specified to. + function Rep_Item_Entity (Rep_Item : Node_Id) return Entity_Id; + -- Return the entity for which Rep_Item is specified + + --------------------- + -- Rep_Item_Entity -- + --------------------- + + function Rep_Item_Entity (Rep_Item : Node_Id) return Entity_Id is + begin + if Nkind (Rep_Item) = N_Aspect_Specification then + return Entity (Rep_Item); + + else + pragma Assert (Nkind_In (Rep_Item, + N_Pragma, + N_Attribute_Definition_Clause)); + return Entity (Name (Rep_Item)); + end if; + end Rep_Item_Entity; + -------------------------------------------------- -- Is_Pragma_Or_Corr_Pragma_Present_In_Rep_Item -- -------------------------------------------------- @@ -11650,8 +11670,8 @@ package body Sem_Ch13 is and then Has_Rep_Item (Typ, Name_Bit_Order) then Set_Reverse_Bit_Order (Bas_Typ, - Reverse_Bit_Order (Entity (Name - (Get_Rep_Item (Typ, Name_Bit_Order))))); + Reverse_Bit_Order (Rep_Item_Entity + (Get_Rep_Item (Typ, Name_Bit_Order)))); end if; end if; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index be0c2c3..c96126c 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2018-09-26 Thomas Quinot <quinot@adacore.com> + + * gnat.dg/sso13.adb: New testcase. + 2018-09-26 Justin Squirek <squirek@adacore.com> * gnat.dg/warn16.adb: New testcase. diff --git a/gcc/testsuite/gnat.dg/sso13.adb b/gcc/testsuite/gnat.dg/sso13.adb new file mode 100644 index 0000000..26ef762 --- /dev/null +++ b/gcc/testsuite/gnat.dg/sso13.adb @@ -0,0 +1,24 @@ +-- { dg-do compile } + +with System; + +procedure SSO13 is + type Pulse_Buffer_Type is abstract tagged null record + with Bit_Order => System.High_Order_First, + Scalar_Storage_order =>System.High_order_First; -- { dg-warning "scalar storage order specified but no component clause" } + type Pulse_Train_Type is abstract new Pulse_Buffer_Type with null record; + type WO_Pulse_Train_Type is new Pulse_Train_Type with null record; + type WO_Confirmation_Pulse_Train_Type is new WO_Pulse_Train_Type with record + null; + end record; + + type Update_Dwell_Type is abstract tagged null record + with Bit_Order => System.High_Order_First, + Scalar_Storage_order =>System.High_order_First; -- { dg-warning "scalar storage order specified but no component clause" } + type Confirmation_Dwell_Type is new Update_Dwell_Type with + record + Pulses : aliased WO_Pulse_Train_Type; -- (Location of Error #1) + end record; +begin + null; +end; |