OSDN Git Service

2007-04-20 Javier Miranda <miranda@adacore.com>
[pf3gnuchains/gcc-fork.git] / gcc / ada / freeze.adb
index 6f4c4c7..f7876ba 100644 (file)
@@ -6,9 +6,7 @@
 --                                                                          --
 --                                 B o d y                                  --
 --                                                                          --
---                            $Revision: 1.281 $
---                                                                          --
---          Copyright (C) 1992-2001, Free Software Foundation, Inc.         --
+--          Copyright (C) 1992-2006, Free Software Foundation, Inc.         --
 --                                                                          --
 -- GNAT is free software;  you can  redistribute it  and/or modify it under --
 -- terms of the  GNU General Public License as published  by the Free Soft- --
 -- or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License --
 -- for  more details.  You should have  received  a copy of the GNU General --
 -- Public License  distributed with GNAT;  see file COPYING.  If not, write --
--- to  the Free Software Foundation,  59 Temple Place - Suite 330,  Boston, --
--- MA 02111-1307, USA.                                                      --
+-- to  the  Free Software Foundation,  51  Franklin  Street,  Fifth  Floor, --
+-- Boston, MA 02110-1301, USA.                                              --
 --                                                                          --
 -- GNAT was originally developed  by the GNAT team at  New York University. --
--- It is now maintained by Ada Core Technologies Inc (http://www.gnat.com). --
+-- Extensive contributions were provided by Ada Core Technologies Inc.      --
 --                                                                          --
 ------------------------------------------------------------------------------
 
@@ -32,14 +30,16 @@ with Einfo;    use Einfo;
 with Elists;   use Elists;
 with Errout;   use Errout;
 with Exp_Ch7;  use Exp_Ch7;
-with Exp_Ch11; use Exp_Ch11;
 with Exp_Pakd; use Exp_Pakd;
 with Exp_Util; use Exp_Util;
+with Exp_Tss;  use Exp_Tss;
 with Layout;   use Layout;
+with Lib.Xref; use Lib.Xref;
 with Nlists;   use Nlists;
 with Nmake;    use Nmake;
 with Opt;      use Opt;
 with Restrict; use Restrict;
+with Rident;   use Rident;
 with Sem;      use Sem;
 with Sem_Cat;  use Sem_Cat;
 with Sem_Ch6;  use Sem_Ch6;
@@ -77,7 +77,11 @@ package body Freeze is
      (Decl  : Node_Id;
       New_S : Entity_Id;
       After : in out Node_Id);
-   --  Build body for a renaming declaration, insert in tree and analyze.
+   --  Build body for a renaming declaration, insert in tree and analyze
+
+   procedure Check_Address_Clause (E : Entity_Id);
+   --  Apply legality checks to address clauses for object declarations,
+   --  at the point the object is frozen.
 
    procedure Check_Strict_Alignment (E : Entity_Id);
    --  E is a base type. If E is tagged or has a component that is aliased
@@ -105,11 +109,6 @@ package body Freeze is
    --  that if a foreign convention is specified, and no specific size
    --  is given, then the size must be at least Integer'Size.
 
-   procedure Freeze_Fixed_Point_Type (Typ : Entity_Id);
-   --  Freeze fixed point type. For fixed-point types, we have to defer
-   --  setting the size and bounds till the freeze point, since they are
-   --  potentially affected by the presence of size and small clauses.
-
    procedure Freeze_Static_Object (E : Entity_Id);
    --  If an object is frozen which has Is_Statically_Allocated set, then
    --  all referenced types must also be marked with this flag. This routine
@@ -125,7 +124,12 @@ package body Freeze is
    --  a subprogram type (i.e. an access to a subprogram).
 
    function Is_Fully_Defined (T : Entity_Id) return Boolean;
-   --  true if T is not private, or has a full view.
+   --  True if T is not private and has no private components, or has a full
+   --  view. Used to determine whether the designated type of an access type
+   --  should be frozen when the access type is frozen. This is done when an
+   --  allocator is frozen, or an expression that may involve attributes of
+   --  the designated type. Otherwise freezing the access type does not freeze
+   --  the designated type.
 
    procedure Process_Default_Expressions
      (E     : Entity_Id;
@@ -159,6 +163,21 @@ package body Freeze is
    --  needed -- see body for details). Never has any effect on T if the
    --  Debug_Info_Off flag is set.
 
+   procedure Undelay_Type (T : Entity_Id);
+   --  T is a type of a component that we know to be an Itype.
+   --  We don't want this to have a Freeze_Node, so ensure it doesn't.
+   --  Do the same for any Full_View or Corresponding_Record_Type.
+
+   procedure Warn_Overlay
+     (Expr : Node_Id;
+      Typ  : Entity_Id;
+      Nam  : Node_Id);
+   --  Expr is the expression for an address clause for entity Nam whose type
+   --  is Typ. If Typ has a default initialization, and there is no explicit
+   --  initialization in the source declaration, check whether the address
+   --  clause might cause overlaying of an entity, and emit a warning on the
+   --  side effect that the initialization will cause.
+
    -------------------------------
    -- Adjust_Esize_For_Alignment --
    -------------------------------
@@ -188,7 +207,6 @@ package body Freeze is
       After : in out Node_Id)
    is
       Body_Node : constant Node_Id := Build_Renamed_Body (Decl, New_S);
-
    begin
       Insert_After (After, Body_Node);
       Mark_Rewrite_Insertion (Body_Node);
@@ -202,8 +220,7 @@ package body Freeze is
 
    function Build_Renamed_Body
      (Decl  : Node_Id;
-      New_S : Entity_Id)
-      return Node_Id
+      New_S : Entity_Id) return Node_Id
    is
       Loc : constant Source_Ptr := Sloc (New_S);
       --  We use for the source location of the renamed body, the location
@@ -238,7 +255,6 @@ package body Freeze is
          Old_S := Etype (Nam);
 
       elsif Nkind (Nam) = N_Indexed_Component then
-
          if Is_Entity_Name (Prefix (Nam)) then
             Old_S := Entity (Prefix (Nam));
          else
@@ -253,7 +269,18 @@ package body Freeze is
       end if;
 
       if Is_Entity_Name (Nam) then
-         Call_Name := New_Reference_To (Old_S, Loc);
+
+         --  If the renamed entity is a predefined operator, retain full
+         --  name to ensure its visibility.
+
+         if Ekind (Old_S) = E_Operator
+           and then Nkind (Nam) = N_Expanded_Name
+         then
+            Call_Name := New_Copy (Name (N));
+         else
+            Call_Name := New_Reference_To (Old_S, Loc);
+         end if;
+
       else
          Call_Name := New_Copy (Name (N));
 
@@ -296,6 +323,8 @@ package body Freeze is
       --  in the declaration. However, default values that are aggregates
       --  are rewritten when partially analyzed, so we recover the original
       --  aggregate to insure that subsequent conformity checking works.
+      --  Similarly, if the default expression was constant-folded, recover
+      --  the original expression.
 
       Formal := First_Formal (Defining_Entity (Decl));
 
@@ -313,7 +342,10 @@ package body Freeze is
                   Set_Entity (Parameter_Type (Param_Spec), Etype (O_Formal));
                end if;
 
-            elsif Nkind (Default_Value (O_Formal)) = N_Aggregate then
+            elsif Nkind (Default_Value (O_Formal)) = N_Aggregate
+              or else Nkind (Original_Node (Default_Value (O_Formal))) /=
+                                           Nkind (Default_Value (O_Formal))
+            then
                Set_Expression (Param_Spec,
                  New_Copy_Tree (Original_Node (Default_Value (O_Formal))));
             end if;
@@ -359,7 +391,7 @@ package body Freeze is
              Parameter_Associations => Actuals);
       end if;
 
-      --  Create entities for subprogram body and formals.
+      --  Create entities for subprogram body and formals
 
       Set_Defining_Unit_Name (Spec,
         Make_Defining_Identifier (Loc, Chars => Chars (New_S)));
@@ -402,26 +434,93 @@ package body Freeze is
       return Body_Node;
    end Build_Renamed_Body;
 
+   --------------------------
+   -- Check_Address_Clause --
+   --------------------------
+
+   procedure Check_Address_Clause (E : Entity_Id) is
+      Addr : constant Node_Id   := Address_Clause (E);
+      Expr : Node_Id;
+      Decl : constant Node_Id   := Declaration_Node (E);
+      Typ  : constant Entity_Id := Etype (E);
+
+   begin
+      if Present (Addr) then
+         Expr := Expression (Addr);
+
+         --  If we have no initialization of any kind, then we don't
+         --  need to place any restrictions on the address clause, because
+         --  the object will be elaborated after the address clause is
+         --  evaluated. This happens if the declaration has no initial
+         --  expression, or the type has no implicit initialization, or
+         --  the object is imported.
+
+         --  The same holds for all initialized scalar types and all
+         --  access types. Packed bit arrays of size up to 64 are
+         --  represented using a modular type with an initialization
+         --  (to zero) and can be processed like other initialized
+         --  scalar types.
+
+         --  If the type is controlled, code to attach the object to a
+         --  finalization chain is generated at the point of declaration,
+         --  and therefore the elaboration of the object cannot be delayed:
+         --  the address expression must be a constant.
+
+         if (No (Expression (Decl))
+              and then not Controlled_Type (Typ)
+              and then
+                (not Has_Non_Null_Base_Init_Proc (Typ)
+                  or else Is_Imported (E)))
+
+           or else
+             (Present (Expression (Decl))
+               and then Is_Scalar_Type (Typ))
+
+           or else
+             Is_Access_Type (Typ)
+
+           or else
+             (Is_Bit_Packed_Array (Typ)
+               and then
+                 Is_Modular_Integer_Type (Packed_Array_Type (Typ)))
+         then
+            null;
+
+         --  Otherwise, we require the address clause to be constant
+         --  because the call to the initialization procedure (or the
+         --  attach code) has to happen at the point of the declaration.
+
+         else
+            Check_Constant_Address_Clause (Expr, E);
+            Set_Has_Delayed_Freeze (E, False);
+         end if;
+
+         if not Error_Posted (Expr)
+           and then not Controlled_Type (Typ)
+         then
+            Warn_Overlay (Expr, Typ, Name (Addr));
+         end if;
+      end if;
+   end Check_Address_Clause;
+
    -----------------------------
    -- Check_Compile_Time_Size --
    -----------------------------
 
    procedure Check_Compile_Time_Size (T : Entity_Id) is
 
-      procedure Set_Small_Size (S : Uint);
+      procedure Set_Small_Size (T : Entity_Id; S : Uint);
       --  Sets the compile time known size (32 bits or less) in the Esize
-      --  field, checking for a size clause that was given which attempts
+      --  field, of T checking for a size clause that was given which attempts
       --  to give a smaller size.
 
       function Size_Known (T : Entity_Id) return Boolean;
-      --  Recursive function that does all the work.
-      --  Is this right??? isn't recursive case already handled???
-      --  certainly yes for normal call, but what about bogus sem_res call???
+      --  Recursive function that does all the work
 
       function Static_Discriminated_Components (T : Entity_Id) return Boolean;
       --  If T is a constrained subtype, its size is not known if any of its
       --  discriminant constraints is not static and it is not a null record.
-      --  The test is conservative  and doesn't check that the components are
+      --  The test is conservative and doesn't check that the components are
       --  in fact constrained by non-static discriminant values. Could be made
       --  more precise ???
 
@@ -429,7 +528,7 @@ package body Freeze is
       -- Set_Small_Size --
       --------------------
 
-      procedure Set_Small_Size (S : Uint) is
+      procedure Set_Small_Size (T : Entity_Id; S : Uint) is
       begin
          if S > 32 then
             return;
@@ -473,23 +572,26 @@ package body Freeze is
          if Size_Known_At_Compile_Time (T) then
             return True;
 
-         elsif Error_Posted (T) then
-            return False;
-
          elsif Is_Scalar_Type (T)
            or else Is_Task_Type (T)
          then
             return not Is_Generic_Type (T);
 
          elsif Is_Array_Type (T) then
-
             if Ekind (T) = E_String_Literal_Subtype then
-               Set_Small_Size (Component_Size (T) * String_Literal_Length (T));
+               Set_Small_Size (T, Component_Size (T)
+                               * String_Literal_Length (T));
                return True;
 
             elsif not Is_Constrained (T) then
                return False;
 
+            --  Don't do any recursion on type with error posted, since
+            --  we may have a malformed type that leads us into a loop
+
+            elsif Error_Posted (T) then
+               return False;
+
             elsif not Size_Known (Component_Type (T)) then
                return False;
             end if;
@@ -503,7 +605,6 @@ package body Freeze is
 
             begin
                Index := First_Index (T);
-
                while Present (Index) loop
                   if Nkind (Index) = N_Range then
                      Get_Index_Bounds (Index, Low, High);
@@ -535,7 +636,7 @@ package body Freeze is
                   Next_Index (Index);
                end loop;
 
-               Set_Small_Size (Esiz);
+               Set_Small_Size (T, Esiz);
                return True;
             end;
 
@@ -546,105 +647,220 @@ package body Freeze is
            and then not Is_Generic_Type (T)
            and then Present (Underlying_Type (T))
          then
-            return Size_Known (Underlying_Type (T));
+            --  Don't do any recursion on type with error posted, since
+            --  we may have a malformed type that leads us into a loop
+
+            if Error_Posted (T) then
+               return False;
+            else
+               return Size_Known (Underlying_Type (T));
+            end if;
 
          elsif Is_Record_Type (T) then
+
+            --  A class-wide type is never considered to have a known size
+
             if Is_Class_Wide_Type (T) then
                return False;
 
-            elsif T /= Base_Type (T) then
-               return Size_Known_At_Compile_Time (Base_Type (T))
-                 and then Static_Discriminated_Components (T);
+            --  A subtype of a variant record must not have non-static
+            --  discriminanted components.
 
-            else
-               declare
-                  Packed_Size_Known : Boolean := Is_Packed (T);
-                  Packed_Size       : Uint    := Uint_0;
+            elsif T /= Base_Type (T)
+              and then not Static_Discriminated_Components (T)
+            then
+               return False;
 
-               begin
-                  --  Test for variant part present
-
-                  if Has_Discriminants (T)
-                    and then Present (Parent (T))
-                    and then Nkind (Parent (T)) = N_Full_Type_Declaration
-                    and then Nkind (Type_Definition (Parent (T))) =
-                               N_Record_Definition
-                    and then not Null_Present (Type_Definition (Parent (T)))
-                    and then Present (Variant_Part
-                               (Component_List (Type_Definition (Parent (T)))))
+            --  Don't do any recursion on type with error posted, since
+            --  we may have a malformed type that leads us into a loop
+
+            elsif Error_Posted (T) then
+               return False;
+            end if;
+
+            --  Now look at the components of the record
+
+            declare
+               --  The following two variables are used to keep track of
+               --  the size of packed records if we can tell the size of
+               --  the packed record in the front end. Packed_Size_Known
+               --  is True if so far we can figure out the size. It is
+               --  initialized to True for a packed record, unless the
+               --  record has discriminants. The reason we eliminate the
+               --  discriminated case is that we don't know the way the
+               --  back end lays out discriminated packed records. If
+               --  Packed_Size_Known is True, then Packed_Size is the
+               --  size in bits so far.
+
+               Packed_Size_Known : Boolean :=
+                                     Is_Packed (T)
+                                       and then not Has_Discriminants (T);
+
+               Packed_Size : Uint := Uint_0;
+
+            begin
+               --  Test for variant part present
+
+               if Has_Discriminants (T)
+                 and then Present (Parent (T))
+                 and then Nkind (Parent (T)) = N_Full_Type_Declaration
+                 and then Nkind (Type_Definition (Parent (T))) =
+                            N_Record_Definition
+                 and then not Null_Present (Type_Definition (Parent (T)))
+                 and then Present (Variant_Part
+                            (Component_List (Type_Definition (Parent (T)))))
+               then
+                  --  If variant part is present, and type is unconstrained,
+                  --  then we must have defaulted discriminants, or a size
+                  --  clause must be present for the type, or else the size
+                  --  is definitely not known at compile time.
+
+                  if not Is_Constrained (T)
+                    and then
+                      No (Discriminant_Default_Value
+                           (First_Discriminant (T)))
+                    and then Unknown_Esize (T)
                   then
-                     --  If variant part is present, and type is unconstrained,
-                     --  then we must have defaulted discriminants, or a size
-                     --  clause must be present for the type, or else the size
-                     --  is definitely not known at compile time.
-
-                     if not Is_Constrained (T)
-                       and then
-                         No (Discriminant_Default_Value
-                              (First_Discriminant (T)))
-                       and then Unknown_Esize (T)
-                     then
-                        return False;
-                     else
-                        --  We do not know the packed size, it is too much
-                        --  trouble to figure it out.
+                     return False;
+                  end if;
+               end if;
 
-                        Packed_Size_Known := False;
-                     end if;
+               --  Loop through components
+
+               Comp := First_Component_Or_Discriminant (T);
+               while Present (Comp) loop
+                  Ctyp := Etype (Comp);
+
+                  --  We do not know the packed size if there is a component
+                  --  clause present (we possibly could, but this would only
+                  --  help in the case of a record with partial rep clauses.
+                  --  That's because in the case of full rep clauses, the
+                  --  size gets figured out anyway by a different circuit).
+
+                  if Present (Component_Clause (Comp)) then
+                     Packed_Size_Known := False;
                   end if;
 
-                  Comp := First_Entity (T);
+                  --  We need to identify a component that is an array where
+                  --  the index type is an enumeration type with non-standard
+                  --  representation, and some bound of the type depends on a
+                  --  discriminant.
 
-                  while Present (Comp) loop
-                     if Ekind (Comp) = E_Component
-                          or else
-                        Ekind (Comp) = E_Discriminant
-                     then
-                        Ctyp := Etype (Comp);
+                  --  This is because gigi computes the size by doing a
+                  --  substituation of the appropriate discriminant value in
+                  --  the size expression for the base type, and gigi is not
+                  --  clever enough to evaluate the resulting expression (which
+                  --  involves a call to rep_to_pos) at compile time.
 
-                        if Present (Component_Clause (Comp)) then
-                           Packed_Size_Known := False;
-                        end if;
+                  --  It would be nice if gigi would either recognize that
+                  --  this expression can be computed at compile time, or
+                  --  alternatively figured out the size from the subtype
+                  --  directly, where all the information is at hand ???
+
+                  if Is_Array_Type (Etype (Comp))
+                    and then Present (Packed_Array_Type (Etype (Comp)))
+                  then
+                     declare
+                        Ocomp  : constant Entity_Id :=
+                                   Original_Record_Component (Comp);
+                        OCtyp  : constant Entity_Id := Etype (Ocomp);
+                        Ind    : Node_Id;
+                        Indtyp : Entity_Id;
+                        Lo, Hi : Node_Id;
+
+                     begin
+                        Ind := First_Index (OCtyp);
+                        while Present (Ind) loop
+                           Indtyp := Etype (Ind);
+
+                           if Is_Enumeration_Type (Indtyp)
+                             and then Has_Non_Standard_Rep (Indtyp)
+                           then
+                              Lo := Type_Low_Bound  (Indtyp);
+                              Hi := Type_High_Bound (Indtyp);
+
+                              if Is_Entity_Name (Lo)
+                                and then Ekind (Entity (Lo)) = E_Discriminant
+                              then
+                                 return False;
+
+                              elsif Is_Entity_Name (Hi)
+                                and then Ekind (Entity (Hi)) = E_Discriminant
+                              then
+                                 return False;
+                              end if;
+                           end if;
 
-                        if not Size_Known (Ctyp) then
-                           return False;
+                           Next_Index (Ind);
+                        end loop;
+                     end;
+                  end if;
 
-                        elsif Packed_Size_Known then
+                  --  Clearly size of record is not known if the size of
+                  --  one of the components is not known.
 
-                           --  If RM_Size is known and static, then we can
-                           --  keep accumulating the packed size.
+                  if not Size_Known (Ctyp) then
+                     return False;
+                  end if;
 
-                           if Known_Static_RM_Size (Ctyp) then
+                  --  Accumulate packed size if possible
 
-                              --  A little glitch, to be removed sometime ???
-                              --  gigi does not understand zero sizes yet.
+                  if Packed_Size_Known then
 
-                              if RM_Size (Ctyp) = Uint_0 then
-                                 Packed_Size_Known := False;
-                              end if;
+                     --  We can only deal with elementary types, since for
+                     --  non-elementary components, alignment enters into the
+                     --  picture, and we don't know enough to handle proper
+                     --  alignment in this context. Packed arrays count as
+                     --  elementary if the representation is a modular type.
+
+                     if Is_Elementary_Type (Ctyp)
+                       or else (Is_Array_Type (Ctyp)
+                                and then Present (Packed_Array_Type (Ctyp))
+                                and then Is_Modular_Integer_Type
+                                           (Packed_Array_Type (Ctyp)))
+                     then
+                        --  If RM_Size is known and static, then we can
+                        --  keep accumulating the packed size.
 
-                              Packed_Size :=
-                                Packed_Size + RM_Size (Ctyp);
+                        if Known_Static_RM_Size (Ctyp) then
 
-                           --  If we have a field whose RM_Size is not known
-                           --  then we can't figure out the packed size here.
+                           --  A little glitch, to be removed sometime ???
+                           --  gigi does not understand zero sizes yet.
 
-                           else
+                           if RM_Size (Ctyp) = Uint_0 then
                               Packed_Size_Known := False;
+
+                           --  Normal case where we can keep accumulating the
+                           --  packed array size.
+
+                           else
+                              Packed_Size := Packed_Size + RM_Size (Ctyp);
                            end if;
+
+                        --  If we have a field whose RM_Size is not known then
+                        --  we can't figure out the packed size here.
+
+                        else
+                           Packed_Size_Known := False;
                         end if;
-                     end if;
 
-                     Next_Entity (Comp);
-                  end loop;
+                     --  If we have a non-elementary type we can't figure out
+                     --  the packed array size (alignment issues).
 
-                  if Packed_Size_Known then
-                     Set_Small_Size (Packed_Size);
+                     else
+                        Packed_Size_Known := False;
+                     end if;
                   end if;
 
-                  return True;
-               end;
-            end if;
+                  Next_Component_Or_Discriminant (Comp);
+               end loop;
+
+               if Packed_Size_Known then
+                  Set_Small_Size (T, Packed_Size);
+               end if;
+
+               return True;
+            end;
 
          else
             return False;
@@ -656,8 +872,7 @@ package body Freeze is
       -------------------------------------
 
       function Static_Discriminated_Components
-        (T    : Entity_Id)
-         return Boolean
+        (T : Entity_Id) return Boolean
       is
          Constraint : Elmt_Id;
 
@@ -667,7 +882,6 @@ package body Freeze is
            and then Present (First_Component (T))
          then
             Constraint := First_Elmt (Discriminant_Constraint (T));
-
             while Present (Constraint) loop
                if not Compile_Time_Known_Value (Node (Constraint)) then
                   return False;
@@ -728,7 +942,7 @@ package body Freeze is
          while Present (Comp) loop
             if not Is_Type (Comp)
               and then (Strict_Alignment (Etype (Comp))
-                        or else Is_Aliased (Comp))
+                         or else Is_Aliased (Comp))
             then
                Set_Strict_Alignment (E);
                return;
@@ -814,12 +1028,51 @@ package body Freeze is
                end if;
 
                return;
-
             end if;
          end if;
       end loop;
    end Check_Unsigned_Type;
 
+   -----------------------------
+   -- Expand_Atomic_Aggregate --
+   -----------------------------
+
+   procedure Expand_Atomic_Aggregate (E : Entity_Id; Typ : Entity_Id) is
+      Loc   : constant Source_Ptr := Sloc (E);
+      New_N : Node_Id;
+      Temp  : Entity_Id;
+
+   begin
+      if (Nkind (Parent (E)) = N_Object_Declaration
+            or else Nkind (Parent (E)) = N_Assignment_Statement)
+        and then Comes_From_Source (Parent (E))
+        and then Nkind (E) = N_Aggregate
+      then
+         Temp :=
+           Make_Defining_Identifier (Loc,
+             New_Internal_Name ('T'));
+
+         New_N :=
+           Make_Object_Declaration (Loc,
+             Defining_Identifier => Temp,
+             Object_definition => New_Occurrence_Of (Typ, Loc),
+             Expression => Relocate_Node (E));
+         Insert_Before (Parent (E), New_N);
+         Analyze (New_N);
+
+         Set_Expression (Parent (E), New_Occurrence_Of (Temp, Loc));
+
+         --  To prevent the temporary from being constant-folded (which
+         --  would lead to the same piecemeal assignment on the original
+         --  target) indicate to the back-end that the temporary  is a
+         --  variable with real storage. See description of this flag
+         --  in Einfo, and the notes on N_Assignment_Statement and
+         --  N_Object_Declaration in Sinfo.
+
+         Set_Is_True_Constant (Temp, False);
+      end if;
+   end Expand_Atomic_Aggregate;
+
    ----------------
    -- Freeze_All --
    ----------------
@@ -842,6 +1095,10 @@ package body Freeze is
       --  should not be recursive, we don't want to analyze those till
       --  we are sure that ALL the types are frozen).
 
+      --------------------
+      -- Freeze_All_Ent --
+      --------------------
+
       procedure Freeze_All_Ent
         (From  : Entity_Id;
          After : in out Node_Id)
@@ -854,6 +1111,10 @@ package body Freeze is
          --  If freeze nodes are present, insert and analyze, and reset
          --  cursor for next insertion.
 
+         -------------------
+         -- Process_Flist --
+         -------------------
+
          procedure Process_Flist is
          begin
             if Is_Non_Empty_List (Flist) then
@@ -868,6 +1129,8 @@ package body Freeze is
             end if;
          end Process_Flist;
 
+      --  Start or processing for Freeze_All_Ent
+
       begin
          E := From;
          while Present (E) loop
@@ -896,7 +1159,7 @@ package body Freeze is
             elsif Ekind (E) in Task_Kind
               and then
                 (Nkind (Parent (E)) = N_Task_Type_Declaration
-                  or else
+                   or else
                  Nkind (Parent (E)) = N_Single_Task_Declaration)
             then
                New_Scope (E);
@@ -916,8 +1179,9 @@ package body Freeze is
                declare
                   Prim_List : constant Elist_Id :=
                                Primitive_Operations (Etype (E));
-                  Prim      : Elmt_Id;
-                  Subp      : Entity_Id;
+
+                  Prim : Elmt_Id;
+                  Subp : Entity_Id;
 
                begin
                   Prim  := First_Elmt (Prim_List);
@@ -942,6 +1206,38 @@ package body Freeze is
                Process_Flist;
             end if;
 
+            --  If an incomplete type is still not frozen, this may be
+            --  a premature freezing because of a body declaration that
+            --  follows. Indicate where the freezing took place.
+
+            --  If the freezing is caused by the end of the current
+            --  declarative part, it is a Taft Amendment type, and there
+            --  is no error.
+
+            if not Is_Frozen (E)
+              and then Ekind (E) = E_Incomplete_Type
+            then
+               declare
+                  Bod : constant Node_Id := Next (After);
+
+               begin
+                  if (Nkind (Bod) = N_Subprogram_Body
+                        or else Nkind (Bod) = N_Entry_Body
+                        or else Nkind (Bod) = N_Package_Body
+                        or else Nkind (Bod) = N_Protected_Body
+                        or else Nkind (Bod) = N_Task_Body
+                        or else Nkind (Bod) in N_Body_Stub)
+                     and then
+                       List_Containing (After) = List_Containing (Parent (E))
+                  then
+                     Error_Msg_Sloc := Sloc (Next (After));
+                     Error_Msg_NE
+                       ("type& is frozen# before its full declaration",
+                         Parent (E), E);
+                  end if;
+               end;
+            end if;
+
             Next_Entity (E);
          end loop;
       end Freeze_All_Ent;
@@ -955,6 +1251,7 @@ package body Freeze is
       --  that require us to build a default expression functions. This is the
       --  point at which such functions are constructed (after all types that
       --  might be used in such expressions have been frozen).
+
       --  We also add finalization chains to access types whose designated
       --  types are controlled. This is normally done when freezing the type,
       --  but this misses recursive type definitions where the later members
@@ -964,7 +1261,6 @@ package body Freeze is
 
       E := From;
       while Present (E) loop
-
          if Is_Subprogram (E) then
 
             if not Default_Expressions_Processed (E) then
@@ -981,7 +1277,7 @@ package body Freeze is
                  and then Present (Corresponding_Body (Decl))
                  and then
                    Nkind (Unit_Declaration_Node (Corresponding_Body (Decl)))
-                   = N_Subprogram_Renaming_Declaration
+                                          = N_Subprogram_Renaming_Declaration
                then
                   Build_And_Analyze_Renamed_Body
                     (Decl, Corresponding_Body (Decl), After);
@@ -991,12 +1287,11 @@ package body Freeze is
          elsif Ekind (E) in Task_Kind
            and then
              (Nkind (Parent (E)) = N_Task_Type_Declaration
-               or else
+                or else
               Nkind (Parent (E)) = N_Single_Task_Declaration)
          then
             declare
                Ent : Entity_Id;
-
             begin
                Ent := First_Entity (E);
 
@@ -1023,7 +1318,6 @@ package body Freeze is
 
          Next_Entity (E);
       end loop;
-
    end Freeze_All;
 
    -----------------------
@@ -1036,7 +1330,6 @@ package body Freeze is
       Result : in out List_Id)
    is
       L : constant List_Id := Freeze_Entity (Ent, Loc);
-
    begin
       if Is_Non_Empty_List (L) then
          if Result = No_List then
@@ -1053,15 +1346,9 @@ package body Freeze is
 
    procedure Freeze_Before (N : Node_Id; T : Entity_Id) is
       Freeze_Nodes : constant List_Id := Freeze_Entity (T, Sloc (N));
-      F            : Node_Id;
-
    begin
       if Is_Non_Empty_List (Freeze_Nodes) then
-         F := First (Freeze_Nodes);
-
-         if Present (F) then
-            Insert_Actions (N, Freeze_Nodes);
-         end if;
+         Insert_Actions (N, Freeze_Nodes);
       end if;
    end Freeze_Before;
 
@@ -1070,6 +1357,7 @@ package body Freeze is
    -------------------
 
    function Freeze_Entity (E : Entity_Id; Loc : Source_Ptr) return List_Id is
+      Test_E : Entity_Id := E;
       Comp   : Entity_Id;
       F_Node : Node_Id;
       Result : List_Id;
@@ -1078,24 +1366,24 @@ package body Freeze is
       Atype  : Entity_Id;
 
       procedure Check_Current_Instance (Comp_Decl : Node_Id);
-      --  Check that an Access or Unchecked_Access attribute with
-      --  a prefix which is the current instance type can only be
-      --  applied when the type is limited.
+      --  Check that an Access or Unchecked_Access attribute with a prefix
+      --  which is the current instance type can only be applied when the type
+      --  is limited.
 
       function After_Last_Declaration return Boolean;
       --  If Loc is a freeze_entity that appears after the last declaration
       --  in the scope, inhibit error messages on late completion.
 
       procedure Freeze_Record_Type (Rec : Entity_Id);
-      --  Freeze each component, handle some representation clauses, and
-      --  freeze primitive operations if this is a tagged type.
+      --  Freeze each component, handle some representation clauses, and freeze
+      --  primitive operations if this is a tagged type.
 
       ----------------------------
       -- After_Last_Declaration --
       ----------------------------
 
       function After_Last_Declaration return Boolean is
-         Spec  : Node_Id := Parent (Current_Scope);
+         Spec  : constant Node_Id := Parent (Current_Scope);
 
       begin
          if Nkind (Spec) = N_Package_Specification then
@@ -1120,7 +1408,11 @@ package body Freeze is
       procedure Check_Current_Instance (Comp_Decl : Node_Id) is
 
          function Process (N : Node_Id) return Traverse_Result;
-         --  Process routine to apply check to given node.
+         --  Process routine to apply check to given node
+
+         -------------
+         -- Process --
+         -------------
 
          function Process (N : Node_Id) return Traverse_Result is
          begin
@@ -1158,8 +1450,10 @@ package body Freeze is
 
       procedure Freeze_Record_Type (Rec : Entity_Id) is
          Comp : Entity_Id;
+         IR   : Node_Id;
          Junk : Boolean;
          ADC  : Node_Id;
+         Prev : Entity_Id;
 
          Unplaced_Component : Boolean := False;
          --  Set True if we find at least one component with no component
@@ -1169,95 +1463,126 @@ package body Freeze is
          --  Set True if we find at least one component with a component
          --  clause (used to warn about useless Bit_Order pragmas).
 
-      begin
-         --  Freeze components and embedded subtypes
+         procedure Check_Itype (Desig : Entity_Id);
+         --  If the component subtype is an access to a constrained subtype
+         --  of an already frozen type, make the subtype frozen as well. It
+         --  might otherwise be frozen in the wrong scope, and a freeze node
+         --  on subtype has no effect.
 
-         Comp := First_Entity (Rec);
+         -----------------
+         -- Check_Itype --
+         -----------------
 
-         while Present (Comp) loop
-
-            if not Is_Type (Comp) then
-               Freeze_And_Append (Etype (Comp), Loc, Result);
-            end if;
-
-            --  If the component is an access type with an allocator
-            --  as default value, the designated type will be frozen
-            --  by the corresponding expression in init_proc. In  order
-            --  to place the freeze node for the designated type before
-            --  that for the current record type, freeze it now.
-
-            --  Same process if the component is an array of access types,
-            --  initialized with an aggregate. If the designated type is
-            --  private, it cannot contain allocators, and it is premature
-            --  to freeze the type, so we check for this as well.
-
-            if Is_Access_Type (Etype (Comp))
-              and then Present (Parent (Comp))
-              and then Present (Expression (Parent (Comp)))
-              and then Nkind (Expression (Parent (Comp))) = N_Allocator
+         procedure Check_Itype (Desig : Entity_Id) is
+         begin
+            if not Is_Frozen (Desig)
+              and then Is_Frozen (Base_Type (Desig))
             then
-               declare
-                  Alloc : constant Node_Id := Expression (Parent (Comp));
+               Set_Is_Frozen (Desig);
 
-               begin
-                  --  If component is pointer to a classwide type, freeze
-                  --  the specific type in the expression being allocated.
-                  --  The expression may be a subtype indication, in which
-                  --  case freeze the subtype mark.
+               --  In addition, add an Itype_Reference to ensure that the
+               --  access subtype is elaborated early enough. This cannot
+               --  be done if the subtype may depend on discriminants.
 
-                  if Is_Class_Wide_Type (Designated_Type (Etype (Comp))) then
+               if Ekind (Comp) = E_Component
+                 and then Is_Itype (Etype (Comp))
+                 and then not Has_Discriminants (Rec)
+               then
+                  IR := Make_Itype_Reference (Sloc (Comp));
+                  Set_Itype (IR, Desig);
 
-                     if Is_Entity_Name (Expression (Alloc)) then
-                        Freeze_And_Append
-                          (Entity (Expression (Alloc)), Loc, Result);
-                     elsif
-                       Nkind (Expression (Alloc)) = N_Subtype_Indication
-                     then
-                        Freeze_And_Append
-                         (Entity (Subtype_Mark (Expression (Alloc))),
-                           Loc, Result);
-                     end if;
+                  if No (Result) then
+                     Result := New_List (IR);
                   else
-                     Freeze_And_Append
-                       (Designated_Type (Etype (Comp)), Loc, Result);
+                     Append (IR, Result);
                   end if;
-               end;
+               end if;
+            end if;
+         end Check_Itype;
 
-            elsif Is_Array_Type (Etype (Comp))
-              and then Is_Access_Type (Component_Type (Etype (Comp)))
-              and then Present (Parent (Comp))
-              and then Nkind (Parent (Comp)) = N_Component_Declaration
-              and then Present (Expression (Parent (Comp)))
-              and then Nkind (Expression (Parent (Comp))) = N_Aggregate
-              and then Is_Fully_Defined
-                 (Designated_Type (Component_Type (Etype (Comp))))
+      --  Start of processing for Freeze_Record_Type
+
+      begin
+         --  If this is a subtype of a controlled type, declared without
+         --  a constraint, the _controller may not appear in the component
+         --  list if the parent was not frozen at the point of subtype
+         --  declaration. Inherit the _controller component now.
+
+         if Rec /= Base_Type (Rec)
+           and then Has_Controlled_Component (Rec)
+         then
+            if Nkind (Parent (Rec)) = N_Subtype_Declaration
+              and then Is_Entity_Name (Subtype_Indication (Parent (Rec)))
             then
-               Freeze_And_Append
-                 (Designated_Type
-                   (Component_Type (Etype (Comp))), Loc, Result);
+               Set_First_Entity (Rec, First_Entity (Base_Type (Rec)));
+
+            --  If this is an internal type without a declaration, as for
+            --  record component, the base type may not yet be frozen, and its
+            --  controller has not been created. Add an explicit freeze node
+            --  for the itype, so it will be frozen after the base type. This
+            --  freeze node is used to communicate with the expander, in order
+            --  to create the controller for the enclosing record, and it is
+            --  deleted afterwards (see exp_ch3). It must not be created when
+            --  expansion is off, because it might appear in the wrong context
+            --  for the back end.
+
+            elsif Is_Itype (Rec)
+              and then Has_Delayed_Freeze (Base_Type (Rec))
+              and then
+                Nkind (Associated_Node_For_Itype (Rec)) =
+                                                     N_Component_Declaration
+              and then Expander_Active
+            then
+               Ensure_Freeze_Node (Rec);
             end if;
+         end if;
 
-            --  Processing for real components (exclude anonymous subtypes)
+         --  Freeze components and embedded subtypes
+
+         Comp := First_Entity (Rec);
+         Prev := Empty;
+         while Present (Comp) loop
+
+            --  First handle the (real) component case
 
             if Ekind (Comp) = E_Component
               or else Ekind (Comp) = E_Discriminant
             then
-               --  Check for error of component clause given for variable
-               --  sized type. We have to delay this test till this point,
-               --  since the component type has to be frozen for us to know
-               --  if it is variable length. We omit this test in a generic
-               --  context, it will be applied at instantiation time.
-
                declare
                   CC : constant Node_Id := Component_Clause (Comp);
 
                begin
+                  --  Freezing a record type freezes the type of each of its
+                  --  components. However, if the type of the component is
+                  --  part of this record, we do not want or need a separate
+                  --  Freeze_Node. Note that Is_Itype is wrong because that's
+                  --  also set in private type cases. We also can't check for
+                  --  the Scope being exactly Rec because of private types and
+                  --  record extensions.
+
+                  if Is_Itype (Etype (Comp))
+                    and then Is_Record_Type (Underlying_Type
+                                             (Scope (Etype (Comp))))
+                  then
+                     Undelay_Type (Etype (Comp));
+                  end if;
+
+                  Freeze_And_Append (Etype (Comp), Loc, Result);
+
+                  --  Check for error of component clause given for variable
+                  --  sized type. We have to delay this test till this point,
+                  --  since the component type has to be frozen for us to know
+                  --  if it is variable length. We omit this test in a generic
+                  --  context, it will be applied at instantiation time.
+
                   if Present (CC) then
                      Placed_Component := True;
 
-                     if not Size_Known_At_Compile_Time
+                     if Inside_A_Generic then
+                        null;
+
+                     elsif not Size_Known_At_Compile_Time
                               (Underlying_Type (Etype (Comp)))
-                       and then not Inside_A_Generic
                      then
                         Error_Msg_N
                           ("component clause not allowed for variable " &
@@ -1267,159 +1592,336 @@ package body Freeze is
                   else
                      Unplaced_Component := True;
                   end if;
-               end;
 
-               --  If component clause is present, then deal with the
-               --  non-default bit order case. We cannot do this before
-               --  the freeze point, because there is no required order
-               --  for the component clause and the bit_order clause.
+                  --  Case of component requires byte alignment
 
-               --  We only do this processing for the base type, and in
-               --  fact that's important, since otherwise if there are
-               --  record subtypes, we could reverse the bits once for
-               --  each subtype, which would be incorrect.
+                  if Must_Be_On_Byte_Boundary (Etype (Comp)) then
 
-               if Present (Component_Clause (Comp))
-                 and then Reverse_Bit_Order (Rec)
-                 and then Ekind (E) = E_Record_Type
-               then
-                  declare
-                     CFB : constant Uint    := Component_Bit_Offset (Comp);
-                     CSZ : constant Uint    := Esize (Comp);
-                     CLC : constant Node_Id := Component_Clause (Comp);
-                     Pos : constant Node_Id := Position (CLC);
-                     FB  : constant Node_Id := First_Bit (CLC);
+                     --  Set the enclosing record to also require byte align
 
-                     Storage_Unit_Offset : constant Uint :=
-                                             CFB / System_Storage_Unit;
+                     Set_Must_Be_On_Byte_Boundary (Rec);
 
-                     Start_Bit : constant Uint :=
-                                   CFB mod System_Storage_Unit;
+                     --  Check for component clause that is inconsistent
+                     --  with the required byte boundary alignment.
 
-                  begin
-                     --  Cases where field goes over storage unit boundary
+                     if Present (CC)
+                       and then Normalized_First_Bit (Comp) mod
+                                  System_Storage_Unit /= 0
+                     then
+                        Error_Msg_N
+                          ("component & must be byte aligned",
+                           Component_Name (Component_Clause (Comp)));
+                     end if;
+                  end if;
 
-                     if Start_Bit + CSZ > System_Storage_Unit then
+                  --  If component clause is present, then deal with the
+                  --  non-default bit order case for Ada 95 mode. The required
+                  --  processing for Ada 2005 mode is handled separately after
+                  --  processing all components.
 
-                        --  Allow multi-byte field but generate warning
+                  --  We only do this processing for the base type, and in
+                  --  fact that's important, since otherwise if there are
+                  --  record subtypes, we could reverse the bits once for
+                  --  each subtype, which would be incorrect.
 
-                        if Start_Bit mod System_Storage_Unit = 0
-                          and then CSZ mod System_Storage_Unit = 0
-                        then
-                           Error_Msg_N
-                             ("multi-byte field specified with non-standard"
-                                & " Bit_Order?", CLC);
+                  if Present (CC)
+                    and then Reverse_Bit_Order (Rec)
+                    and then Ekind (E) = E_Record_Type
+                    and then Ada_Version <= Ada_95
+                  then
+                     declare
+                        CFB : constant Uint    := Component_Bit_Offset (Comp);
+                        CSZ : constant Uint    := Esize (Comp);
+                        CLC : constant Node_Id := Component_Clause (Comp);
+                        Pos : constant Node_Id := Position (CLC);
+                        FB  : constant Node_Id := First_Bit (CLC);
+
+                        Storage_Unit_Offset : constant Uint :=
+                                                CFB / System_Storage_Unit;
 
-                           if Bytes_Big_Endian then
+                        Start_Bit : constant Uint :=
+                                      CFB mod System_Storage_Unit;
+
+                     begin
+                        --  Cases where field goes over storage unit boundary
+
+                        if Start_Bit + CSZ > System_Storage_Unit then
+
+                           --  Allow multi-byte field but generate warning
+
+                           if Start_Bit mod System_Storage_Unit = 0
+                             and then CSZ mod System_Storage_Unit = 0
+                           then
                               Error_Msg_N
-                                ("bytes are not reversed "
-                                   & "(component is big-endian)?", CLC);
+                                ("multi-byte field specified with non-standard"
+                                 & " Bit_Order?", CLC);
+
+                              if Bytes_Big_Endian then
+                                 Error_Msg_N
+                                   ("bytes are not reversed "
+                                    & "(component is big-endian)?", CLC);
+                              else
+                                 Error_Msg_N
+                                   ("bytes are not reversed "
+                                    & "(component is little-endian)?", CLC);
+                              end if;
+
+                              --  Do not allow non-contiguous field
+
                            else
                               Error_Msg_N
-                                ("bytes are not reversed "
-                                   & "(component is little-endian)?", CLC);
+                                ("attempt to specify non-contiguous field"
+                                 & " not permitted", CLC);
+                              Error_Msg_N
+                                ("\(caused by non-standard Bit_Order "
+                                 & "specified)", CLC);
                            end if;
 
-                        --  Do not allow non-contiguous field
+                           --  Case where field fits in one storage unit
 
                         else
-                           Error_Msg_N
-                             ("attempt to specify non-contiguous field"
-                                & " not permitted", CLC);
-                           Error_Msg_N
-                             ("\(caused by non-standard Bit_Order "
-                                & "specified)", CLC);
-                        end if;
+                           --  Give warning if suspicious component clause
 
-                     --  Case where field fits in one storage unit
+                           if Intval (FB) >= System_Storage_Unit
+                             and then Warn_On_Reverse_Bit_Order
+                           then
+                              Error_Msg_N
+                                ("?Bit_Order clause does not affect " &
+                                 "byte ordering", Pos);
+                              Error_Msg_Uint_1 :=
+                                Intval (Pos) + Intval (FB) /
+                                  System_Storage_Unit;
+                              Error_Msg_N
+                                ("?position normalized to ^ before bit " &
+                                 "order interpreted", Pos);
+                           end if;
 
-                     else
-                        --  Give warning if suspicious component clause
+                           --  Here is where we fix up the Component_Bit_Offset
+                           --  value to account for the reverse bit order.
+                           --  Some examples of what needs to be done are:
 
-                        if Intval (FB) >= System_Storage_Unit then
-                           Error_Msg_N
-                             ("?Bit_Order clause does not affect " &
-                              "byte ordering", Pos);
-                           Error_Msg_Uint_1 :=
-                             Intval (Pos) + Intval (FB) / System_Storage_Unit;
-                           Error_Msg_N
-                             ("?position normalized to ^ before bit " &
-                              "order interpreted", Pos);
+                           --    First_Bit .. Last_Bit     Component_Bit_Offset
+                           --      old          new          old       new
+
+                           --     0 .. 0       7 .. 7         0         7
+                           --     0 .. 1       6 .. 7         0         6
+                           --     0 .. 2       5 .. 7         0         5
+                           --     0 .. 7       0 .. 7         0         4
+
+                           --     1 .. 1       6 .. 6         1         6
+                           --     1 .. 4       3 .. 6         1         3
+                           --     4 .. 7       0 .. 3         4         0
+
+                           --  The general rule is that the first bit is
+                           --  is obtained by subtracting the old ending bit
+                           --  from storage_unit - 1.
+
+                           Set_Component_Bit_Offset
+                             (Comp,
+                              (Storage_Unit_Offset * System_Storage_Unit) +
+                                (System_Storage_Unit - 1) -
+                                  (Start_Bit + CSZ - 1));
+
+                           Set_Normalized_First_Bit
+                             (Comp,
+                                Component_Bit_Offset (Comp) mod
+                                  System_Storage_Unit);
                         end if;
+                     end;
+                  end if;
+               end;
+            end if;
+
+            --  If the component is an Itype with Delayed_Freeze and is either
+            --  a record or array subtype and its base type has not yet been
+            --  frozen, we must remove this from the entity list of this
+            --  record and put it on the entity list of the scope of its base
+            --  type. Note that we know that this is not the type of a
+            --  component since we cleared Has_Delayed_Freeze for it in the
+            --  previous loop. Thus this must be the Designated_Type of an
+            --  access type, which is the type of a component.
+
+            if Is_Itype (Comp)
+              and then Is_Type (Scope (Comp))
+              and then Is_Composite_Type (Comp)
+              and then Base_Type (Comp) /= Comp
+              and then Has_Delayed_Freeze (Comp)
+              and then not Is_Frozen (Base_Type (Comp))
+            then
+               declare
+                  Will_Be_Frozen : Boolean := False;
+                  S : Entity_Id := Scope (Rec);
+
+               begin
+                  --  We have a pretty bad kludge here. Suppose Rec is subtype
+                  --  being defined in a subprogram that's created as part of
+                  --  the freezing of Rec'Base. In that case, we know that
+                  --  Comp'Base must have already been frozen by the time we
+                  --  get to elaborate this because Gigi doesn't elaborate any
+                  --  bodies until it has elaborated all of the declarative
+                  --  part. But Is_Frozen will not be set at this point because
+                  --  we are processing code in lexical order.
+
+                  --  We detect this case by going up the Scope chain of Rec
+                  --  and seeing if we have a subprogram scope before reaching
+                  --  the top of the scope chain or that of Comp'Base. If we
+                  --  do, then mark that Comp'Base will actually be frozen. If
+                  --  so, we merely undelay it.
+
+                  while Present (S) loop
+                     if Is_Subprogram (S) then
+                        Will_Be_Frozen := True;
+                        exit;
+                     elsif S = Scope (Base_Type (Comp)) then
+                        exit;
+                     end if;
+
+                     S := Scope (S);
+                  end loop;
+
+                  if Will_Be_Frozen then
+                     Undelay_Type (Comp);
+                  else
+                     if Present (Prev) then
+                        Set_Next_Entity (Prev, Next_Entity (Comp));
+                     else
+                        Set_First_Entity (Rec, Next_Entity (Comp));
+                     end if;
 
-                        --  Here is where we fix up the Component_Bit_Offset
-                        --  value to account for the reverse bit order.
-                        --  Some examples of what needs to be done are:
+                     --  Insert in entity list of scope of base type (which
+                     --  must be an enclosing scope, because still unfrozen).
 
-                        --    First_Bit .. Last_Bit     Component_Bit_Offset
-                        --      old          new          old       new
+                     Append_Entity (Comp, Scope (Base_Type (Comp)));
+                  end if;
+               end;
 
-                        --     0 .. 0       7 .. 7         0         7
-                        --     0 .. 1       6 .. 7         0         6
-                        --     0 .. 2       5 .. 7         0         5
-                        --     0 .. 7       0 .. 7         0         4
+            --  If the component is an access type with an allocator as
+            --  default value, the designated type will be frozen by the
+            --  corresponding expression in init_proc. In order to place the
+            --  freeze node for the designated type before that for the
+            --  current record type, freeze it now.
 
-                        --     1 .. 1       6 .. 6         1         6
-                        --     1 .. 4       3 .. 6         1         3
-                        --     4 .. 7       0 .. 3         4         0
+            --  Same process if the component is an array of access types,
+            --  initialized with an aggregate. If the designated type is
+            --  private, it cannot contain allocators, and it is premature to
+            --  freeze the type, so we check for this as well.
 
-                        --  The general rule is that the first bit is
-                        --  is obtained by subtracting the old ending bit
-                        --  from storage_unit - 1.
+            elsif Is_Access_Type (Etype (Comp))
+              and then Present (Parent (Comp))
+              and then Present (Expression (Parent (Comp)))
+              and then Nkind (Expression (Parent (Comp))) = N_Allocator
+            then
+               declare
+                  Alloc : constant Node_Id := Expression (Parent (Comp));
 
-                        Set_Component_Bit_Offset (Comp,
-                          (Storage_Unit_Offset * System_Storage_Unit)
-                          + (System_Storage_Unit - 1)
-                          - (Start_Bit + CSZ - 1));
+               begin
+                  --  If component is pointer to a classwide type, freeze
+                  --  the specific type in the expression being allocated.
+                  --  The expression may be a subtype indication, in which
+                  --  case freeze the subtype mark.
 
-                        Set_Normalized_First_Bit (Comp,
-                          Component_Bit_Offset (Comp) mod System_Storage_Unit);
+                  if Is_Class_Wide_Type (Designated_Type (Etype (Comp))) then
+                     if Is_Entity_Name (Expression (Alloc)) then
+                        Freeze_And_Append
+                          (Entity (Expression (Alloc)), Loc, Result);
+                     elsif
+                       Nkind (Expression (Alloc)) = N_Subtype_Indication
+                     then
+                        Freeze_And_Append
+                         (Entity (Subtype_Mark (Expression (Alloc))),
+                           Loc, Result);
                      end if;
-                  end;
-               end if;
+
+                  elsif Is_Itype (Designated_Type (Etype (Comp))) then
+                     Check_Itype (Designated_Type (Etype (Comp)));
+
+                  else
+                     Freeze_And_Append
+                       (Designated_Type (Etype (Comp)), Loc, Result);
+                  end if;
+               end;
+
+            elsif Is_Access_Type (Etype (Comp))
+              and then Is_Itype (Designated_Type (Etype (Comp)))
+            then
+               Check_Itype (Designated_Type (Etype (Comp)));
+
+            elsif Is_Array_Type (Etype (Comp))
+              and then Is_Access_Type (Component_Type (Etype (Comp)))
+              and then Present (Parent (Comp))
+              and then Nkind (Parent (Comp)) = N_Component_Declaration
+              and then Present (Expression (Parent (Comp)))
+              and then Nkind (Expression (Parent (Comp))) = N_Aggregate
+              and then Is_Fully_Defined
+                 (Designated_Type (Component_Type (Etype (Comp))))
+            then
+               Freeze_And_Append
+                 (Designated_Type
+                   (Component_Type (Etype (Comp))), Loc, Result);
             end if;
 
+            Prev := Comp;
             Next_Entity (Comp);
          end loop;
 
-         --  Check for useless pragma Bit_Order
+         --  Deal with pragma Bit_Order
+
+         if Reverse_Bit_Order (Rec) and then Base_Type (Rec) = Rec then
+            if not Placed_Component then
+               ADC :=
+                 Get_Attribute_Definition_Clause (Rec, Attribute_Bit_Order);
+               Error_Msg_N
+                 ("?Bit_Order specification has no effect", ADC);
+               Error_Msg_N
+                 ("\?since no component clauses were specified", ADC);
+
+            --  Here is where we do Ada 2005 processing for bit order (the
+            --  Ada 95 case was already taken care of above).
 
-         if not Placed_Component and then Reverse_Bit_Order (Rec) then
-            ADC := Get_Attribute_Definition_Clause (Rec, Attribute_Bit_Order);
-            Error_Msg_N ("?Bit_Order specification has no effect", ADC);
-            Error_Msg_N ("\?since no component clauses were specified", ADC);
+            elsif Ada_Version >= Ada_05 then
+               Adjust_Record_For_Reverse_Bit_Order (Rec);
+            end if;
          end if;
 
-         --  Check for useless pragma Pack when all components placed
+         --  Check for useless pragma Pack when all components placed. We only
+         --  do this check for record types, not subtypes, since a subtype may
+         --  have all its components placed, and it still makes perfectly good
+         --  sense to pack other subtypes or the parent type.
 
-         if Is_Packed (Rec)
+         if Ekind (Rec) = E_Record_Type
+           and then Is_Packed (Rec)
            and then not Unplaced_Component
-           and then Warn_On_Redundant_Constructs
          then
-            Error_Msg_N
-              ("?pragma Pack has no effect, no unplaced components",
-               Get_Rep_Pragma (Rec, Name_Pack));
+            --  Reset packed status. Probably not necessary, but we do it
+            --  so that there is no chance of the back end doing something
+            --  strange with this redundant indication of packing.
+
             Set_Is_Packed (Rec, False);
+
+            --  Give warning if redundant constructs warnings on
+
+            if Warn_On_Redundant_Constructs then
+               Error_Msg_N
+                 ("?pragma Pack has no effect, no unplaced components",
+                  Get_Rep_Pragma (Rec, Name_Pack));
+            end if;
          end if;
 
-         --  If this is the record corresponding to a remote type,
-         --  freeze the remote type here since that is what we are
-         --  semantically freeing.  This prevents having the freeze node
-         --  for that type in an inner scope.
+         --  If this is the record corresponding to a remote type, freeze the
+         --  remote type here since that is what we are semantically freezing.
+         --  This prevents the freeze node for that type in an inner scope.
 
          --  Also, Check for controlled components and unchecked unions.
-         --  Finally, enforce the restriction that access attributes with
-         --  current instance prefix can only apply to limited types.
+         --  Finally, enforce the restriction that access attributes with a
+         --  current instance prefix can only apply to limited types.
 
          if  Ekind (Rec) = E_Record_Type then
-
             if Present (Corresponding_Remote_Type (Rec)) then
                Freeze_And_Append
                  (Corresponding_Remote_Type (Rec), Loc, Result);
             end if;
 
             Comp := First_Component (Rec);
-
             while Present (Comp) loop
                if Has_Controlled_Component (Etype (Comp))
                  or else (Chars (Comp) /= Name_uParent
@@ -1441,9 +1943,8 @@ package body Freeze is
                if Has_Per_Object_Constraint (Comp)
                  and then not Is_Limited_Type (Rec)
                then
-                  --  Scan component declaration for likely misuses of
-                  --  current instance, either in a constraint or in a
-                  --  default expression.
+                  --  Scan component declaration for likely misuses of current
+                  --  instance, either in a constraint or a default expression.
 
                   Check_Current_Instance (Parent (Comp));
                end if;
@@ -1454,20 +1955,23 @@ package body Freeze is
 
          Set_Component_Alignment_If_Not_Set (Rec);
 
-         --  For first subtypes, check if there are any fixed-point
-         --  fields with component clauses, where we must check the size.
-         --  This is not done till the freeze point, since for fixed-point
-         --  types, we do not know the size until the type is frozen.
+         --  For first subtypes, check if there are any fixed-point fields with
+         --  component clauses, where we must check the size. This is not done
+         --  till the freeze point, since for fixed-point types, we do not know
+         --  the size until the type is frozen. Similar processing applies to
+         --  bit packed arrays.
 
          if Is_First_Subtype (Rec) then
             Comp := First_Component (Rec);
 
             while Present (Comp) loop
                if Present (Component_Clause (Comp))
-                 and then Is_Fixed_Point_Type (Etype (Comp))
+                 and then (Is_Fixed_Point_Type (Etype (Comp))
+                             or else
+                           Is_Bit_Packed_Array (Etype (Comp)))
                then
                   Check_Size
-                    (Component_Clause (Comp),
+                    (Component_Name (Component_Clause (Comp)),
                      Etype (Comp),
                      Esize (Comp),
                      Junk);
@@ -1481,19 +1985,28 @@ package body Freeze is
    --  Start of processing for Freeze_Entity
 
    begin
-      --  Do not freeze if already frozen since we only need one freeze node.
+      --  We are going to test for various reasons why this entity need not be
+      --  frozen here, but in the case of an Itype that's defined within a
+      --  record, that test actually applies to the record.
+
+      if Is_Itype (E) and then Is_Record_Type (Scope (E)) then
+         Test_E := Scope (E);
+      elsif Is_Itype (E) and then Present (Underlying_Type (Scope (E)))
+        and then Is_Record_Type (Underlying_Type (Scope (E)))
+      then
+         Test_E := Underlying_Type (Scope (E));
+      end if;
+
+      --  Do not freeze if already frozen since we only need one freeze node
 
       if Is_Frozen (E) then
          return No_List;
 
-      --  It is improper to freeze an external entity within a generic
-      --  because its freeze node will appear in a non-valid context.
-      --  ??? We should probably freeze the entity at that point and insert
-      --  the freeze node in a proper place but this proper place is not
-      --  easy to find, and the proper scope is not easy to restore. For
-      --  now, just wait to get out of the generic to freeze ???
+      --  It is improper to freeze an external entity within a generic because
+      --  its freeze node will appear in a non-valid context. The entity will
+      --  be frozen in the proper scope after the current generic is analyzed.
 
-      elsif Inside_A_Generic and then External_Ref_In_Generic (E) then
+      elsif Inside_A_Generic and then External_Ref_In_Generic (Test_E) then
          return No_List;
 
       --  Do not freeze a global entity within an inner scope created during
@@ -1506,10 +2019,12 @@ package body Freeze is
       --  must be diagnosed (see e.g. 1522-005). If the enclosing subprogram
       --  comes from source, or is a generic instance, then the freeze point
       --  is the one mandated by the language. and we freze the entity.
+      --  A subprogram that is a child unit body that acts as a spec does not
+      --  have a spec that comes from source, but can only come from source.
 
-      elsif In_Open_Scopes (Scope (E))
-        and then Scope (E) /= Current_Scope
-        and then Ekind (E) /= E_Constant
+      elsif In_Open_Scopes (Scope (Test_E))
+        and then Scope (Test_E) /= Current_Scope
+        and then Ekind (Test_E) /= E_Constant
       then
          declare
             S : Entity_Id := Current_Scope;
@@ -1519,6 +2034,7 @@ package body Freeze is
                if Is_Overloadable (S) then
                   if Comes_From_Source (S)
                     or else Is_Generic_Instance (S)
+                    or else Is_Child_Unit (S)
                   then
                      exit;
                   else
@@ -1529,6 +2045,36 @@ package body Freeze is
                S := Scope (S);
             end loop;
          end;
+
+      --  Similarly, an inlined instance body may make reference to global
+      --  entities, but these references cannot be the proper freezing point
+      --  for them, and the the absence of inlining freezing will take place
+      --  in their own scope. Normally instance bodies are analyzed after
+      --  the enclosing compilation, and everything has been frozen at the
+      --  proper place, but with front-end inlining an instance body is
+      --  compiled before the end of the enclosing scope, and as a result
+      --  out-of-order freezing must be prevented.
+
+      elsif Front_End_Inlining
+        and then  In_Instance_Body
+        and then Present (Scope (Test_E))
+      then
+         declare
+            S : Entity_Id := Scope (Test_E);
+
+         begin
+            while Present (S) loop
+               if Is_Generic_Instance (S) then
+                  exit;
+               else
+                  S := Scope (S);
+               end if;
+            end loop;
+
+            if No (S) then
+               return No_List;
+            end if;
+         end;
       end if;
 
       --  Here to freeze the entity
@@ -1551,20 +2097,53 @@ package body Freeze is
          then
             Set_Encoded_Interface_Name
               (E, Get_Default_External_Name (E));
+
+         --  Special processing for atomic objects appearing in object decls
+
+         elsif Is_Atomic (E)
+           and then Nkind (Parent (E)) = N_Object_Declaration
+           and then Present (Expression (Parent (E)))
+         then
+            declare
+               Expr : constant Node_Id := Expression (Parent (E));
+
+            begin
+               --  If expression is an aggregate, assign to a temporary to
+               --  ensure that the actual assignment is done atomically rather
+               --  than component-wise (the assignment to the temp may be done
+               --  component-wise, but that is harmless.
+
+               if Nkind (Expr) = N_Aggregate then
+                  Expand_Atomic_Aggregate (Expr, Etype (E));
+
+               --  If the expression is a reference to a record or array object
+               --  entity, then reset Is_True_Constant to False so that the
+               --  compiler will not optimize away the intermediate object,
+               --  which we need in this case for the same reason (to ensure
+               --  that the actual assignment is atomic, rather than
+               --  component-wise).
+
+               elsif Is_Entity_Name (Expr)
+                 and then (Is_Record_Type (Etype (Expr))
+                             or else
+                           Is_Array_Type (Etype (Expr)))
+               then
+                  Set_Is_True_Constant (Entity (Expr), False);
+               end if;
+            end;
          end if;
 
          --  For a subprogram, freeze all parameter types and also the return
-         --  type (RM 13.14(13)). However skip this for internal subprograms.
+         --  type (RM 13.14(14)). However skip this for internal subprograms.
          --  This is also the point where any extra formal parameters are
          --  created since we now know whether the subprogram will use
          --  a foreign convention.
 
          if Is_Subprogram (E) then
-
             if not Is_Internal (E) then
-
                declare
-                  F_Type : Entity_Id;
+                  F_Type    : Entity_Id;
+                  Warn_Node : Node_Id;
 
                   function Is_Fat_C_Ptr_Type (T : Entity_Id) return Boolean;
                   --  Determines if given type entity is a fat pointer type
@@ -1588,9 +2167,7 @@ package body Freeze is
                   --  Loop through formals
 
                   Formal := First_Formal (E);
-
                   while Present (Formal) loop
-
                      F_Type := Etype (Formal);
                      Freeze_And_Append (F_Type, Loc, Result);
 
@@ -1606,6 +2183,7 @@ package body Freeze is
                         --  an artifact of our need to regard the end of an
                         --  instantiation as a freeze point. Otherwise it is
                         --  a definite error.
+
                         --  and then not Is_Wrapper_Package (Current_Scope) ???
 
                         if In_Instance then
@@ -1622,7 +2200,9 @@ package body Freeze is
 
                      --  Check bad use of fat C pointer
 
-                     if Is_Fat_C_Ptr_Type (F_Type) then
+                     if Warn_On_Export_Import and then
+                       Is_Fat_C_Ptr_Type (F_Type)
+                     then
                         Error_Msg_Qual_Level := 1;
                         Error_Msg_N
                            ("?type of & does not correspond to C pointer",
@@ -1637,17 +2217,65 @@ package body Freeze is
                        and then not Is_Imported (E)
                        and then Is_Array_Type (F_Type)
                        and then not Is_Constrained (F_Type)
+                       and then Warn_On_Export_Import
                      then
                         Error_Msg_Qual_Level := 1;
-                        Error_Msg_N
+
+                        --  If this is an inherited operation, place the
+                        --  warning on the derived type declaration, rather
+                        --  than on the original subprogram.
+
+                        if Nkind (Original_Node (Parent (E))) =
+                          N_Full_Type_Declaration
+                        then
+                           Warn_Node := Parent (E);
+
+                           if Formal = First_Formal (E) then
+                              Error_Msg_NE
+                                ("?in inherited operation&", Warn_Node, E);
+                           end if;
+                        else
+                           Warn_Node := Formal;
+                        end if;
+
+                        Error_Msg_NE
                           ("?type of argument& is unconstrained array",
-                           Formal);
-                        Error_Msg_N
+                           Warn_Node, Formal);
+                        Error_Msg_NE
                           ("?foreign caller must pass bounds explicitly",
-                           Formal);
+                           Warn_Node, Formal);
                         Error_Msg_Qual_Level := 0;
                      end if;
 
+                     --  Ada 2005 (AI-326): Check wrong use of tag incomplete
+                     --  types with unknown discriminants. For example:
+
+                     --    type T (<>) is tagged;
+                     --    procedure P (X : access T); -- ERROR
+                     --    procedure P (X : T);        -- ERROR
+
+                     if not From_With_Type (F_Type) then
+                        if Is_Access_Type (F_Type) then
+                           F_Type := Designated_Type (F_Type);
+                        end if;
+
+                        if Ekind (F_Type) = E_Incomplete_Type
+                          and then Is_Tagged_Type (F_Type)
+                          and then not Is_Class_Wide_Type (F_Type)
+                          and then No (Full_View (F_Type))
+                          and then Unknown_Discriminants_Present
+                                     (Parent (F_Type))
+                          and then No (Stored_Constraint (F_Type))
+                        then
+                           Error_Msg_N
+                             ("(Ada 2005): invalid use of unconstrained tagged"
+                              & " incomplete type", E);
+
+                        elsif Ekind (F_Type) = E_Subprogram_Type then
+                           Freeze_And_Append (F_Type, Loc, Result);
+                        end if;
+                     end if;
+
                      Next_Formal (Formal);
                   end loop;
 
@@ -1656,7 +2284,9 @@ package body Freeze is
                   if Ekind (E) = E_Function then
                      Freeze_And_Append (Etype (E), Loc, Result);
 
-                     if Is_Fat_C_Ptr_Type (Etype (E)) then
+                     if Warn_On_Export_Import
+                       and then Is_Fat_C_Ptr_Type (Etype (E))
+                     then
                         Error_Msg_N
                           ("?return type of& does not correspond to C pointer",
                            E);
@@ -1665,10 +2295,25 @@ package body Freeze is
                        and then not Is_Constrained (Etype (E))
                        and then not Is_Imported (E)
                        and then Convention (E) in Foreign_Convention
+                       and then Warn_On_Export_Import
                      then
                         Error_Msg_N
-                          ("foreign convention function may not " &
+                          ("?foreign convention function& should not " &
                            "return unconstrained array", E);
+
+                     --  Ada 2005 (AI-326): Check wrong use of tagged
+                     --  incomplete type
+                     --
+                     --    type T is tagged;
+                     --    function F (X : Boolean) return T; -- ERROR
+
+                     elsif Ekind (Etype (E)) = E_Incomplete_Type
+                       and then Is_Tagged_Type (Etype (E))
+                       and then No (Full_View (Etype (E)))
+                     then
+                        Error_Msg_N
+                          ("(Ada 2005): invalid use of tagged incomplete type",
+                           E);
                      end if;
                   end if;
                end;
@@ -1680,17 +2325,6 @@ package body Freeze is
                Freeze_And_Append (Alias (E), Loc, Result);
             end if;
 
-            --  If the return type requires a transient scope, and we are on
-            --  a target allowing functions to return with a depressed stack
-            --  pointer, then we mark the function as requiring this treatment.
-
-            if Ekind (E) = E_Function
-              and then Functions_Return_By_DSP_On_Target
-              and then Requires_Transient_Scope (Etype (E))
-            then
-               Set_Function_Returns_With_DSP (E);
-            end if;
-
             if not Is_Internal (E) then
                Freeze_Subprogram (E);
             end if;
@@ -1707,13 +2341,35 @@ package body Freeze is
                Freeze_And_Append (Etype (E), Loc, Result);
             end if;
 
-            --  For object created by object declaration, perform required
-            --  categorization (preelaborate and pure) checks. Defer these
-            --  checks to freeze time since pragma Import inhibits default
-            --  initialization and thus pragma Import affects these checks.
+            --  Special processing for objects created by object declaration
 
             if Nkind (Declaration_Node (E)) = N_Object_Declaration then
+
+               --  For object created by object declaration, perform required
+               --  categorization (preelaborate and pure) checks. Defer these
+               --  checks to freeze time since pragma Import inhibits default
+               --  initialization and thus pragma Import affects these checks.
+
                Validate_Object_Declaration (Declaration_Node (E));
+
+               --  If there is an address clause, check it is valid
+
+               Check_Address_Clause (E);
+
+               --  For imported objects, set Is_Public unless there is also
+               --  an address clause, which means that there is no external
+               --  symbol needed for the Import (Is_Public may still be set
+               --  for other unrelated reasons). Note that we delayed this
+               --  processing till freeze time so that we can be sure not
+               --  to set the flag if there is an address clause. If there
+               --  is such a clause, then the only purpose of the import
+               --  pragma is to suppress implicit initialization.
+
+               if Is_Imported (E)
+                 and then No (Address_Clause (E))
+               then
+                  Set_Is_Public (E);
+               end if;
             end if;
 
             --  Check that a constant which has a pragma Volatile[_Components]
@@ -1729,14 +2385,21 @@ package body Freeze is
                --  inherited the indication from elsewhere (e.g. an address
                --  clause, which is not good enough in RM terms!)
 
-               if Present (Get_Rep_Pragma (E, Name_Atomic))            or else
-                  Present (Get_Rep_Pragma (E, Name_Atomic_Components)) or else
-                  Present (Get_Rep_Pragma (E, Name_Volatile))          or else
-                  Present (Get_Rep_Pragma (E, Name_Volatile_Components))
+               if Has_Rep_Pragma (E, Name_Atomic)
+                    or else
+                  Has_Rep_Pragma (E, Name_Atomic_Components)
                then
                   Error_Msg_N
-                    ("stand alone atomic/volatile constant must be imported",
-                     E);
+                    ("stand alone atomic constant must be " &
+                     "imported ('R'M 'C.6(13))", E);
+
+               elsif Has_Rep_Pragma (E, Name_Volatile)
+                       or else
+                     Has_Rep_Pragma (E, Name_Volatile_Components)
+               then
+                  Error_Msg_N
+                    ("stand alone volatile constant must be " &
+                     "imported ('R'M 'C.6(13))", E);
                end if;
             end if;
 
@@ -1765,6 +2428,16 @@ package body Freeze is
       --  Case of a type or subtype being frozen
 
       else
+         --  Check preelaborable initialization for full type completing a
+         --  private type for which pragma Preelaborable_Initialization given.
+
+         if Must_Have_Preelab_Init (E)
+           and then not Has_Preelaborable_Initialization (E)
+         then
+            Error_Msg_N
+              ("full view of & does not have preelaborable initialization", E);
+         end if;
+
          --  The type may be defined in a generic unit. This can occur when
          --  freezing a generic function that returns the type (which is
          --  defined in a parent unit). It is clearly meaningless to freeze
@@ -1792,13 +2465,13 @@ package body Freeze is
                Freeze_And_Append (Atype, Loc, Result);
 
             --  Otherwise freeze the base type of the entity before
-            --  freezing the entity itself, (RM 13.14(14)).
+            --  freezing the entity itself, (RM 13.14(15)).
 
             elsif E /= Base_Type (E) then
                Freeze_And_Append (Base_Type (E), Loc, Result);
             end if;
 
-         --  For a derived type, freeze its parent type first (RM 13.14(14))
+         --  For a derived type, freeze its parent type first (RM 13.14(15))
 
          elsif Is_Derived_Type (E) then
             Freeze_And_Append (Etype (E), Loc, Result);
@@ -1806,11 +2479,12 @@ package body Freeze is
          end if;
 
          --  For array type, freeze index types and component type first
-         --  before freezing the array (RM 13.14(14)).
+         --  before freezing the array (RM 13.14(15)).
 
          if Is_Array_Type (E) then
             declare
-               Ctyp  : constant Entity_Id := Component_Type (E);
+               Ctyp : constant Entity_Id := Component_Type (E);
+               Pnod : Node_Id;
 
                Non_Standard_Enum : Boolean := False;
                --  Set true if any of the index types is an enumeration
@@ -1832,9 +2506,12 @@ package body Freeze is
                   Next_Index (Indx);
                end loop;
 
-               --  For base type, propagate flags for component type
+               --  Processing that is done only for base types
 
                if Ekind (E) = E_Array_Type then
+
+                  --  Propagate flags for component type
+
                   if Is_Controlled (Component_Type (E))
                     or else Has_Controlled_Component (Ctyp)
                   then
@@ -1844,18 +2521,16 @@ package body Freeze is
                   if Has_Unchecked_Union (Component_Type (E)) then
                      Set_Has_Unchecked_Union (E);
                   end if;
-               end if;
 
-               --  If packing was requested or if the component size was set
-               --  explicitly, then see if bit packing is required. This
-               --  processing is only done for base types, since all the
-               --  representation aspects involved are type-related. This
-               --  is not just an optimization, if we start processing the
-               --  subtypes, they intefere with the settings on the base
-               --  type (this is because Is_Packed has a slightly different
-               --  meaning before and after freezing).
+                  --  If packing was requested or if the component size was set
+                  --  explicitly, then see if bit packing is required. This
+                  --  processing is only done for base types, since all the
+                  --  representation aspects involved are type-related. This
+                  --  is not just an optimization, if we start processing the
+                  --  subtypes, they intefere with the settings on the base
+                  --  type (this is because Is_Packed has a slightly different
+                  --  meaning before and after freezing).
 
-               if E = Base_Type (E) then
                   declare
                      Csiz : Uint;
                      Esiz : Uint;
@@ -1911,6 +2586,33 @@ package body Freeze is
 
                         Set_Component_Size (Base_Type (E), Csiz);
 
+                        --  Check for base type of 8,16,32 bits, where the
+                        --  subtype has a length one less than the base type
+                        --  and is unsigned (e.g. Natural subtype of Integer)
+
+                        --  In such cases, if a component size was not set
+                        --  explicitly, then generate a warning.
+
+                        if Has_Pragma_Pack (E)
+                          and then not Has_Component_Size_Clause (E)
+                          and then
+                            (Csiz = 7 or else Csiz = 15 or else Csiz = 31)
+                          and then Esize (Base_Type (Ctyp)) = Csiz + 1
+                        then
+                           Error_Msg_Uint_1 := Csiz;
+                           Pnod :=
+                             Get_Rep_Pragma (First_Subtype (E), Name_Pack);
+
+                           if Present (Pnod) then
+                              Error_Msg_N
+                                ("pragma Pack causes component size to be ^?",
+                                 Pnod);
+                              Error_Msg_N
+                                ("\use Component_Size to set desired value",
+                                 Pnod);
+                           end if;
+                        end if;
+
                         --  Actual packing is not needed for 8,16,32,64
                         --  Also not needed for 24 if alignment is 1
 
@@ -1944,8 +2646,97 @@ package body Freeze is
                         end if;
                      end if;
                   end;
+
+               --  Processing that is done only for subtypes
+
+               else
+                  --  Acquire alignment from base type
+
+                  if Unknown_Alignment (E) then
+                     Set_Alignment (E, Alignment (Base_Type (E)));
+                  end if;
+               end if;
+
+               --  For bit-packed arrays, check the size
+
+               if Is_Bit_Packed_Array (E)
+                 and then Known_Esize (E)
+               then
+                  declare
+                     Discard : Boolean;
+                     SizC    : constant Node_Id := Size_Clause (E);
+
+                  begin
+                     --  It is not clear if it is possible to have no size
+                     --  clause at this stage, but this is not worth worrying
+                     --  about. Post the error on the entity name in the size
+                     --  clause if present, else on the type entity itself.
+
+                     if Present (SizC) then
+                        Check_Size (Name (SizC), E, Esize (E), Discard);
+                     else
+                        Check_Size (E, E, Esize (E), Discard);
+                     end if;
+                  end;
                end if;
 
+               --  Check one common case of a size given where the array
+               --  needs to be packed, but was not so the size cannot be
+               --  honored. This would of course be caught by the backend,
+               --  and indeed we don't catch all cases. The point is that
+               --  we can give a better error message in those cases that
+               --  we do catch with the circuitry here.
+
+               declare
+                  Lo, Hi : Node_Id;
+                  Ctyp   : constant Entity_Id := Component_Type (E);
+
+               begin
+                  if Present (Size_Clause (E))
+                    and then Known_Static_Esize (E)
+                    and then not Is_Bit_Packed_Array (E)
+                    and then not Has_Pragma_Pack (E)
+                    and then Number_Dimensions (E) = 1
+                    and then not Has_Component_Size_Clause (E)
+                    and then Known_Static_Esize (Ctyp)
+                  then
+                     Get_Index_Bounds (First_Index (E), Lo, Hi);
+
+                     if Compile_Time_Known_Value (Lo)
+                       and then Compile_Time_Known_Value (Hi)
+                       and then Known_Static_RM_Size (Ctyp)
+                       and then RM_Size (Ctyp) < 64
+                     then
+                        declare
+                           Lov  : constant Uint := Expr_Value (Lo);
+                           Hiv  : constant Uint := Expr_Value (Hi);
+                           Len  : constant Uint :=
+                                    UI_Max (Uint_0, Hiv - Lov + 1);
+                           Rsiz : constant Uint := RM_Size (Ctyp);
+
+                        --  What we are looking for here is the situation
+                        --  where the Esize given would be exactly right
+                        --  if there was a pragma Pack (resulting in the
+                        --  component size being the same as the RM_Size).
+                        --  Furthermore, the component type size must be
+                        --  an odd size (not a multiple of storage unit)
+
+                        begin
+                           if Esize (E) = Len * Rsiz
+                             and then Rsiz mod System_Storage_Unit /= 0
+                           then
+                              Error_Msg_NE
+                                ("size given for& too small",
+                                   Size_Clause (E), E);
+                              Error_Msg_N
+                                ("\explicit pragma Pack is required",
+                                   Size_Clause (E));
+                           end if;
+                        end;
+                     end if;
+                  end if;
+               end;
+
                --  If any of the index types was an enumeration type with
                --  a non-standard rep clause, then we indicate that the
                --  array type is always packed (even if it is not bit packed).
@@ -1954,30 +2745,53 @@ package body Freeze is
                   Set_Has_Non_Standard_Rep (Base_Type (E));
                   Set_Is_Packed            (Base_Type (E));
                end if;
-            end;
 
-            Set_Component_Alignment_If_Not_Set (E);
+               Set_Component_Alignment_If_Not_Set (E);
 
-            --  If the array is packed, we must create the packed array
-            --  type to be used to actually implement the type. This is
-            --  only needed for real array types (not for string literal
-            --  types, since they are present only for the front end).
+               --  If the array is packed, we must create the packed array
+               --  type to be used to actually implement the type. This is
+               --  only needed for real array types (not for string literal
+               --  types, since they are present only for the front end).
 
-            if Is_Packed (E)
-              and then Ekind (E) /= E_String_Literal_Subtype
-            then
-               Create_Packed_Array_Type (E);
-               Freeze_And_Append (Packed_Array_Type (E), Loc, Result);
+               if Is_Packed (E)
+                 and then Ekind (E) /= E_String_Literal_Subtype
+               then
+                  Create_Packed_Array_Type (E);
+                  Freeze_And_Append (Packed_Array_Type (E), Loc, Result);
 
-               --  Size information of packed array type is copied to the
-               --  array type, since this is really the representation.
+                  --  Size information of packed array type is copied to the
+                  --  array type, since this is really the representation. But
+                  --  do not override explicit existing size values.
 
-               Set_Size_Info (E, Packed_Array_Type (E));
-               Set_RM_Size   (E, RM_Size (Packed_Array_Type (E)));
-            end if;
+                  if not Has_Size_Clause (E) then
+                     Set_Esize     (E, Esize     (Packed_Array_Type (E)));
+                     Set_RM_Size   (E, RM_Size   (Packed_Array_Type (E)));
+                  end if;
 
-         --  For a class wide type, the corresponding specific type is
-         --  frozen as well (RM 13.14(14))
+                  if not Has_Alignment_Clause (E) then
+                     Set_Alignment (E, Alignment (Packed_Array_Type (E)));
+                  end if;
+               end if;
+
+               --  For non-packed arrays set the alignment of the array
+               --  to the alignment of the component type if it is unknown.
+               --  Skip this in the atomic case, since atomic arrays may
+               --  need larger alignments.
+
+               if not Is_Packed (E)
+                 and then Unknown_Alignment (E)
+                 and then Known_Alignment (Ctyp)
+                 and then Known_Static_Component_Size (E)
+                 and then Known_Static_Esize (Ctyp)
+                 and then Esize (Ctyp) = Component_Size (E)
+                 and then not Is_Atomic (E)
+               then
+                  Set_Alignment (E, Alignment (Component_Type (E)));
+               end if;
+            end;
+
+         --  For a class-wide type, the corresponding specific type is
+         --  frozen as well (RM 13.14(15))
 
          elsif Is_Class_Wide_Type (E) then
             Freeze_And_Append (Root_Type (E), Loc, Result);
@@ -1991,9 +2805,8 @@ package body Freeze is
             if Is_Itype (E)
               and then Is_Compilation_Unit (Scope (E))
             then
-
                declare
-                  Ref : Node_Id := Make_Itype_Reference (Loc);
+                  Ref : constant Node_Id := Make_Itype_Reference (Loc);
 
                begin
                   Set_Itype (Ref, E);
@@ -2005,8 +2818,20 @@ package body Freeze is
                end;
             end if;
 
-         --  For record (sub)type, freeze all the component types (RM
-         --  13.14(14). We test for E_Record_(sub)Type here, rather than
+            --  The equivalent type associated with a class-wide subtype
+            --  needs to be frozen to ensure that its layout is done.
+            --  Class-wide subtypes are currently only frozen on targets
+            --  requiring front-end layout (see New_Class_Wide_Subtype
+            --  and Make_CW_Equivalent_Type in exp_util.adb).
+
+            if Ekind (E) = E_Class_Wide_Subtype
+              and then Present (Equivalent_Type (E))
+            then
+               Freeze_And_Append (Equivalent_Type (E), Loc, Result);
+            end if;
+
+         --  For a record (sub)type, freeze all the component types (RM
+         --  13.14(15). We test for E_Record_(sub)Type here, rather than
          --  using Is_Record_Type, because we don't want to attempt the
          --  freeze for the case of a private type with record extension
          --  (we will do that later when the full type is frozen).
@@ -2036,29 +2861,33 @@ package body Freeze is
                   Freeze_And_Append (Comp, Loc, Result);
 
                elsif (Ekind (Comp)) /= E_Function then
+                  if Is_Itype (Etype (Comp))
+                    and then Underlying_Type (Scope (Etype (Comp))) = E
+                  then
+                     Undelay_Type (Etype (Comp));
+                  end if;
+
                   Freeze_And_Append (Etype (Comp), Loc, Result);
                end if;
 
                Next_Entity (Comp);
             end loop;
 
-         --  Private types are required to point to the same freeze node
-         --  as their corresponding full views. The freeze node itself
-         --  has to point to the partial view of the entity (because
-         --  from the partial view, we can retrieve the full view, but
-         --  not the reverse). However, in order to freeze correctly,
-         --  we need to freeze the full view. If we are freezing at the
-         --  end of a scope (or within the scope of the private type),
-         --  the partial and full views will have been swapped, the
-         --  full view appears first in the entity chain and the swapping
-         --  mechanism enusres that the pointers are properly set (on
-         --  scope exit).
-
-         --  If we encounter the partial view before the full view
-         --  (e.g. when freezing from another scope), we freeze the
-         --  full view, and then set the pointers appropriately since
-         --  we cannot rely on swapping to fix things up (subtypes in an
-         --  outer scope might not get swapped).
+         --  Private types are required to point to the same freeze node as
+         --  their corresponding full views. The freeze node itself has to
+         --  point to the partial view of the entity (because from the partial
+         --  view, we can retrieve the full view, but not the reverse).
+         --  However, in order to freeze correctly, we need to freeze the full
+         --  view. If we are freezing at the end of a scope (or within the
+         --  scope of the private type), the partial and full views will have
+         --  been swapped, the full view appears first in the entity chain and
+         --  the swapping mechanism ensures that the pointers are properly set
+         --  (on scope exit).
+
+         --  If we encounter the partial view before the full view (e.g. when
+         --  freezing from another scope), we freeze the full view, and then
+         --  set the pointers appropriately since we cannot rely on swapping to
+         --  fix things up (subtypes in an outer scope might not get swapped).
 
          elsif Is_Incomplete_Or_Private_Type (E)
            and then not Is_Generic_Type (E)
@@ -2067,8 +2896,8 @@ package body Freeze is
 
             if Present (Full_View (E)) then
 
-               --  If full view has already been frozen, then no
-               --  further processing is required
+               --  If full view has already been frozen, then no further
+               --  processing is required
 
                if Is_Frozen (Full_View (E)) then
 
@@ -2076,41 +2905,48 @@ package body Freeze is
                   Set_Freeze_Node (E, Empty);
                   Check_Debug_Info_Needed (E);
 
-               --  Otherwise freeze full view and patch the pointers
+               --  Otherwise freeze full view and patch the pointers so that
+               --  the freeze node will elaborate both views in the back-end.
+
+               else
+                  declare
+                     Full : constant Entity_Id := Full_View (E);
+
+                  begin
+                     if Is_Private_Type (Full)
+                       and then Present (Underlying_Full_View (Full))
+                     then
+                        Freeze_And_Append
+                          (Underlying_Full_View (Full), Loc, Result);
+                     end if;
 
-               else
-                  if Is_Private_Type (Full_View (E))
-                    and then Present (Underlying_Full_View (Full_View (E)))
-                  then
-                     Freeze_And_Append
-                       (Underlying_Full_View (Full_View (E)), Loc, Result);
-                  end if;
+                     Freeze_And_Append (Full, Loc, Result);
 
-                  Freeze_And_Append (Full_View (E), Loc, Result);
+                     if Has_Delayed_Freeze (E) then
+                        F_Node := Freeze_Node (Full);
 
-                  if Has_Delayed_Freeze (E) then
-                     F_Node := Freeze_Node (Full_View (E));
+                        if Present (F_Node) then
+                           Set_Freeze_Node (E, F_Node);
+                           Set_Entity (F_Node, E);
 
-                     if Present (F_Node) then
-                        Set_Freeze_Node (E, F_Node);
-                        Set_Entity (F_Node, E);
-                     else
-                        --  {Incomplete,Private}_Subtypes
-                        --  with Full_Views constrained by discriminants
+                        else
+                           --  {Incomplete,Private}_Subtypes
+                           --  with Full_Views constrained by discriminants
 
-                        Set_Has_Delayed_Freeze (E, False);
-                        Set_Freeze_Node (E, Empty);
+                           Set_Has_Delayed_Freeze (E, False);
+                           Set_Freeze_Node (E, Empty);
+                        end if;
                      end if;
-                  end if;
+                  end;
 
                   Check_Debug_Info_Needed (E);
                end if;
 
-               --  AI-117 requires that the convention of a partial view
-               --  be the same as the convention of the full view. Note
-               --  that this is a recognized breach of privacy, but it's
-               --  essential for logical consistency of representation,
-               --  and the lack of a rule in RM95 was an oversight.
+               --  AI-117 requires that the convention of a partial view be the
+               --  same as the convention of the full view. Note that this is a
+               --  recognized breach of privacy, but it's essential for logical
+               --  consistency of representation, and the lack of a rule in
+               --  RM95 was an oversight.
 
                Set_Convention (E, Convention (Full_View (E)));
 
@@ -2120,10 +2956,10 @@ package body Freeze is
                --  Size information is copied from the full view to the
                --  incomplete or private view for consistency
 
-               --  We skip this is the full view is not a type. This is
-               --  very strange of course, and can only happen as a result
-               --  of certain illegalities, such as a premature attempt to
-               --  derive from an incomplete type.
+               --  We skip this is the full view is not a type. This is very
+               --  strange of course, and can only happen as a result of
+               --  certain illegalities, such as a premature attempt to derive
+               --  from an incomplete type.
 
                if Is_Type (Full_View (E)) then
                   Set_Size_Info (E, Full_View (E));
@@ -2158,27 +2994,55 @@ package body Freeze is
                Next_Formal (Formal);
             end loop;
 
-            --  If the return type requires a transient scope, and we are on
-            --  a target allowing functions to return with a depressed stack
-            --  pointer, then we mark the function as requiring this treatment.
+            Freeze_Subprogram (E);
+
+            --  Ada 2005 (AI-326): Check wrong use of tag incomplete type
+            --
+            --    type T is tagged;
+            --    type Acc is access function (X : T) return T; -- ERROR
 
-            if Functions_Return_By_DSP_On_Target
-              and then Requires_Transient_Scope (Etype (E))
+            if Ekind (Etype (E)) = E_Incomplete_Type
+              and then Is_Tagged_Type (Etype (E))
+              and then No (Full_View (Etype (E)))
             then
-               Set_Function_Returns_With_DSP (E);
+               Error_Msg_N
+                 ("(Ada 2005): invalid use of tagged incomplete type", E);
             end if;
 
-            Freeze_Subprogram (E);
+         --  For access to a protected subprogram, freeze the equivalent type
+         --  (however this is not set if we are not generating code or if this
+         --  is an anonymous type used just for resolution).
 
-         --  For access to a protected subprogram, freeze the equivalent
-         --  type (however this is not set if we are not generating code)
-         --  or if this is an anonymous type used just for resolution).
+         elsif Is_Access_Protected_Subprogram_Type (E) then
 
-         elsif Ekind (E) = E_Access_Protected_Subprogram_Type
-           and then Operating_Mode = Generate_Code
-           and then Present (Equivalent_Type (E))
-         then
-            Freeze_And_Append (Equivalent_Type (E), Loc, Result);
+            --  AI-326: Check wrong use of tagged incomplete types
+
+            --    type T is tagged;
+            --    type As3D is access protected
+            --      function (X : Float) return T; -- ERROR
+
+            declare
+               Etyp : Entity_Id;
+
+            begin
+               Etyp := Etype (Directly_Designated_Type (E));
+
+               if Is_Class_Wide_Type (Etyp) then
+                  Etyp := Etype (Etyp);
+               end if;
+
+               if Ekind (Etyp) = E_Incomplete_Type
+                 and then Is_Tagged_Type (Etyp)
+                 and then No (Full_View (Etyp))
+               then
+                  Error_Msg_N
+                    ("(Ada 2005): invalid use of tagged incomplete type", E);
+               end if;
+            end;
+
+            if Present (Equivalent_Type (E)) then
+               Freeze_And_Append (Equivalent_Type (E), Loc, Result);
+            end if;
          end if;
 
          --  Generic types are never seen by the back-end, and are also not
@@ -2195,94 +3059,77 @@ package body Freeze is
          if Is_Fixed_Point_Type (E) then
             Freeze_Fixed_Point_Type (E);
 
-         elsif Is_Enumeration_Type (E) then
-            Freeze_Enumeration_Type (E);
+            --  Some error checks required for ordinary fixed-point type. Defer
+            --  these till the freeze-point since we need the small and range
+            --  values. We only do these checks for base types
 
-         elsif Is_Integer_Type (E) then
-            Adjust_Esize_For_Alignment (E);
+            if Is_Ordinary_Fixed_Point_Type (E)
+              and then E = Base_Type (E)
+            then
+               if Small_Value (E) < Ureal_2_M_80 then
+                  Error_Msg_Name_1 := Name_Small;
+                  Error_Msg_N
+                    ("`&''%` is too small, minimum is 2.0'*'*(-80)", E);
 
-         elsif Is_Access_Type (E)
-           and then No (Associated_Storage_Pool (E))
-         then
-            Check_Restriction (No_Standard_Storage_Pools, E);
-         end if;
+               elsif Small_Value (E) > Ureal_2_80 then
+                  Error_Msg_Name_1 := Name_Small;
+                  Error_Msg_N
+                    ("`&''%` is too large, maximum is 2.0'*'*80", E);
+               end if;
 
-         --  If the current entity is an array or record subtype and has
-         --  discriminants used to constrain it, it must not freeze, because
-         --  Freeze_Entity nodes force Gigi to process the frozen type.
+               if Expr_Value_R (Type_Low_Bound (E)) < Ureal_M_10_36 then
+                  Error_Msg_Name_1 := Name_First;
+                  Error_Msg_N
+                    ("`&''%` is too small, minimum is -10.0'*'*36", E);
+               end if;
 
-         if Is_Composite_Type (E) then
+               if Expr_Value_R (Type_High_Bound (E)) > Ureal_10_36 then
+                  Error_Msg_Name_1 := Name_Last;
+                  Error_Msg_N
+                    ("`&''%` is too large, maximum is 10.0'*'*36", E);
+               end if;
+            end if;
 
-            if Is_Array_Type (E) then
+         elsif Is_Enumeration_Type (E) then
+            Freeze_Enumeration_Type (E);
 
-               declare
-                  Index : Node_Id := First_Index (E);
-                  Expr1 : Node_Id;
-                  Expr2 : Node_Id;
+         elsif Is_Integer_Type (E) then
+            Adjust_Esize_For_Alignment (E);
 
-               begin
-                  while Present (Index) loop
-                     if Etype (Index) /= Any_Type then
-                        Get_Index_Bounds (Index, Expr1, Expr2);
+         elsif Is_Access_Type (E) then
 
-                        for J in 1 .. 2 loop
-                           if Nkind (Expr1) = N_Identifier
-                             and then Ekind (Entity (Expr1)) = E_Discriminant
-                           then
-                              Set_Has_Delayed_Freeze (E, False);
-                              Set_Freeze_Node (E, Empty);
-                              Check_Debug_Info_Needed (E);
-                              return Result;
-                           end if;
+            --  Check restriction for standard storage pool
 
-                           Expr1 := Expr2;
-                        end loop;
-                     end if;
+            if No (Associated_Storage_Pool (E)) then
+               Check_Restriction (No_Standard_Storage_Pools, E);
+            end if;
 
-                     Next_Index (Index);
-                  end loop;
-               end;
+            --  Deal with error message for pure access type. This is not an
+            --  error in Ada 2005 if there is no pool (see AI-366).
 
-            elsif Has_Discriminants (E)
-              and Is_Constrained (E)
+            if Is_Pure_Unit_Access_Type (E)
+              and then (Ada_Version < Ada_05
+                        or else not No_Pool_Assigned (E))
             then
+               Error_Msg_N ("named access type not allowed in pure unit", E);
+            end if;
+         end if;
 
-               declare
-                  Constraint : Elmt_Id;
-                  Expr       : Node_Id;
-               begin
-                  Constraint := First_Elmt (Discriminant_Constraint (E));
-
-                  while Present (Constraint) loop
-
-                     Expr := Node (Constraint);
-                     if Nkind (Expr) = N_Identifier
-                       and then Ekind (Entity (Expr)) = E_Discriminant
-                     then
-                        Set_Has_Delayed_Freeze (E, False);
-                        Set_Freeze_Node (E, Empty);
-                        Check_Debug_Info_Needed (E);
-                        return Result;
-                     end if;
-
-                     Next_Elmt (Constraint);
-                  end loop;
-               end;
+         --  Case of composite types
 
-            end if;
+         if Is_Composite_Type (E) then
 
-            --  AI-117 requires that all new primitives of a tagged type
-            --  must inherit the convention of the full view of the type.
-            --  Inherited and overriding operations are defined to inherit
-            --  the convention of their parent or overridden subprogram
-            --  (also specified in AI-117), and that will have occurred
-            --  earlier (in Derive_Subprogram and New_Overloaded_Entity).
-            --  Here we set the convention of primitives that are still
-            --  convention Ada, which will ensure that any new primitives
-            --  inherit the type's convention. Class-wide types can have
-            --  a foreign convention inherited from their specific type,
-            --  but are excluded from this since they don't have any
-            --  associated primitives.
+            --  AI-117 requires that all new primitives of a tagged type must
+            --  inherit the convention of the full view of the type. Inherited
+            --  and overriding operations are defined to inherit the convention
+            --  of their parent or overridden subprogram (also specified in
+            --  AI-117), which will have occurred earlier (in Derive_Subprogram
+            --  and New_Overloaded_Entity). Here we set the convention of
+            --  primitives that are still convention Ada, which will ensure
+            --  that any new primitives inherit the type's convention.
+            --  Class-wide types can have a foreign convention inherited from
+            --  their specific type, but are excluded from this since they
+            --  don't have any associated primitives.
 
             if Is_Tagged_Type (E)
               and then not Is_Class_Wide_Type (E)
@@ -2290,9 +3137,9 @@ package body Freeze is
             then
                declare
                   Prim_List : constant Elist_Id := Primitive_Operations (E);
-                  Prim      : Elmt_Id           := First_Elmt (Prim_List);
-
+                  Prim      : Elmt_Id;
                begin
+                  Prim := First_Elmt (Prim_List);
                   while Present (Prim) loop
                      if Convention (Node (Prim)) = Convention_Ada then
                         Set_Convention (Node (Prim), Convention (E));
@@ -2304,9 +3151,70 @@ package body Freeze is
             end if;
          end if;
 
-         --  Now that all types from which E may depend are frozen, see
-         --  if the size is known at compile time, if it must be unsigned,
-         --  or if strict alignent is required
+         --  Generate primitive operation references for a tagged type
+
+         if Is_Tagged_Type (E)
+           and then not Is_Class_Wide_Type (E)
+         then
+            declare
+               Prim_List : Elist_Id;
+               Prim      : Elmt_Id;
+               Ent       : Entity_Id;
+               Aux_E     : Entity_Id;
+
+            begin
+               --  Handle subtypes
+
+               if Ekind (E) = E_Protected_Subtype
+                 or else Ekind (E) = E_Task_Subtype
+               then
+                  Aux_E := Etype (E);
+               else
+                  Aux_E := E;
+               end if;
+
+               --  Ada 2005 (AI-345): In case of concurrent type generate
+               --  reference to the wrapper that allow us to dispatch calls
+               --  through their implemented abstract interface types.
+
+               --  The check for Present here is to protect against previously
+               --  reported critical errors.
+
+               if Is_Concurrent_Type (Aux_E)
+                 and then Present (Corresponding_Record_Type (Aux_E))
+               then
+                  Prim_List := Primitive_Operations
+                                (Corresponding_Record_Type (Aux_E));
+               else
+                  Prim_List := Primitive_Operations (Aux_E);
+               end if;
+
+               --  Loop to generate references for primitive operations
+
+               if Present (Prim_List) then
+                  Prim := First_Elmt (Prim_List);
+                  while Present (Prim) loop
+
+                     --  If the operation is derived, get the original for
+                     --  cross-reference purposes (it is the original for
+                     --  which we want the xref, and for which the comes
+                     --  from source test needs to be performed).
+
+                     Ent := Node (Prim);
+                     while Present (Alias (Ent)) loop
+                        Ent := Alias (Ent);
+                     end loop;
+
+                     Generate_Reference (E, Ent, 'p', Set_Ref => False);
+                     Next_Elmt (Prim);
+                  end loop;
+               end if;
+            end;
+         end if;
+
+         --  Now that all types from which E may depend are frozen, see if the
+         --  size is known at compile time, if it must be unsigned, or if
+         --  strict alignent is required
 
          Check_Compile_Time_Size (E);
          Check_Unsigned_Type (E);
@@ -2321,16 +3229,21 @@ package body Freeze is
          if Has_Size_Clause (E)
            and then not Size_Known_At_Compile_Time (E)
          then
-            Error_Msg_N
-              ("size clause not allowed for variable length type",
-               Size_Clause (E));
+            --  Supress this message if errors posted on E, even if we are
+            --  in all errors mode, since this is often a junk message
+
+            if not Error_Posted (E) then
+               Error_Msg_N
+                 ("size clause not allowed for variable length type",
+                  Size_Clause (E));
+            end if;
          end if;
 
          --  Remaining process is to set/verify the representation information,
          --  in particular the size and alignment values. This processing is
          --  not required for generic types, since generic types do not play
          --  any part in code generation, and so the size and alignment values
-         --  for suhc types are irrelevant.
+         --  for such types are irrelevant.
 
          if Is_Generic_Type (E) then
             return Result;
@@ -2375,6 +3288,43 @@ package body Freeze is
             Append (F_Node, Result);
          end if;
 
+         --  A final pass over record types with discriminants. If the type
+         --  has an incomplete declaration, there may be constrained access
+         --  subtypes declared elsewhere, which do not depend on the discrimi-
+         --  nants of the type, and which are used as component types (i.e.
+         --  the full view is a recursive type). The designated types of these
+         --  subtypes can only be elaborated after the type itself, and they
+         --  need an itype reference.
+
+         if Ekind (E) = E_Record_Type
+           and then Has_Discriminants (E)
+         then
+            declare
+               Comp : Entity_Id;
+               IR   : Node_Id;
+               Typ  : Entity_Id;
+
+            begin
+               Comp := First_Component (E);
+
+               while Present (Comp) loop
+                  Typ  := Etype (Comp);
+
+                  if Ekind (Comp) = E_Component
+                    and then Is_Access_Type (Typ)
+                    and then Scope (Typ) /= E
+                    and then Base_Type (Designated_Type (Typ)) = E
+                    and then Is_Itype (Designated_Type (Typ))
+                  then
+                     IR := Make_Itype_Reference (Sloc (Comp));
+                     Set_Itype (IR, Designated_Type (Typ));
+                     Append (IR, Result);
+                  end if;
+
+                  Next_Component (Comp);
+               end loop;
+            end;
+         end if;
       end if;
 
       --  When a type is frozen, the first subtype of the type is frozen as
@@ -2423,11 +3373,7 @@ package body Freeze is
             if Result = No_List then
                Result := Empty_List;
             end if;
-
-            Generate_Subprogram_Descriptor_For_Imported_Subprogram
-              (E, Result);
          end if;
-
       end if;
 
       return Result;
@@ -2444,7 +3390,6 @@ package body Freeze is
         and then Esize (Typ) < Standard_Integer_Size
       then
          Init_Esize (Typ, Standard_Integer_Size);
-
       else
          Adjust_Esize_For_Alignment (Typ);
       end if;
@@ -2472,10 +3417,14 @@ package body Freeze is
 
       function In_Exp_Body (N : Node_Id) return Boolean;
       --  Given an N_Handled_Sequence_Of_Statements node N, determines whether
-      --  it is the handled statement sequence of an expander generated
+      --  it is the handled statement sequence of an expander-generated
       --  subprogram (init proc, or stream subprogram). If so, it returns
       --  True, otherwise False.
 
+      -----------------
+      -- In_Exp_Body --
+      -----------------
+
       function In_Exp_Body (N : Node_Id) return Boolean is
          P : Node_Id;
 
@@ -2493,27 +3442,26 @@ package body Freeze is
             P := Defining_Unit_Name (Specification (P));
 
             if Nkind (P) = N_Defining_Identifier
-              and then (Chars (P) = Name_uInit_Proc or else
-                        Chars (P) = Name_uInput     or else
-                        Chars (P) = Name_uOutput    or else
-                        Chars (P) = Name_uRead      or else
-                        Chars (P) = Name_uWrite)
+              and then (Is_Init_Proc (P)              or else
+                        Is_TSS (P, TSS_Stream_Input)  or else
+                        Is_TSS (P, TSS_Stream_Output) or else
+                        Is_TSS (P, TSS_Stream_Read)   or else
+                        Is_TSS (P, TSS_Stream_Write))
             then
                return True;
             else
                return False;
             end if;
          end if;
-
       end In_Exp_Body;
 
    --  Start of processing for Freeze_Expression
 
    begin
-      --  Immediate return if freezing is inhibited. This flag is set by
-      --  the analyzer to stop freezing on generated expressions that would
-      --  cause freezing if they were in the source program, but which are
-      --  not supposed to freeze, since they are created.
+      --  Immediate return if freezing is inhibited. This flag is set by the
+      --  analyzer to stop freezing on generated expressions that would cause
+      --  freezing if they were in the source program, but which are not
+      --  supposed to freeze, since they are created.
 
       if Must_Not_Freeze (N) then
          return;
@@ -2534,12 +3482,18 @@ package body Freeze is
 
       --  Freeze type of expression if not frozen already
 
-      if Nkind (N) in N_Has_Etype
-        and then not Is_Frozen (Etype (N))
-      then
-         Typ := Etype (N);
-      else
-         Typ := Empty;
+      Typ := Empty;
+
+      if Nkind (N) in N_Has_Etype then
+         if not Is_Frozen (Etype (N)) then
+            Typ := Etype (N);
+
+         --  Base type may be an derived numeric type that is frozen at
+         --  the point of declaration, but first_subtype is still unfrozen.
+
+         elsif not Is_Frozen (First_Subtype (Etype (N))) then
+            Typ := First_Subtype (Etype (N));
+         end if;
       end if;
 
       --  For entity name, freeze entity if not frozen already. A special
@@ -2557,22 +3511,21 @@ package body Freeze is
                    or else not Comes_From_Source (Entity (N)))
       then
          Nam := Entity (N);
-
       else
          Nam := Empty;
       end if;
 
-      --  For an allocator freeze designated type if not frozen already.
+      --  For an allocator freeze designated type if not frozen already
 
-      --  For an aggregate whose component type is an access type, freeze
-      --  the designated type now, so that its freeze  does not appear within
-      --  the loop that might be created in the expansion of the aggregate.
-      --  If the designated type is a private type without full view, the
-      --  expression cannot contain an allocator, so the type is not frozen.
+      --  For an aggregate whose component type is an access type, freeze the
+      --  designated type now, so that its freeze does not appear within the
+      --  loop that might be created in the expansion of the aggregate. If the
+      --  designated type is a private type without full view, the expression
+      --  cannot contain an allocator, so the type is not frozen.
 
       Desig_Typ := Empty;
-      case Nkind (N) is
 
+      case Nkind (N) is
          when N_Allocator =>
             Desig_Typ := Designated_Type (Etype (N));
 
@@ -2593,7 +3546,6 @@ package body Freeze is
 
          when others =>
             null;
-
       end case;
 
       if Desig_Typ /= Empty
@@ -2623,11 +3575,11 @@ package body Freeze is
       loop
          Parent_P := Parent (P);
 
-         --  If we don't have a parent, then we are not in a well-formed
-         --  tree. This is an unusual case, but there are some legitimate
-         --  situations in which this occurs, notably when the expressions
-         --  in the range of a type declaration are resolved. We simply
-         --  ignore the freeze request in this case. Is this right ???
+         --  If we don't have a parent, then we are not in a well-formed tree.
+         --  This is an unusual case, but there are some legitimate situations
+         --  in which this occurs, notably when the expressions in the range of
+         --  a type declaration are resolved. We simply ignore the freeze
+         --  request in this case. Is this right ???
 
          if No (Parent_P) then
             return;
@@ -2637,12 +3589,12 @@ package body Freeze is
 
          case Nkind (Parent_P) is
 
-            --  A special test for the exception of (RM 13.14(8)) for the
-            --  case of per-object expressions (RM 3.8(18)) occurring in a
-            --  component definition or a discrete subtype definition. Note
-            --  that we test for a component declaration which includes both
-            --  cases we are interested in, and furthermore the tree does not
-            --  have explicit nodes for either of these two constructs.
+            --  A special test for the exception of (RM 13.14(8)) for the case
+            --  of per-object expressions (RM 3.8(18)) occurring in component
+            --  definition or a discrete subtype definition. Note that we test
+            --  for a component declaration which includes both cases we are
+            --  interested in, and furthermore the tree does not have explicit
+            --  nodes for either of these two constructs.
 
             when N_Component_Declaration =>
 
@@ -2673,9 +3625,9 @@ package body Freeze is
                   end if;
                end if;
 
-            --  If we have an enumeration literal that appears as the
-            --  choice in the aggregate of an enumeration representation
-            --  clause, then freezing does not occur (RM 13.14(9)).
+            --  If we have an enumeration literal that appears as the choice in
+            --  the aggregate of an enumeration representation clause, then
+            --  freezing does not occur (RM 13.14(10)).
 
             when N_Enumeration_Representation_Clause =>
 
@@ -2692,9 +3644,9 @@ package body Freeze is
                   then
                      return;
 
-                  --  If enumeration literal appears as the name of a
-                  --  function which is the choice, then also do not freeze.
-                  --  This happens in the overloaded literal case, where the
+                  --  If enumeration literal appears as the name of function
+                  --  which is the choice, then also do not freeze. This
+                  --  happens in the overloaded literal case, where the
                   --  enumeration literal is temporarily changed to a function
                   --  call for overloading analysis purposes.
 
@@ -2714,11 +3666,11 @@ package body Freeze is
 
             when N_Handled_Sequence_Of_Statements =>
 
-               --  An exception occurs when the sequence of statements is
-               --  for an expander generated body that did not do the usual
-               --  freeze all operation. In this case we usually want to
-               --  freeze outside this body, not inside it, and we skip
-               --  past the subprogram body that we are inside.
+               --  An exception occurs when the sequence of statements is for
+               --  an expander generated body that did not do the usual freeze
+               --  all operation. In this case we usually want to freeze
+               --  outside this body, not inside it, and we skip past the
+               --  subprogram body that we are inside.
 
                if In_Exp_Body (Parent_P) then
 
@@ -2766,9 +3718,9 @@ package body Freeze is
                   exit;
                end if;
 
-            --  If parent is a body or a spec or a block, then the current
-            --  node is a statement or declaration and we can insert the
-            --  freeze node before it.
+            --  If parent is a body or a spec or a block, then the current node
+            --  is a statement or declaration and we can insert the freeze node
+            --  before it.
 
             when N_Package_Specification |
                  N_Package_Body          |
@@ -2800,11 +3752,11 @@ package body Freeze is
 
             --  Note: The N_Loop_Statement is a special case. A type that
             --  appears in the source can never be frozen in a loop (this
-            --  occurs only because of a loop expanded by the expander),
-            --  so we keep on going. Otherwise we terminate the search.
-            --  Same is true of any entity which comes from source. (if they
-            --  have a predefined type, that type does not appear to come
-            --  from source, but the entity should not be frozen here).
+            --  occurs only because of a loop expanded by the expander), so we
+            --  keep on going. Otherwise we terminate the search. Same is true
+            --  of any entity which comes from source. (if they have predefined
+            --  type, that type does not appear to come from source, but the
+            --  entity should not be frozen here).
 
             when N_Loop_Statement =>
                exit when not Comes_From_Source (Etype (N))
@@ -2822,22 +3774,27 @@ package body Freeze is
          P := Parent_P;
       end loop;
 
-      --  If the expression appears in a record or an initialization
-      --  procedure, the freeze nodes are collected and attached to
-      --  the current scope, to be inserted and analyzed on exit from
-      --  the scope, to insure that generated entities appear in the
-      --  correct scope. If the expression is a default for a discriminant
-      --  specification, the scope is still void. The expression can also
-      --  appear in the discriminant part of a private or concurrent type.
+      --  If the expression appears in a record or an initialization procedure,
+      --  the freeze nodes are collected and attached to the current scope, to
+      --  be inserted and analyzed on exit from the scope, to insure that
+      --  generated entities appear in the correct scope. If the expression is
+      --  a default for a discriminant specification, the scope is still void.
+      --  The expression can also appear in the discriminant part of a private
+      --  or concurrent type.
 
-      --  The other case requiring this special handling is if we are in
-      --  a default expression, since in that case we are about to freeze
-      --  a static type, and the freeze scope needs to be the outer scope,
-      --  not the scope of the subprogram with the default parameter.
+      --  If the expression appears in a constrained subcomponent of an
+      --  enclosing record declaration, the freeze nodes must be attached to
+      --  the outer record type so they can eventually be placed in the
+      --  enclosing declaration list.
+
+      --  The other case requiring this special handling is if we are in a
+      --  default expression, since in that case we are about to freeze a
+      --  static type, and the freeze scope needs to be the outer scope, not
+      --  the scope of the subprogram with the default parameter.
 
       --  For default expressions in generic units, the Move_Freeze_Nodes
-      --  mechanism (see sem_ch12.adb) takes care of placing them at the
-      --  proper place, after the generic unit.
+      --  mechanism (see sem_ch12.adb) takes care of placing them at the proper
+      --  place, after the generic unit.
 
       if (In_Def_Exp and not Inside_A_Generic)
         or else Freeze_Outside
@@ -2849,6 +3806,7 @@ package body Freeze is
          declare
             Loc          : constant Source_Ptr := Sloc (Current_Scope);
             Freeze_Nodes : List_Id := No_List;
+            Pos          : Int := Scope_Stack.Last;
 
          begin
             if Present (Desig_Typ) then
@@ -2863,17 +3821,21 @@ package body Freeze is
                Freeze_And_Append (Nam, Loc, Freeze_Nodes);
             end if;
 
-            if Is_Non_Empty_List (Freeze_Nodes) then
+            --  The current scope may be that of a constrained component of
+            --  an enclosing record declaration, which is above the current
+            --  scope in the scope stack.
 
-               if No (Scope_Stack.Table
-                 (Scope_Stack.Last).Pending_Freeze_Actions)
-               then
-                  Scope_Stack.Table
-                    (Scope_Stack.Last).Pending_Freeze_Actions :=
+            if Is_Record_Type (Scope (Current_Scope)) then
+               Pos := Pos - 1;
+            end if;
+
+            if Is_Non_Empty_List (Freeze_Nodes) then
+               if No (Scope_Stack.Table (Pos).Pending_Freeze_Actions) then
+                  Scope_Stack.Table (Pos).Pending_Freeze_Actions :=
                       Freeze_Nodes;
                else
                   Append_List (Freeze_Nodes, Scope_Stack.Table
-                                   (Scope_Stack.Last).Pending_Freeze_Actions);
+                                   (Pos).Pending_Freeze_Actions);
                end if;
             end if;
          end;
@@ -2883,30 +3845,29 @@ package body Freeze is
 
       --  Now we have the right place to do the freezing. First, a special
       --  adjustment, if we are in default expression analysis mode, these
-      --  freeze actions must not be thrown away (normally all inserted
-      --  actions are thrown away in this mode. However, the freeze actions
-      --  are from static expressions and one of the important reasons we
-      --  are doing this special analysis is to get these freeze actions.
-      --  Therefore we turn off the In_Default_Expression mode to propagate
-      --  these freeze actions. This also means they get properly analyzed
-      --  and expanded.
+      --  freeze actions must not be thrown away (normally all inserted actions
+      --  are thrown away in this mode. However, the freeze actions are from
+      --  static expressions and one of the important reasons we are doing this
+      --  special analysis is to get these freeze actions. Therefore we turn
+      --  off the In_Default_Expression mode to propagate these freeze actions.
+      --  This also means they get properly analyzed and expanded.
 
       In_Default_Expression := False;
 
-      --  Freeze the designated type of an allocator (RM 13.14(12))
+      --  Freeze the designated type of an allocator (RM 13.14(13))
 
       if Present (Desig_Typ) then
          Freeze_Before (P, Desig_Typ);
       end if;
 
-      --  Freeze type of expression (RM 13.14(9)). Note that we took care of
+      --  Freeze type of expression (RM 13.14(10)). Note that we took care of
       --  the enumeration representation clause exception in the loop above.
 
       if Present (Typ) then
          Freeze_Before (P, Typ);
       end if;
 
-      --  Freeze name if one is present (RM 13.14(10))
+      --  Freeze name if one is present (RM 13.14(11))
 
       if Present (Nam) then
          Freeze_Before (P, Nam);
@@ -2919,15 +3880,15 @@ package body Freeze is
    -- Freeze_Fixed_Point_Type --
    -----------------------------
 
-   --  Certain fixed-point types and subtypes, including implicit base
-   --  types and declared first subtypes, have not yet set up a range.
-   --  This is because the range cannot be set until the Small and Size
-   --  values are known, and these are not known till the type is frozen.
+   --  Certain fixed-point types and subtypes, including implicit base types
+   --  and declared first subtypes, have not yet set up a range. This is
+   --  because the range cannot be set until the Small and Size values are
+   --  known, and these are not known till the type is frozen.
 
-   --  To signal this case, Scalar_Range contains an unanalyzed syntactic
-   --  range whose bounds are unanalyzed real literals. This routine will
-   --  recognize this case, and transform this range node into a properly
-   --  typed range with properly analyzed and resolved values.
+   --  To signal this case, Scalar_Range contains an unanalyzed syntactic range
+   --  whose bounds are unanalyzed real literals. This routine will recognize
+   --  this case, and transform this range node into a properly typed range
+   --  with properly analyzed and resolved values.
 
    procedure Freeze_Fixed_Point_Type (Typ : Entity_Id) is
       Rng   : constant Node_Id    := Scalar_Range (Typ);
@@ -2948,6 +3909,10 @@ package body Freeze is
       --  Returns size of type with given bounds. Also leaves these
       --  bounds set as the current bounds of the Typ.
 
+      -----------
+      -- Fsize --
+      -----------
+
       function Fsize (Lov, Hiv : Ureal) return Nat is
       begin
          Set_Realval (Lo, Lov);
@@ -2955,7 +3920,7 @@ package body Freeze is
          return Minimum_Size (Typ);
       end Fsize;
 
-   --  Start of processing for Freeze_Fixed_Point_Type;
+   --  Start of processing for Freeze_Fixed_Point_Type
 
    begin
       --  If Esize of a subtype has not previously been set, set it now
@@ -2964,15 +3929,15 @@ package body Freeze is
          Atype := Ancestor_Subtype (Typ);
 
          if Present (Atype) then
-            Set_Size_Info (Typ, Atype);
+            Set_Esize (Typ, Esize (Atype));
          else
-            Set_Size_Info (Typ, Base_Type (Typ));
+            Set_Esize (Typ, Esize (Base_Type (Typ)));
          end if;
       end if;
 
-      --  Immediate return if the range is already analyzed. This means
-      --  that the range is already set, and does not need to be computed
-      --  by this routine.
+      --  Immediate return if the range is already analyzed. This means that
+      --  the range is already set, and does not need to be computed by this
+      --  routine.
 
       if Analyzed (Rng) then
          return;
@@ -2994,16 +3959,16 @@ package body Freeze is
       if Is_Ordinary_Fixed_Point_Type (Typ) then
 
          --  For the ordinary fixed-point case, we are allowed to fudge the
-         --  end-points up or down by small. Generally we prefer to fudge
-         --  up, i.e. widen the bounds for non-model numbers so that the
-         --  end points are included. However there are cases in which this
-         --  cannot be done, and indeed cases in which we may need to narrow
-         --  the bounds. The following circuit makes the decision.
+         --  end-points up or down by small. Generally we prefer to fudge up,
+         --  i.e. widen the bounds for non-model numbers so that the end points
+         --  are included. However there are cases in which this cannot be
+         --  done, and indeed cases in which we may need to narrow the bounds.
+         --  The following circuit makes the decision.
 
-         --  Note: our terminology here is that Incl_EP means that the
-         --  bounds are widened by Small if necessary to include the end
-         --  points, and Excl_EP means that the bounds are narrowed by
-         --  Small to exclude the end-points if this reduces the size.
+         --  Note: our terminology here is that Incl_EP means that the bounds
+         --  are widened by Small if necessary to include the end points, and
+         --  Excl_EP means that the bounds are narrowed by Small to exclude the
+         --  end-points if this reduces the size.
 
          --  Note that in the Incl case, all we care about is including the
          --  end-points. In the Excl case, we want to narrow the bounds as
@@ -3047,10 +4012,10 @@ package body Freeze is
             end if;
 
             --  Compute the fudged bounds. If the number is a model number,
-            --  then we do nothing to include it, but we are allowed to
-            --  backoff to the next adjacent model number when we exclude
-            --  it. If it is not a model number then we straddle the two
-            --  values with the model numbers on either side.
+            --  then we do nothing to include it, but we are allowed to backoff
+            --  to the next adjacent model number when we exclude it. If it is
+            --  not a model number then we straddle the two values with the
+            --  model numbers on either side.
 
             Model_Num := UR_Trunc (Loval / Small) * Small;
 
@@ -3087,11 +4052,11 @@ package body Freeze is
                Hival_Excl_EP := Hival_Incl_EP;
             end if;
 
-            --  One further adjustment is needed. In the case of subtypes,
-            --  we cannot go outside the range of the base type, or we get
+            --  One further adjustment is needed. In the case of subtypes, we
+            --  cannot go outside the range of the base type, or we get
             --  peculiarities, and the base type range is already set. This
-            --  only applies to the Incl values, since clearly the Excl
-            --  values are already as restricted as they are allowed to be.
+            --  only applies to the Incl values, since clearly the Excl values
+            --  are already as restricted as they are allowed to be.
 
             if Typ /= Btyp then
                Loval_Incl_EP := UR_Max (Loval_Incl_EP, Realval (BLo));
@@ -3183,28 +4148,26 @@ package body Freeze is
                   Actual_Hi   := Hival_Incl_EP;
                end if;
 
-               --  One pathological case: normally we never fudge a low
-               --  bound down, since it would seem to increase the size
-               --  (if it has any effect), but for ranges containing a
-               --  single value, or no values, the high bound can be
-               --  small too large. Consider:
+               --  One pathological case: normally we never fudge a low bound
+               --  down, since it would seem to increase the size (if it has
+               --  any effect), but for ranges containing single value, or no
+               --  values, the high bound can be small too large. Consider:
 
                --    type t is delta 2.0**(-14)
                --      range 131072.0 .. 0;
 
-               --  That lower bound is *just* outside the range of 32
-               --  bits, and does need fudging down in this case. Note
-               --  that the bounds will always have crossed here, since
-               --  the high bound will be fudged down if necessary, as
-               --  in the case of:
+               --  That lower bound is *just* outside the range of 32 bits, and
+               --  does need fudging down in this case. Note that the bounds
+               --  will always have crossed here, since the high bound will be
+               --  fudged down if necessary, as in the case of:
 
                --    type t is delta 2.0**(-14)
                --      range 131072.0 .. 131072.0;
 
-               --  So we can detect the situation by looking for crossed
-               --  bounds, and if the bounds are crossed, and the low
-               --  bound is greater than zero, we will always back it
-               --  off by small, since this is completely harmless.
+               --  So we detect the situation by looking for crossed bounds,
+               --  and if the bounds are crossed, and the low bound is greater
+               --  than zero, we will always back it off by small, since this
+               --  is completely harmless.
 
                if Actual_Lo > Actual_Hi then
                   if UR_Is_Positive (Actual_Lo) then
@@ -3274,9 +4237,9 @@ package body Freeze is
          Adjust_Esize_For_Alignment (Typ);
       end if;
 
-      --  If we have a base type, then expand the bounds so that they
-      --  extend to the full width of the allocated size in bits, to
-      --  avoid junk range checks on intermediate computations.
+      --  If we have a base type, then expand the bounds so that they extend to
+      --  the full width of the allocated size in bits, to avoid junk range
+      --  checks on intermediate computations.
 
       if Base_Type (Typ) = Typ then
          Set_Realval (Lo, -(Small * (Uint_2 ** (Actual_Size - 1))));
@@ -3290,9 +4253,9 @@ package body Freeze is
       Set_Analyzed (Lo, False);
       Analyze (Lo);
 
-      --  Resolve with universal fixed if the base type, and the base
-      --  type if it is a subtype. Note we can't resolve the base type
-      --  with itself, that would be a reference before definition.
+      --  Resolve with universal fixed if the base type, and the base type if
+      --  it is a subtype. Note we can't resolve the base type with itself,
+      --  that would be a reference before definition.
 
       if Typ = Btyp then
          Resolve (Lo, Universal_Fixed);
@@ -3324,9 +4287,11 @@ package body Freeze is
 
       Set_Etype (Rng, Etype (Lo));
 
-      --  Set Esize to calculated size and also set RM_Size
+      --  Set Esize to calculated size if not set already
 
-      Init_Esize (Typ, Actual_Size);
+      if Unknown_Esize (Typ) then
+         Init_Esize (Typ, Actual_Size);
+      end if;
 
       --  Set RM_Size if not already set. If already set, check value
 
@@ -3347,7 +4312,6 @@ package body Freeze is
             Set_RM_Size (Typ, Minsiz);
          end if;
       end;
-
    end Freeze_Fixed_Point_Type;
 
    ------------------
@@ -3377,10 +4341,10 @@ package body Freeze is
       --  static. This happens if the type depends on non-global objects.
 
       procedure Ensure_Expression_Is_SA (N : Node_Id);
-      --  Called to ensure that an expression used as part of a type
-      --  definition is statically allocatable, which means that the type
-      --  of the expression is statically allocatable, and the expression
-      --  is either static, or a reference to a library level constant.
+      --  Called to ensure that an expression used as part of a type definition
+      --  is statically allocatable, which means that the expression type is
+      --  statically allocatable, and the expression is either static, or a
+      --  reference to a library level constant.
 
       procedure Ensure_Type_Is_SA (Typ : Entity_Id);
       --  Called to mark a type as static, checking that it is possible
@@ -3429,8 +4393,8 @@ package body Freeze is
             return;
          end if;
 
-         --  We are also OK if the type is already marked as statically
-         --  allocated, which means we processed it before.
+         --  We are also OK if the type already marked as statically allocated,
+         --  which means we processed it before.
 
          if Is_Statically_Allocated (Typ) then
             return;
@@ -3481,7 +4445,6 @@ package body Freeze is
 
          elsif Is_Record_Type (Typ) then
             C := First_Entity (Typ);
-
             while Present (C) loop
                if Ekind (C) = E_Discriminant
                  or else Ekind (C) = E_Component
@@ -3514,6 +4477,18 @@ package body Freeze is
    begin
       Ensure_Type_Is_SA (Etype (E));
 
+      --  Reset True_Constant flag, since something strange is going on with
+      --  the scoping here, and our simple value tracing may not be sufficient
+      --  for this indication to be reliable. We kill the Constant_Value
+      --  and Last_Assignment indications for the same reason.
+
+      Set_Is_True_Constant (E, False);
+      Set_Current_Value    (E, Empty);
+
+      if Ekind (E) = E_Variable then
+         Set_Last_Assignment  (E, Empty);
+      end if;
+
    exception
       when Cannot_Be_Static =>
 
@@ -3554,6 +4529,20 @@ package body Freeze is
          end if;
       end if;
 
+      --  Reset the Pure indication on an imported subprogram unless an
+      --  explicit Pure_Function pragma was present. We do this because
+      --  otherwise it is an insidious error to call a non-pure function from
+      --  pure unit and have calls mysteriously optimized away. What happens
+      --  here is that the Import can bypass the normal check to ensure that
+      --  pure units call only pure subprograms.
+
+      if Is_Imported (E)
+        and then Is_Pure (E)
+        and then not Has_Pragma_Pure_Function (E)
+      then
+         Set_Is_Pure (E, False);
+      end if;
+
       --  For non-foreign convention subprograms, this is where we create
       --  the extra formals (for accessibility level and constrained bit
       --  information). We delay this till the freeze point precisely so
@@ -3568,6 +4557,7 @@ package body Freeze is
          if Ekind (E) = E_Procedure
            and then Is_Valued_Procedure (E)
            and then Convention (E) = Convention_Ada
+           and then Warn_On_Export_Import
          then
             Error_Msg_N
               ("?Valued_Procedure has no effect for convention Ada", E);
@@ -3579,7 +4569,7 @@ package body Freeze is
       else
          Set_Mechanisms (E);
 
-         --  For foreign conventions, do not permit return of an
+         --  For foreign conventions, warn about return of an
          --  unconstrained array.
 
          --  Note: we *do* allow a return by descriptor for the VMS case,
@@ -3595,8 +4585,8 @@ package body Freeze is
                null;
 
             --  If the return type is generic, we have emitted a warning
-            --  earlier on, and there is nothing else to check here.
-            --  Specific instantiations may lead to erroneous behavior.
+            --  earlier on, and there is nothing else to check here. Specific
+            --  instantiations may lead to erroneous behavior.
 
             elsif Is_Generic_Type (Etype (E)) then
                null;
@@ -3604,22 +4594,25 @@ package body Freeze is
             elsif Is_Array_Type (Retype)
               and then not Is_Constrained (Retype)
               and then Mechanism (E) not in Descriptor_Codes
+              and then Warn_On_Export_Import
             then
-               Error_Msg_NE
-                ("convention for& does not permit returning " &
-                  "unconstrained array type", E, E);
+               Error_Msg_N
+                ("?foreign convention function& should not return " &
+                  "unconstrained array", E);
                return;
             end if;
          end if;
 
          --  If any of the formals for an exported foreign convention
-         --  subprogram have defaults, then emit an appropriate warning
-         --  since this is odd (default cannot be used from non-Ada code)
+         --  subprogram have defaults, then emit an appropriate warning since
+         --  this is odd (default cannot be used from non-Ada code)
 
          if Is_Exported (E) then
             F := First_Formal (E);
             while Present (F) loop
-               if Present (Default_Value (F)) then
+               if Warn_On_Export_Import
+                 and then Present (Default_Value (F))
+               then
                   Error_Msg_N
                     ("?parameter cannot be defaulted in non-Ada call",
                      Default_Value (F));
@@ -3649,21 +4642,54 @@ package body Freeze is
          end if;
       end if;
 
-   end Freeze_Subprogram;
+      --  Pragma Inline_Always is disallowed for dispatching subprograms
+      --  because the address of such subprograms is saved in the dispatch
+      --  table to support dispatching calls, and dispatching calls cannot
+      --  be inlined. This is consistent with the restriction against using
+      --  'Access or 'Address on an Inline_Always subprogram.
 
-   -----------------------
-   --  Is_Fully_Defined --
-   -----------------------
+      if Is_Dispatching_Operation (E) and then Is_Always_Inlined (E) then
+         Error_Msg_N
+           ("pragma Inline_Always not allowed for dispatching subprograms", E);
+      end if;
+   end Freeze_Subprogram;
 
-   --  Should this be in Sem_Util ???
+   ----------------------
+   -- Is_Fully_Defined --
+   ----------------------
 
    function Is_Fully_Defined (T : Entity_Id) return Boolean is
    begin
       if Ekind (T) = E_Class_Wide_Type then
          return Is_Fully_Defined (Etype (T));
-      else
-         return not Is_Private_Type (T)
-           or else Present (Full_View (Base_Type (T)));
+
+      elsif Is_Array_Type (T) then
+         return Is_Fully_Defined (Component_Type (T));
+
+      elsif Is_Record_Type (T)
+        and not Is_Private_Type (T)
+      then
+         --  Verify that the record type has no components with private types
+         --  without completion.
+
+         declare
+            Comp : Entity_Id;
+
+         begin
+            Comp := First_Component (T);
+
+            while Present (Comp) loop
+               if not Is_Fully_Defined (Etype (Comp)) then
+                  return False;
+               end if;
+
+               Next_Component (Comp);
+            end loop;
+            return True;
+         end;
+
+      else return not Is_Private_Type (T)
+        or else Present (Full_View (Base_Type (T)));
       end if;
    end Is_Fully_Defined;
 
@@ -3684,10 +4710,10 @@ package body Freeze is
    begin
       Set_Default_Expressions_Processed (E);
 
-      --  A subprogram instance and its associated anonymous subprogram
-      --  share their signature. The default expression functions are defined
-      --  in the wrapper packages for the anonymous subprogram, and should
-      --  not be generated again for the instance.
+      --  A subprogram instance and its associated anonymous subprogram share
+      --  their signature. The default expression functions are defined in the
+      --  wrapper packages for the anonymous subprogram, and should not be
+      --  generated again for the instance.
 
       if Is_Generic_Instance (E)
         and then Present (Alias (E))
@@ -3724,15 +4750,13 @@ package body Freeze is
               or else (Nkind (Dcopy) = N_Attribute_Reference
                         and then
                        Attribute_Name (Dcopy) = Name_Null_Parameter)
-
             then
 
                --  If there is no default function, we must still do a full
-               --  analyze call on the default value, to ensure that all
-               --  error checks are performed, e.g. those associated with
-               --  static evaluation. Note that this branch will always be
-               --  taken if the analyzer is turned off (but we still need the
-               --  error checks).
+               --  analyze call on the default value, to ensure that all error
+               --  checks are performed, e.g. those associated with static
+               --  evaluation. Note: this branch will always be taken if the
+               --  analyzer is turned off (but we still need the error checks).
 
                --  Note: the setting of parent here is to meet the requirement
                --  that we can only analyze the expression while attached to
@@ -3746,7 +4770,7 @@ package body Freeze is
                --  context is generic, to avoid anomalies with private types.
 
                if Ekind (Scope (E)) = E_Generic_Package then
-                  Resolve (Dcopy, Etype (Dcopy));
+                  Resolve (Dcopy);
                else
                   Resolve (Dcopy, Etype (Formal));
                end if;
@@ -3897,7 +4921,173 @@ package body Freeze is
             Set_Debug_Info_Needed (Corresponding_Record_Type (T));
          end if;
       end if;
-
    end Set_Debug_Info_Needed;
 
+   ------------------
+   -- Undelay_Type --
+   ------------------
+
+   procedure Undelay_Type (T : Entity_Id) is
+   begin
+      Set_Has_Delayed_Freeze (T, False);
+      Set_Freeze_Node (T, Empty);
+
+      --  Since we don't want T to have a Freeze_Node, we don't want its
+      --  Full_View or Corresponding_Record_Type to have one either.
+
+      --  ??? Fundamentally, this whole handling is a kludge. What we really
+      --  want is to be sure that for an Itype that's part of record R and is a
+      --  subtype of type T, that it's frozen after the later of the freeze
+      --  points of R and T. We have no way of doing that directly, so what we
+      --  do is force most such Itypes to be frozen as part of freezing R via
+      --  this procedure and only delay the ones that need to be delayed
+      --  (mostly the designated types of access types that are defined as part
+      --  of the record).
+
+      if Is_Private_Type (T)
+        and then Present (Full_View (T))
+        and then Is_Itype (Full_View (T))
+        and then Is_Record_Type (Scope (Full_View (T)))
+      then
+         Undelay_Type (Full_View (T));
+      end if;
+
+      if Is_Concurrent_Type (T)
+        and then Present (Corresponding_Record_Type (T))
+        and then Is_Itype (Corresponding_Record_Type (T))
+        and then Is_Record_Type (Scope (Corresponding_Record_Type (T)))
+      then
+         Undelay_Type (Corresponding_Record_Type (T));
+      end if;
+   end Undelay_Type;
+
+   ------------------
+   -- Warn_Overlay --
+   ------------------
+
+   procedure Warn_Overlay
+     (Expr : Node_Id;
+      Typ  : Entity_Id;
+      Nam  : Entity_Id)
+   is
+      Ent : constant Entity_Id := Entity (Nam);
+      --  The object to which the address clause applies
+
+      Init : Node_Id;
+      Old  : Entity_Id := Empty;
+      Decl : Node_Id;
+
+   begin
+      --  No warning if address clause overlay warnings are off
+
+      if not Address_Clause_Overlay_Warnings then
+         return;
+      end if;
+
+      --  No warning if there is an explicit initialization
+
+      Init := Original_Node (Expression (Declaration_Node (Ent)));
+
+      if Present (Init) and then Comes_From_Source (Init) then
+         return;
+      end if;
+
+      --  We only give the warning for non-imported entities of a type for
+      --  which a non-null base init proc is defined (or for access types which
+      --  have implicit null initialization).
+
+      if Present (Expr)
+        and then (Has_Non_Null_Base_Init_Proc (Typ)
+                    or else Is_Access_Type (Typ))
+        and then not Is_Imported (Ent)
+      then
+         if Nkind (Expr) = N_Attribute_Reference
+           and then Is_Entity_Name (Prefix (Expr))
+         then
+            Old := Entity (Prefix (Expr));
+
+         elsif Is_Entity_Name (Expr)
+           and then Ekind (Entity (Expr)) = E_Constant
+         then
+            Decl := Declaration_Node (Entity (Expr));
+
+            if Nkind (Decl) = N_Object_Declaration
+              and then Present (Expression (Decl))
+              and then Nkind (Expression (Decl)) = N_Attribute_Reference
+              and then Is_Entity_Name (Prefix (Expression (Decl)))
+            then
+               Old := Entity (Prefix (Expression (Decl)));
+
+            elsif Nkind (Expr) = N_Function_Call then
+               return;
+            end if;
+
+         --  A function call (most likely to To_Address) is probably not an
+         --  overlay, so skip warning. Ditto if the function call was inlined
+         --  and transformed into an entity.
+
+         elsif Nkind (Original_Node (Expr)) = N_Function_Call then
+            return;
+         end if;
+
+         Decl := Next (Parent (Expr));
+
+         --  If a pragma Import follows, we assume that it is for the current
+         --  target of the address clause, and skip the warning.
+
+         if Present (Decl)
+           and then Nkind (Decl) = N_Pragma
+           and then Chars (Decl) = Name_Import
+         then
+            return;
+         end if;
+
+         if Present (Old) then
+            Error_Msg_Node_2 := Old;
+            Error_Msg_N
+              ("default initialization of & may modify &?",
+               Nam);
+         else
+            Error_Msg_N
+              ("default initialization of & may modify overlaid storage?",
+               Nam);
+         end if;
+
+         --  Add friendly warning if initialization comes from a packed array
+         --  component.
+
+         if Is_Record_Type (Typ)  then
+            declare
+               Comp : Entity_Id;
+
+            begin
+               Comp := First_Component (Typ);
+
+               while Present (Comp) loop
+                  if Nkind (Parent (Comp)) = N_Component_Declaration
+                    and then Present (Expression (Parent (Comp)))
+                  then
+                     exit;
+                  elsif Is_Array_Type (Etype (Comp))
+                     and then Present (Packed_Array_Type (Etype (Comp)))
+                  then
+                     Error_Msg_NE
+                       ("\packed array component& " &
+                        "will be initialized to zero?",
+                        Nam, Comp);
+                     exit;
+                  else
+                     Next_Component (Comp);
+                  end if;
+               end loop;
+            end;
+         end if;
+
+         Error_Msg_N
+           ("\use pragma Import for & to " &
+            "suppress initialization ('R'M B.1(24))?",
+            Nam);
+      end if;
+   end Warn_Overlay;
+
 end Freeze;