diff options
| author | Steve Baird <baird@adacore.com> | 2022-08-10 14:04:29 -0700 |
|---|---|---|
| committer | Marc Poulhiès <poulhies@adacore.com> | 2022-09-06 09:14:21 +0200 |
| commit | c56c7d3aeb050ddfe7a41cc00ce5eaa5c4edd636 (patch) | |
| tree | b1359d98e90bbce8161e9765f3eec774d86a51c8 | |
| parent | 6d16658d7d0aa0b3d1cede5d7a5853b2d62caf1c (diff) | |
| download | gcc-c56c7d3aeb050ddfe7a41cc00ce5eaa5c4edd636.zip gcc-c56c7d3aeb050ddfe7a41cc00ce5eaa5c4edd636.tar.gz gcc-c56c7d3aeb050ddfe7a41cc00ce5eaa5c4edd636.tar.bz2 | |
[Ada] Cope with scalar subtypes that have a non-scalar basetype.
In some cases, the compiler builds a subtype entity Typ such that
Is_Scalar (Typ) is True and Is_Scalar (Base_Type (Typ)) is False.
This comes up in some cases involving a subtype of a private type,
where the full view of the private type is a scalar type. In such a
situation, it may also be the case that Is_Enumeration_Type (Typ) is True
and Is_Enumeration_Type (Base_Type (Typ)) is False. Some code incorrectly
assumed that if a subtype is known to be a scalar (respectively, enumeration)
type, then the same must be true of the base type of that subtype. Fix that
code to handle the case where that assumption does not hold.
gcc/ada/
* exp_attr.adb
(Attribute_Valid): Ensure that PBtyp is initialized to a value for
which Is_Scalar_Type is True.
* checks.adb
(Determine_Range): Call Implemention_Base_Type instead of
Base_Type in order to ensure that result is suitable for passing
to Enum_Pos_To_Rep.
| -rw-r--r-- | gcc/ada/checks.adb | 3 | ||||
| -rw-r--r-- | gcc/ada/exp_attr.adb | 5 |
2 files changed, 6 insertions, 2 deletions
diff --git a/gcc/ada/checks.adb b/gcc/ada/checks.adb index 22577c8..26d5a4e 100644 --- a/gcc/ada/checks.adb +++ b/gcc/ada/checks.adb @@ -5094,7 +5094,8 @@ package body Checks is -- Don't deal with enumerated types with non-standard representation or else (Is_Enumeration_Type (Typ) - and then Present (Enum_Pos_To_Rep (Base_Type (Typ)))) + and then Present (Enum_Pos_To_Rep + (Implementation_Base_Type (Typ)))) -- Ignore type for which an error has been posted, since range in -- this case may well be a bogosity deriving from the error. Also diff --git a/gcc/ada/exp_attr.adb b/gcc/ada/exp_attr.adb index 33eec37..c4187c9 100644 --- a/gcc/ada/exp_attr.adb +++ b/gcc/ada/exp_attr.adb @@ -7103,7 +7103,10 @@ package body Exp_Attr is -- See separate sections below for the generated code in each case. when Attribute_Valid => Valid : declare - PBtyp : Entity_Id := Base_Type (Validated_View (Ptyp)); + PBtyp : Entity_Id := Implementation_Base_Type (Validated_View (Ptyp)); + pragma Assert (Is_Scalar_Type (PBtyp) + or else Serious_Errors_Detected > 0); + -- The scalar base type, looking through private types Save_Validity_Checks_On : constant Boolean := Validity_Checks_On; |