From 0fea901b8fe3f3476282e3db52f329002f551381 Mon Sep 17 00:00:00 2001 From: Arnaud Charlet Date: Mon, 4 Aug 2014 12:13:04 +0200 Subject: [PATCH] [multiple changes] 2014-08-04 Thomas Quinot * sem_ch5.adb: Minor reformatting. 2014-08-04 Ed Schonberg * freeze.adb (Late_Freeze_Subprogram): Following AI05-151, a function can return a limited view of a type declared elsewhere. In that case the function cannot be frozen at the end of its enclosing package. If its first use is in a different unit, it cannot be frozen there, but if the call is legal the full view of the return type is available and the subprogram can now be frozen. However the freeze node cannot be inserted at the point of call, but rather must go in the package holding the function, so that the backend can process it in the proper context. From-SVN: r213562 --- gcc/ada/ChangeLog | 16 ++++++++ gcc/ada/freeze.adb | 90 ++++++++++++++++++++++++++++++++++----------- gcc/ada/sem_ch5.adb | 8 ++-- 3 files changed, 90 insertions(+), 24 deletions(-) diff --git a/gcc/ada/ChangeLog b/gcc/ada/ChangeLog index 5684a668265..5ca6f568e16 100644 --- a/gcc/ada/ChangeLog +++ b/gcc/ada/ChangeLog @@ -1,3 +1,19 @@ +2014-08-04 Thomas Quinot + + * sem_ch5.adb: Minor reformatting. + +2014-08-04 Ed Schonberg + + * freeze.adb (Late_Freeze_Subprogram): Following AI05-151, + a function can return a limited view of a type declared + elsewhere. In that case the function cannot be frozen at the end + of its enclosing package. If its first use is in a different unit, + it cannot be frozen there, but if the call is legal the full view + of the return type is available and the subprogram can now be + frozen. However the freeze node cannot be inserted at the point + of call, but rather must go in the package holding the function, + so that the backend can process it in the proper context. + 2014-08-04 Robert Dewar * exp_ch5.adb, sem_ch5.adb, einfo.ads: Minor reformatting. diff --git a/gcc/ada/freeze.adb b/gcc/ada/freeze.adb index 68300e1a4b9..677fb42259b 100644 --- a/gcc/ada/freeze.adb +++ b/gcc/ada/freeze.adb @@ -1815,13 +1815,18 @@ package body Freeze is ------------------- function Freeze_Entity (E : Entity_Id; N : Node_Id) return List_Id is - Loc : constant Source_Ptr := Sloc (N); + Loc : constant Source_Ptr := Sloc (N); + Comp : Entity_Id; + F_Node : Node_Id; + Indx : Node_Id; + Formal : Entity_Id; + Atype : Entity_Id; + Test_E : Entity_Id := E; - Comp : Entity_Id; - F_Node : Node_Id; - Indx : Node_Id; - Formal : Entity_Id; - Atype : Entity_Id; + -- This could use a comment ??? + + Late_Freezing : Boolean := False; + -- Used to detect attempt to freeze function declared in another unit Result : List_Id := No_List; -- List of freezing actions, left at No_List if none @@ -1861,6 +1866,16 @@ package body Freeze is -- Determine whether an arbitrary entity is subject to Boolean aspect -- Import and its value is specified as True. + procedure Late_Freeze_Subprogram (E : Entity_Id); + -- Following AI05-151, a function can return a limited view of a type + -- declared elsewhere. In that case the function cannot be frozen at + -- the end of its enclosing package. If its first use is in a different + -- unit, it cannot be frozen there, but if the call is legal the full + -- view of the return type is available and the subprogram can now be + -- frozen. However the freeze node cannot be inserted at the point of + -- call, but rather must go in the package holding the function, so that + -- the backend can process it in the proper context. + procedure Wrap_Imported_Subprogram (E : Entity_Id); -- If E is an entity for an imported subprogram with pre/post-conditions -- then this procedure will create a wrapper to ensure that proper run- @@ -1885,6 +1900,7 @@ package body Freeze is function After_Last_Declaration return Boolean is Spec : constant Node_Id := Parent (Current_Scope); + begin if Nkind (Spec) = N_Package_Specification then if Present (Private_Declarations (Spec)) then @@ -1894,6 +1910,7 @@ package body Freeze is else return False; end if; + else return False; end if; @@ -2013,8 +2030,7 @@ package body Freeze is else Error_Msg_N ("current instance must be an immutably limited " - & "type (RM-2012, 7.5 (8.1/3))", - Prefix (N)); + & "type (RM-2012, 7.5 (8.1/3))", Prefix (N)); end if; return Abandon; @@ -2182,8 +2198,7 @@ package body Freeze is Error_Msg_Name_1 := CN; Error_Msg_Sloc := Sloc (Arr); Error_Msg_N - ("pragma Pack affects convention % components #??", - PP); + ("pragma Pack affects convention % components #??", PP); Error_Msg_Name_1 := CN; Error_Msg_N ("\array components may not have % compatible " @@ -2260,6 +2275,7 @@ package body Freeze is Comp_Size_C : constant Node_Id := Get_Attribute_Definition_Clause (Ent, Attribute_Component_Size); + begin -- Warn if we have pack and component size so that the -- pack is ignored. @@ -2305,11 +2321,11 @@ package body Freeze is if Present (Pack_Pragma) then Error_Msg_N - ("??pragma Pack causes component size " - & "to be ^!", Pack_Pragma); + ("??pragma Pack causes component size to be ^!", + Pack_Pragma); Error_Msg_N - ("\??use Component_Size to set " - & "desired value!", Pack_Pragma); + ("\??use Component_Size to set desired value!", + Pack_Pragma); end if; end if; @@ -2531,8 +2547,7 @@ package body Freeze is Ilen := Make_Attribute_Reference (Loc, - Prefix => - New_Occurrence_Of (Ityp, Loc), + Prefix => New_Occurrence_Of (Ityp, Loc), Attribute_Name => Name_Range_Length); Analyze_And_Resolve (Ilen); @@ -2562,10 +2577,8 @@ package body Freeze is if Known_RM_Size (Arr) then declare - SizC : constant Node_Id := Size_Clause (Arr); - + SizC : constant Node_Id := Size_Clause (Arr); Discard : Boolean; - pragma Warnings (Off, Discard); begin -- It is not clear if it is possible to have no size clause @@ -3060,6 +3073,7 @@ package body Freeze is if Will_Be_Frozen then Undelay_Type (Comp); + else if Present (Prev) then Set_Next_Entity (Prev, Next_Entity (Comp)); @@ -3107,8 +3121,8 @@ package body Freeze is if Is_Entity_Name (Expression (Alloc)) then Freeze_And_Append (Entity (Expression (Alloc)), N, Result); - elsif - Nkind (Expression (Alloc)) = N_Subtype_Indication + + elsif Nkind (Expression (Alloc)) = N_Subtype_Indication then Freeze_And_Append (Entity (Subtype_Mark (Expression (Alloc))), @@ -3633,6 +3647,25 @@ package body Freeze is return False; end Has_Boolean_Aspect_Import; + ---------------------------- + -- Late_Freeze_Subprogram -- + ---------------------------- + + procedure Late_Freeze_Subprogram (E : Entity_Id) is + Spec : constant Node_Id := + Specification (Unit_Declaration_Node (Scope (E))); + Decls : List_Id; + + begin + if Present (Private_Declarations (Spec)) then + Decls := Private_Declarations (Spec); + else + Decls := Visible_Declarations (Spec); + end if; + + Append_List (Result, Decls); + end Late_Freeze_Subprogram; + ------------------------------ -- Wrap_Imported_Subprogram -- ------------------------------ @@ -4165,6 +4198,16 @@ package body Freeze is if Ekind (E) = E_Function then + -- Check whether function is declared elsewhere. + + Late_Freezing := + Get_Source_Unit (E) /= Get_Source_Unit (N) + and then Expander_Active + and then Ekind (Scope (E)) = E_Package + and then Nkind (Unit_Declaration_Node (Scope (E))) + = N_Package_Declaration + and then not In_Open_Scopes (Scope (E)); + -- Freeze return type R_Type := Etype (E); @@ -4325,6 +4368,11 @@ package body Freeze is Freeze_Subprogram (E); end if; + if Late_Freezing then + Late_Freeze_Subprogram (E); + return No_List; + end if; + -- If warning on suspicious contracts then check for the case of -- a postcondition other than False for a No_Return subprogram. diff --git a/gcc/ada/sem_ch5.adb b/gcc/ada/sem_ch5.adb index 4ece78b8e98..b80efcec704 100644 --- a/gcc/ada/sem_ch5.adb +++ b/gcc/ada/sem_ch5.adb @@ -2204,10 +2204,9 @@ package body Sem_Ch5 is procedure Check_Predicate_Use (T : Entity_Id) is begin - -- A predicated subtype is illegal in loops and related constructs - -- if the predicate is not static, or else if it is a non-static - -- subtype of a statically predicated subtype. + -- if the predicate is not static, or if it is a non-static subtype + -- of a statically predicated subtype. if Is_Discrete_Type (T) and then Has_Predicates (T) @@ -2215,6 +2214,9 @@ package body Sem_Ch5 is or else not Is_Static_Subtype (T) or else Has_Dynamic_Predicate_Aspect (T)) then + -- Seems a confusing message for the case of a static predicate + -- with a non-static subtype??? + Bad_Predicated_Subtype_Use ("cannot use subtype& with non-static predicate for loop " & "iteration", Discrete_Subtype_Definition (N), -- 2.30.2