@@ -1780,14 +1780,25 @@ package body Exp_Attr is
begin
Aux_Decl := Empty;
- -- Generate a nominal type for the constant when the prefix is of
- -- a constrained type. This is achieved by setting the Etype of
- -- the relocated prefix to its base type. Since the prefix is now
- -- the initialization expression of the constant, its freezing
- -- will produce a proper nominal type.
-
Temp_Expr := Relocate_Node (Pref);
- Set_Etype (Temp_Expr, Base_Typ);
+
+ -- For Etype (Temp_Expr) in some cases we cannot use either
+ -- Etype (Pref) or Base_Typ. So we set Etype (Temp_Expr) to null
+ -- and mark Temp_Expr as requiring analysis. Rather than trying
+ -- to sort out exactly when this is needed, we do it
+ -- unconditionally.
+ -- One case where this is needed is when
+ -- 1) Pref is an N_Selected_Component name that
+ -- refers to a component which is subject to a
+ -- discriminant-dependent constraint; and
+ -- 2) The prefix of that N_Selected_Component refers to a
+ -- formal parameter with an unconstrained subtype; and
+ -- 3) Pref has only been preanalyzed (so that
+ -- Build_Actual_Subtype_Of_Component has not been called
+ -- and Etype (Pref) equals the Etype of the component).
+
+ Set_Etype (Temp_Expr, Empty);
+ Set_Analyzed (Temp_Expr, False);
-- Generate:
-- Temp : constant Base_Typ := Pref;
From: Steve Baird <baird@adacore.com> The Etype for an N_Selected_Component node usually should not match the Etype of the referenced component if the component is subject to a discriminant-dependent constraint. Instead Build_Actual_Subtype_Of_Component should be called. Fix a case where this rule was not being followed (because B_A_S_O_C is not called during preanalysis of a component selection), resulting in a tree that confused CodePeer because the subtype was wrong. gcc/ada/ * exp_attr.adb (Expand_Loop_Entry_Attribute): Ensure that Etype of the saved expression is set correctly. Tested on x86_64-pc-linux-gnu, committed on master. --- gcc/ada/exp_attr.adb | 25 ++++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-)