[multiple changes]
authorArnaud Charlet <charlet@gcc.gnu.org>
Mon, 4 Aug 2014 10:13:04 +0000 (12:13 +0200)
committerArnaud Charlet <charlet@gcc.gnu.org>
Mon, 4 Aug 2014 10:13:04 +0000 (12:13 +0200)
2014-08-04  Thomas Quinot  <quinot@adacore.com>

* sem_ch5.adb: Minor reformatting.

2014-08-04  Ed Schonberg  <schonberg@adacore.com>

* 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
gcc/ada/freeze.adb
gcc/ada/sem_ch5.adb

index 5684a668265cf5ed1c118d4f7a941488f172bef8..5ca6f568e163cf2df74964d94c364228b0e32cc1 100644 (file)
@@ -1,3 +1,19 @@
+2014-08-04  Thomas Quinot  <quinot@adacore.com>
+
+       * sem_ch5.adb: Minor reformatting.
+
+2014-08-04  Ed Schonberg  <schonberg@adacore.com>
+
+       * 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  <dewar@adacore.com>
 
        * exp_ch5.adb, sem_ch5.adb, einfo.ads: Minor reformatting.
index 68300e1a4b97c5b5cb9a4a157d3c394364b88676..677fb42259b4c7df11bf40dc861fe4ca02ed27e8 100644 (file)
@@ -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.
 
index 4ece78b8e9897b992e1b3f3481d30bfe73c896ec..b80efcec70495f9d1cf4789f78dfd2b3bc3e0876 100644 (file)
@@ -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),