OSDN Git Service

gcc/ada/
[pf3gnuchains/gcc-fork.git] / gcc / ada / exp_ch4.adb
index 33c6f14..30e08fd 100644 (file)
@@ -6,23 +6,20 @@
 --                                                                          --
 --                                 B o d y                                  --
 --                                                                          --
---                            $Revision$
---                                                                          --
---          Copyright (C) 1992-2001, Free Software Foundation, Inc.         --
+--          Copyright (C) 1992-2007, 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- --
--- ware  Foundation;  either version 2,  or (at your option) any later ver- --
+-- ware  Foundation;  either version 3,  or (at your option) any later ver- --
 -- sion.  GNAT is distributed in the hope that it will be useful, but WITH- --
 -- OUT ANY WARRANTY;  without even the  implied warranty of MERCHANTABILITY --
 -- 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.                                                      --
+-- Public License  distributed with GNAT; see file COPYING3.  If not, go to --
+-- http://www.gnu.org/licenses for a complete copy of the license.          --
 --                                                                          --
 -- 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,7 +29,9 @@ with Einfo;    use Einfo;
 with Elists;   use Elists;
 with Errout;   use Errout;
 with Exp_Aggr; use Exp_Aggr;
+with Exp_Atag; use Exp_Atag;
 with Exp_Ch3;  use Exp_Ch3;
+with Exp_Ch6;  use Exp_Ch6;
 with Exp_Ch7;  use Exp_Ch7;
 with Exp_Ch9;  use Exp_Ch9;
 with Exp_Disp; use Exp_Disp;
@@ -41,23 +40,29 @@ with Exp_Pakd; use Exp_Pakd;
 with Exp_Tss;  use Exp_Tss;
 with Exp_Util; use Exp_Util;
 with Exp_VFpt; use Exp_VFpt;
-with Hostparm; use Hostparm;
+with Freeze;   use Freeze;
 with Inline;   use Inline;
+with Namet;    use Namet;
 with Nlists;   use Nlists;
 with Nmake;    use Nmake;
 with Opt;      use Opt;
+with Restrict; use Restrict;
+with Rident;   use Rident;
 with Rtsfind;  use Rtsfind;
 with Sem;      use Sem;
 with Sem_Cat;  use Sem_Cat;
+with Sem_Ch3;  use Sem_Ch3;
+with Sem_Ch8;  use Sem_Ch8;
 with Sem_Ch13; use Sem_Ch13;
 with Sem_Eval; use Sem_Eval;
 with Sem_Res;  use Sem_Res;
 with Sem_Type; use Sem_Type;
 with Sem_Util; use Sem_Util;
+with Sem_Warn; use Sem_Warn;
 with Sinfo;    use Sinfo;
-with Sinfo.CN; use Sinfo.CN;
 with Snames;   use Snames;
 with Stand;    use Stand;
+with Targparm; use Targparm;
 with Tbuild;   use Tbuild;
 with Ttypes;   use Ttypes;
 with Uintp;    use Uintp;
@@ -66,37 +71,55 @@ with Validsw;  use Validsw;
 
 package body Exp_Ch4 is
 
-   ------------------------
-   --  Local Subprograms --
-   ------------------------
+   -----------------------
+   -- Local Subprograms --
+   -----------------------
 
    procedure Binary_Op_Validity_Checks (N : Node_Id);
    pragma Inline (Binary_Op_Validity_Checks);
    --  Performs validity checks for a binary operator
 
+   procedure Build_Boolean_Array_Proc_Call
+     (N   : Node_Id;
+      Op1 : Node_Id;
+      Op2 : Node_Id);
+   --  If a boolean array assignment can be done in place, build call to
+   --  corresponding library procedure.
+
+   procedure Displace_Allocator_Pointer (N : Node_Id);
+   --  Ada 2005 (AI-251): Subsidiary procedure to Expand_N_Allocator and
+   --  Expand_Allocator_Expression. Allocating class-wide interface objects
+   --  this routine displaces the pointer to the allocated object to reference
+   --  the component referencing the corresponding secondary dispatch table.
+
+   procedure Expand_Allocator_Expression (N : Node_Id);
+   --  Subsidiary to Expand_N_Allocator, for the case when the expression
+   --  is a qualified expression or an aggregate.
+
    procedure Expand_Array_Comparison (N : Node_Id);
    --  This routine handles expansion of the comparison operators (N_Op_Lt,
    --  N_Op_Le, N_Op_Gt, N_Op_Ge) when operating on an array type. The basic
    --  code for these operators is similar, differing only in the details of
-   --  the actual comparison call that is made.
+   --  the actual comparison call that is made. Special processing (call a
+   --  run-time routine)
 
    function Expand_Array_Equality
      (Nod    : Node_Id;
-      Typ    : Entity_Id;
-      A_Typ  : Entity_Id;
       Lhs    : Node_Id;
       Rhs    : Node_Id;
-      Bodies : List_Id)
-      return   Node_Id;
+      Bodies : List_Id;
+      Typ    : Entity_Id) return Node_Id;
    --  Expand an array equality into a call to a function implementing this
    --  equality, and a call to it. Loc is the location for the generated
-   --  nodes. Typ is the type of the array, and Lhs, Rhs are the array
-   --  expressions to be compared. A_Typ is the type of the arguments,
-   --  which may be a private type, in which case Typ is its full view.
+   --  nodes. Lhs and Rhs are the array expressions to be compared.
    --  Bodies is a list on which to attach bodies of local functions that
-   --  are created in the process. This is the responsability of the
+   --  are created in the process. It is the responsibility of the
    --  caller to insert those bodies at the right place. Nod provides
-   --  the Sloc value for the generated code.
+   --  the Sloc value for the generated code. Normally the types used
+   --  for the generated equality routine are taken from Lhs and Rhs.
+   --  However, in some situations of generated code, the Etype fields
+   --  of Lhs and Rhs are not set yet. In such cases, Typ supplies the
+   --  type to be used for the formal parameters.
 
    procedure Expand_Boolean_Operator (N : Node_Id);
    --  Common expansion processing for Boolean operators (And, Or, Xor)
@@ -107,14 +130,14 @@ package body Exp_Ch4 is
       Typ    : Entity_Id;
       Lhs    : Node_Id;
       Rhs    : Node_Id;
-      Bodies : List_Id)
-      return   Node_Id;
+      Bodies : List_Id) return Node_Id;
    --  Local recursive function used to expand equality for nested
    --  composite types. Used by Expand_Record/Array_Equality, Bodies
    --  is a list on which to attach bodies of local functions that are
    --  created in the process. This is the responsability of the caller
    --  to insert those bodies at the right place. Nod provides the Sloc
-   --  value for generated code.
+   --  value for generated code. Lhs and Rhs are the left and right sides
+   --  for the comparison, and Typ is the type of the arrays to compare.
 
    procedure Expand_Concatenate_Other (Cnode : Node_Id; Opnds : List_Id);
    --  This routine handles expansion of concatenation operations, where
@@ -135,14 +158,33 @@ package body Exp_Ch4 is
    --  purpose of this routine is to find the real type by looking up
    --  the tree. We also determine if the operation must be rounded.
 
+   function Get_Allocator_Final_List
+     (N    : Node_Id;
+      T    : Entity_Id;
+      PtrT : Entity_Id) return Entity_Id;
+   --  If the designated type is controlled, build final_list expression
+   --  for created object. If context is an access parameter, create a
+   --  local access type to have a usable finalization list.
+
+   function Has_Inferable_Discriminants (N : Node_Id) return Boolean;
+   --  Ada 2005 (AI-216): A view of an Unchecked_Union object has inferable
+   --  discriminants if it has a constrained nominal type, unless the object
+   --  is a component of an enclosing Unchecked_Union object that is subject
+   --  to a per-object constraint and the enclosing object lacks inferable
+   --  discriminants.
+   --
+   --  An expression of an Unchecked_Union type has inferable discriminants
+   --  if it is either a name of an object with inferable discriminants or a
+   --  qualified expression whose subtype mark denotes a constrained subtype.
+
    procedure Insert_Dereference_Action (N : Node_Id);
-   --  N is an expression whose type is an access. When the type is derived
-   --  from Checked_Pool, expands a call to the primitive 'dereference'.
+   --  N is an expression whose type is an access. When the type of the
+   --  associated storage pool is derived from Checked_Pool, generate a
+   --  call to the 'Dereference' primitive operation.
 
    function Make_Array_Comparison_Op
-     (Typ   : Entity_Id;
-      Nod   : Node_Id)
-      return  Node_Id;
+     (Typ : Entity_Id;
+      Nod : Node_Id) return Node_Id;
    --  Comparisons between arrays are expanded in line. This function
    --  produces the body of the implementation of (a > b), where a and b
    --  are one-dimensional arrays of some discrete type. The original
@@ -150,9 +192,8 @@ package body Exp_Ch4 is
    --  Nod provides the Sloc value for the generated code.
 
    function Make_Boolean_Array_Op
-     (Typ  : Entity_Id;
-      N    : Node_Id)
-      return Node_Id;
+     (Typ : Entity_Id;
+      N   : Node_Id) return Node_Id;
    --  Boolean operations on boolean arrays are expanded in line. This
    --  function produce the body for the node N, which is (a and b),
    --  (a or b), or (a xor b). It is used only the normal case and not
@@ -162,15 +203,25 @@ package body Exp_Ch4 is
    --  this by using Convert_To_Actual_Subtype if necessary).
 
    procedure Rewrite_Comparison (N : Node_Id);
-   --  N is the node for a compile time comparison. If this outcome of this
-   --  comparison can be determined at compile time, then the node N can be
-   --  rewritten with True or False. If the outcome cannot be determined at
-   --  compile time, the call has no effect.
+   --  If N is the node for a comparison whose outcome can be determined at
+   --  compile time, then the node N can be rewritten with True or False. If
+   --  the outcome cannot be determined at compile time, the call has no
+   --  effect. If N is a type conversion, then this processing is applied to
+   --  its expression. If N is neither comparison nor a type conversion, the
+   --  call has no effect.
 
    function Tagged_Membership (N : Node_Id) return Node_Id;
    --  Construct the expression corresponding to the tagged membership test.
    --  Deals with a second operand being (or not) a class-wide type.
 
+   function Safe_In_Place_Array_Op
+     (Lhs : Node_Id;
+      Op1 : Node_Id;
+      Op2 : Node_Id) return Boolean;
+   --  In the context of an assignment, where the right-hand side is a
+   --  boolean operation on arrays, check whether operation can be performed
+   --  in place.
+
    procedure Unary_Op_Validity_Checks (N : Node_Id);
    pragma Inline (Unary_Op_Validity_Checks);
    --  Performs validity checks for a unary operator
@@ -187,12 +238,816 @@ package body Exp_Ch4 is
       end if;
    end Binary_Op_Validity_Checks;
 
+   ------------------------------------
+   -- Build_Boolean_Array_Proc_Call --
+   ------------------------------------
+
+   procedure Build_Boolean_Array_Proc_Call
+     (N   : Node_Id;
+      Op1 : Node_Id;
+      Op2 : Node_Id)
+   is
+      Loc       : constant Source_Ptr := Sloc (N);
+      Kind      : constant Node_Kind := Nkind (Expression (N));
+      Target    : constant Node_Id   :=
+                    Make_Attribute_Reference (Loc,
+                      Prefix         => Name (N),
+                      Attribute_Name => Name_Address);
+
+      Arg1      : constant Node_Id := Op1;
+      Arg2      : Node_Id := Op2;
+      Call_Node : Node_Id;
+      Proc_Name : Entity_Id;
+
+   begin
+      if Kind = N_Op_Not then
+         if Nkind (Op1) in N_Binary_Op then
+
+            --  Use negated version of the binary operators
+
+            if Nkind (Op1) = N_Op_And then
+               Proc_Name := RTE (RE_Vector_Nand);
+
+            elsif Nkind (Op1) = N_Op_Or then
+               Proc_Name := RTE (RE_Vector_Nor);
+
+            else pragma Assert (Nkind (Op1) = N_Op_Xor);
+               Proc_Name := RTE (RE_Vector_Xor);
+            end if;
+
+            Call_Node :=
+              Make_Procedure_Call_Statement (Loc,
+                Name => New_Occurrence_Of (Proc_Name, Loc),
+
+                Parameter_Associations => New_List (
+                  Target,
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Left_Opnd (Op1),
+                    Attribute_Name => Name_Address),
+
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Right_Opnd (Op1),
+                    Attribute_Name => Name_Address),
+
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Left_Opnd (Op1),
+                    Attribute_Name => Name_Length)));
+
+         else
+            Proc_Name := RTE (RE_Vector_Not);
+
+            Call_Node :=
+              Make_Procedure_Call_Statement (Loc,
+                Name => New_Occurrence_Of (Proc_Name, Loc),
+                Parameter_Associations => New_List (
+                  Target,
+
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Op1,
+                    Attribute_Name => Name_Address),
+
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Op1,
+                     Attribute_Name => Name_Length)));
+         end if;
+
+      else
+         --  We use the following equivalences:
+
+         --   (not X) or  (not Y)  =  not (X and Y)  =  Nand (X, Y)
+         --   (not X) and (not Y)  =  not (X or Y)   =  Nor  (X, Y)
+         --   (not X) xor (not Y)  =  X xor Y
+         --   X       xor (not Y)  =  not (X xor Y)  =  Nxor (X, Y)
+
+         if Nkind (Op1) = N_Op_Not then
+            if Kind = N_Op_And then
+               Proc_Name := RTE (RE_Vector_Nor);
+
+            elsif Kind = N_Op_Or then
+               Proc_Name := RTE (RE_Vector_Nand);
+
+            else
+               Proc_Name := RTE (RE_Vector_Xor);
+            end if;
+
+         else
+            if Kind = N_Op_And then
+               Proc_Name := RTE (RE_Vector_And);
+
+            elsif Kind = N_Op_Or then
+               Proc_Name := RTE (RE_Vector_Or);
+
+            elsif Nkind (Op2) = N_Op_Not then
+               Proc_Name := RTE (RE_Vector_Nxor);
+               Arg2 := Right_Opnd (Op2);
+
+            else
+               Proc_Name := RTE (RE_Vector_Xor);
+            end if;
+         end if;
+
+         Call_Node :=
+           Make_Procedure_Call_Statement (Loc,
+             Name => New_Occurrence_Of (Proc_Name, Loc),
+             Parameter_Associations => New_List (
+               Target,
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Arg1,
+                    Attribute_Name => Name_Address),
+                  Make_Attribute_Reference (Loc,
+                    Prefix => Arg2,
+                    Attribute_Name => Name_Address),
+                 Make_Attribute_Reference (Loc,
+                   Prefix => Op1,
+                    Attribute_Name => Name_Length)));
+      end if;
+
+      Rewrite (N, Call_Node);
+      Analyze (N);
+
+   exception
+      when RE_Not_Available =>
+         return;
+   end Build_Boolean_Array_Proc_Call;
+
+   --------------------------------
+   -- Displace_Allocator_Pointer --
+   --------------------------------
+
+   procedure Displace_Allocator_Pointer (N : Node_Id) is
+      Loc       : constant Source_Ptr := Sloc (N);
+      Orig_Node : constant Node_Id := Original_Node (N);
+      Dtyp      : Entity_Id;
+      Etyp      : Entity_Id;
+      PtrT      : Entity_Id;
+
+   begin
+      --  Do nothing in case of VM targets: the virtual machine will handle
+      --  interfaces directly.
+
+      if VM_Target /= No_VM then
+         return;
+      end if;
+
+      pragma Assert (Nkind (N) = N_Identifier
+        and then Nkind (Orig_Node) = N_Allocator);
+
+      PtrT := Etype (Orig_Node);
+      Dtyp := Designated_Type (PtrT);
+      Etyp := Etype (Expression (Orig_Node));
+
+      if Is_Class_Wide_Type (Dtyp)
+        and then Is_Interface (Dtyp)
+      then
+         --  If the type of the allocator expression is not an interface type
+         --  we can generate code to reference the record component containing
+         --  the pointer to the secondary dispatch table.
+
+         if not Is_Interface (Etyp) then
+            declare
+               Saved_Typ : constant Entity_Id := Etype (Orig_Node);
+
+            begin
+               --  1) Get access to the allocated object
+
+               Rewrite (N,
+                 Make_Explicit_Dereference (Loc,
+                   Relocate_Node (N)));
+               Set_Etype (N, Etyp);
+               Set_Analyzed (N);
+
+               --  2) Add the conversion to displace the pointer to reference
+               --     the secondary dispatch table.
+
+               Rewrite (N, Convert_To (Dtyp, Relocate_Node (N)));
+               Analyze_And_Resolve (N, Dtyp);
+
+               --  3) The 'access to the secondary dispatch table will be used
+               --     as the value returned by the allocator.
+
+               Rewrite (N,
+                 Make_Attribute_Reference (Loc,
+                   Prefix         => Relocate_Node (N),
+                   Attribute_Name => Name_Access));
+               Set_Etype (N, Saved_Typ);
+               Set_Analyzed (N);
+            end;
+
+         --  If the type of the allocator expression is an interface type we
+         --  generate a run-time call to displace "this" to reference the
+         --  component containing the pointer to the secondary dispatch table
+         --  or else raise Constraint_Error if the actual object does not
+         --  implement the target interface. This case corresponds with the
+         --  following example:
+
+         --   function Op (Obj : Iface_1'Class) return access Ifac_2e'Class is
+         --   begin
+         --      return new Iface_2'Class'(Obj);
+         --   end Op;
+
+         else
+            Rewrite (N,
+              Unchecked_Convert_To (PtrT,
+                Make_Function_Call (Loc,
+                  Name => New_Reference_To (RTE (RE_Displace), Loc),
+                  Parameter_Associations => New_List (
+                    Unchecked_Convert_To (RTE (RE_Address),
+                      Relocate_Node (N)),
+
+                    New_Occurrence_Of
+                      (Elists.Node
+                        (First_Elmt
+                          (Access_Disp_Table (Etype (Base_Type (Dtyp))))),
+                       Loc)))));
+            Analyze_And_Resolve (N, PtrT);
+         end if;
+      end if;
+   end Displace_Allocator_Pointer;
+
+   ---------------------------------
+   -- Expand_Allocator_Expression --
+   ---------------------------------
+
+   procedure Expand_Allocator_Expression (N : Node_Id) is
+      Loc    : constant Source_Ptr := Sloc (N);
+      Exp    : constant Node_Id    := Expression (Expression (N));
+      PtrT   : constant Entity_Id  := Etype (N);
+      DesigT : constant Entity_Id  := Designated_Type (PtrT);
+
+      procedure Apply_Accessibility_Check
+        (Ref            : Node_Id;
+         Built_In_Place : Boolean := False);
+      --  Ada 2005 (AI-344): For an allocator with a class-wide designated
+      --  type, generate an accessibility check to verify that the level of
+      --  the type of the created object is not deeper than the level of the
+      --  access type. If the type of the qualified expression is class-
+      --  wide, then always generate the check (except in the case where it
+      --  is known to be unnecessary, see comment below). Otherwise, only
+      --  generate the check if the level of the qualified expression type
+      --  is statically deeper than the access type. Although the static
+      --  accessibility will generally have been performed as a legality
+      --  check, it won't have been done in cases where the allocator
+      --  appears in generic body, so a run-time check is needed in general.
+      --  One special case is when the access type is declared in the same
+      --  scope as the class-wide allocator, in which case the check can
+      --  never fail, so it need not be generated. As an open issue, there
+      --  seem to be cases where the static level associated with the
+      --  class-wide object's underlying type is not sufficient to perform
+      --  the proper accessibility check, such as for allocators in nested
+      --  subprograms or accept statements initialized by class-wide formals
+      --  when the actual originates outside at a deeper static level. The
+      --  nested subprogram case might require passing accessibility levels
+      --  along with class-wide parameters, and the task case seems to be
+      --  an actual gap in the language rules that needs to be fixed by the
+      --  ARG. ???
+
+      -------------------------------
+      -- Apply_Accessibility_Check --
+      -------------------------------
+
+      procedure Apply_Accessibility_Check
+        (Ref            : Node_Id;
+         Built_In_Place : Boolean := False)
+      is
+         Ref_Node : Node_Id;
+
+      begin
+         --  Note: we skip the accessibility check for the VM case, since
+         --  there does not seem to be any practical way of implementing it.
+
+         if Ada_Version >= Ada_05
+           and then VM_Target = No_VM
+           and then Is_Class_Wide_Type (DesigT)
+           and then not Scope_Suppress (Accessibility_Check)
+           and then
+             (Type_Access_Level (Etype (Exp)) > Type_Access_Level (PtrT)
+               or else
+                 (Is_Class_Wide_Type (Etype (Exp))
+                   and then Scope (PtrT) /= Current_Scope))
+         then
+            --  If the allocator was built in place Ref is already a reference
+            --  to the access object initialized to the result of the allocator
+            --  (see Exp_Ch6.Make_Build_In_Place_Call_In_Allocator). Otherwise
+            --  it is the entity associated with the object containing the
+            --  address of the allocated object.
+
+            if Built_In_Place then
+               Ref_Node := New_Copy (Ref);
+            else
+               Ref_Node := New_Reference_To (Ref, Loc);
+            end if;
+
+            Insert_Action (N,
+               Make_Raise_Program_Error (Loc,
+                 Condition =>
+                   Make_Op_Gt (Loc,
+                     Left_Opnd  =>
+                       Build_Get_Access_Level (Loc,
+                         Make_Attribute_Reference (Loc,
+                           Prefix => Ref_Node,
+                           Attribute_Name => Name_Tag)),
+                     Right_Opnd =>
+                       Make_Integer_Literal (Loc,
+                         Type_Access_Level (PtrT))),
+                 Reason => PE_Accessibility_Check_Failed));
+         end if;
+      end Apply_Accessibility_Check;
+
+      --  Local variables
+
+      Indic : constant Node_Id   := Subtype_Mark (Expression (N));
+      T     : constant Entity_Id := Entity (Indic);
+      Flist : Node_Id;
+      Node  : Node_Id;
+      Temp  : Entity_Id;
+
+      TagT : Entity_Id := Empty;
+      --  Type used as source for tag assignment
+
+      TagR : Node_Id := Empty;
+      --  Target reference for tag assignment
+
+      Aggr_In_Place : constant Boolean := Is_Delayed_Aggregate (Exp);
+
+      Tag_Assign : Node_Id;
+      Tmp_Node   : Node_Id;
+
+   --  Start of processing for Expand_Allocator_Expression
+
+   begin
+      if Is_Tagged_Type (T) or else Controlled_Type (T) then
+
+         --  Ada 2005 (AI-318-02): If the initialization expression is a
+         --  call to a build-in-place function, then access to the allocated
+         --  object must be passed to the function. Currently we limit such
+         --  functions to those with constrained limited result subtypes,
+         --  but eventually we plan to expand the allowed forms of funtions
+         --  that are treated as build-in-place.
+
+         if Ada_Version >= Ada_05
+           and then Is_Build_In_Place_Function_Call (Exp)
+         then
+            Make_Build_In_Place_Call_In_Allocator (N, Exp);
+            Apply_Accessibility_Check (N, Built_In_Place => True);
+            return;
+         end if;
+
+         --    Actions inserted before:
+         --              Temp : constant ptr_T := new T'(Expression);
+         --   <no CW>    Temp._tag := T'tag;
+         --   <CTRL>     Adjust (Finalizable (Temp.all));
+         --   <CTRL>     Attach_To_Final_List (Finalizable (Temp.all));
+
+         --  We analyze by hand the new internal allocator to avoid
+         --  any recursion and inappropriate call to Initialize
+
+         --  We don't want to remove side effects when the expression must be
+         --  built in place. In the case of a build-in-place function call,
+         --  that could lead to a duplication of the call, which was already
+         --  substituted for the allocator.
+
+         if not Aggr_In_Place then
+            Remove_Side_Effects (Exp);
+         end if;
+
+         Temp :=
+           Make_Defining_Identifier (Loc, New_Internal_Name ('P'));
+
+         --  For a class wide allocation generate the following code:
+
+         --    type Equiv_Record is record ... end record;
+         --    implicit subtype CW is <Class_Wide_Subytpe>;
+         --    temp : PtrT := new CW'(CW!(expr));
+
+         if Is_Class_Wide_Type (T) then
+            Expand_Subtype_From_Expr (Empty, T, Indic, Exp);
+
+            --  Ada 2005 (AI-251): If the expression is a class-wide interface
+            --  object we generate code to move up "this" to reference the
+            --  base of the object before allocating the new object.
+
+            --  Note that Exp'Address is recursively expanded into a call
+            --  to Base_Address (Exp.Tag)
+
+            if Is_Class_Wide_Type (Etype (Exp))
+              and then Is_Interface (Etype (Exp))
+              and then VM_Target = No_VM
+            then
+               Set_Expression
+                 (Expression (N),
+                  Unchecked_Convert_To (Entity (Indic),
+                    Make_Explicit_Dereference (Loc,
+                      Unchecked_Convert_To (RTE (RE_Tag_Ptr),
+                        Make_Attribute_Reference (Loc,
+                          Prefix         => Exp,
+                          Attribute_Name => Name_Address)))));
+
+            else
+               Set_Expression
+                 (Expression (N),
+                  Unchecked_Convert_To (Entity (Indic), Exp));
+            end if;
+
+            Analyze_And_Resolve (Expression (N), Entity (Indic));
+         end if;
+
+         --  Keep separate the management of allocators returning interfaces
+
+         if not Is_Interface (Directly_Designated_Type (PtrT)) then
+            if Aggr_In_Place then
+               Tmp_Node :=
+                 Make_Object_Declaration (Loc,
+                   Defining_Identifier => Temp,
+                   Object_Definition   => New_Reference_To (PtrT, Loc),
+                   Expression          =>
+                     Make_Allocator (Loc,
+                       New_Reference_To (Etype (Exp), Loc)));
+
+               Set_Comes_From_Source
+                 (Expression (Tmp_Node), Comes_From_Source (N));
+
+               Set_No_Initialization (Expression (Tmp_Node));
+               Insert_Action (N, Tmp_Node);
+
+               if Controlled_Type (T)
+                 and then Ekind (PtrT) = E_Anonymous_Access_Type
+               then
+                  --  Create local finalization list for access parameter
+
+                  Flist := Get_Allocator_Final_List (N, Base_Type (T), PtrT);
+               end if;
+
+               Convert_Aggr_In_Allocator (N, Tmp_Node, Exp);
+            else
+               Node := Relocate_Node (N);
+               Set_Analyzed (Node);
+               Insert_Action (N,
+                 Make_Object_Declaration (Loc,
+                   Defining_Identifier => Temp,
+                   Constant_Present    => True,
+                   Object_Definition   => New_Reference_To (PtrT, Loc),
+                   Expression          => Node));
+            end if;
+
+         --  Ada 2005 (AI-251): Handle allocators whose designated type is an
+         --  interface type. In this case we use the type of the qualified
+         --  expression to allocate the object.
+
+         else
+            declare
+               Def_Id   : constant Entity_Id :=
+                            Make_Defining_Identifier (Loc,
+                              New_Internal_Name ('T'));
+               New_Decl : Node_Id;
+
+            begin
+               New_Decl :=
+                 Make_Full_Type_Declaration (Loc,
+                   Defining_Identifier => Def_Id,
+                   Type_Definition =>
+                     Make_Access_To_Object_Definition (Loc,
+                       All_Present            => True,
+                       Null_Exclusion_Present => False,
+                       Constant_Present       => False,
+                       Subtype_Indication     =>
+                         New_Reference_To (Etype (Exp), Loc)));
+
+               Insert_Action (N, New_Decl);
+
+               --  Inherit the final chain to ensure that the expansion of the
+               --  aggregate is correct in case of controlled types
+
+               if Controlled_Type (Directly_Designated_Type (PtrT)) then
+                  Set_Associated_Final_Chain (Def_Id,
+                    Associated_Final_Chain (PtrT));
+               end if;
+
+               --  Declare the object using the previous type declaration
+
+               if Aggr_In_Place then
+                  Tmp_Node :=
+                    Make_Object_Declaration (Loc,
+                      Defining_Identifier => Temp,
+                      Object_Definition   => New_Reference_To (Def_Id, Loc),
+                      Expression          =>
+                        Make_Allocator (Loc,
+                          New_Reference_To (Etype (Exp), Loc)));
+
+                  Set_Comes_From_Source
+                    (Expression (Tmp_Node), Comes_From_Source (N));
+
+                  Set_No_Initialization (Expression (Tmp_Node));
+                  Insert_Action (N, Tmp_Node);
+
+                  if Controlled_Type (T)
+                    and then Ekind (PtrT) = E_Anonymous_Access_Type
+                  then
+                     --  Create local finalization list for access parameter
+
+                     Flist :=
+                       Get_Allocator_Final_List (N, Base_Type (T), PtrT);
+                  end if;
+
+                  Convert_Aggr_In_Allocator (N, Tmp_Node, Exp);
+               else
+                  Node := Relocate_Node (N);
+                  Set_Analyzed (Node);
+                  Insert_Action (N,
+                    Make_Object_Declaration (Loc,
+                      Defining_Identifier => Temp,
+                      Constant_Present    => True,
+                      Object_Definition   => New_Reference_To (Def_Id, Loc),
+                      Expression          => Node));
+               end if;
+
+               --  Generate an additional object containing the address of the
+               --  returned object. The type of this second object declaration
+               --  is the correct type required for the common proceessing
+               --  that is still performed by this subprogram. The displacement
+               --  of this pointer to reference the component associated with
+               --  the interface type will be done at the end of the common
+               --  processing.
+
+               New_Decl :=
+                 Make_Object_Declaration (Loc,
+                   Defining_Identifier => Make_Defining_Identifier (Loc,
+                                             New_Internal_Name ('P')),
+                   Object_Definition   => New_Reference_To (PtrT, Loc),
+                   Expression          => Unchecked_Convert_To (PtrT,
+                                            New_Reference_To (Temp, Loc)));
+
+               Insert_Action (N, New_Decl);
+
+               Tmp_Node := New_Decl;
+               Temp     := Defining_Identifier (New_Decl);
+            end;
+         end if;
+
+         Apply_Accessibility_Check (Temp);
+
+         --  Generate the tag assignment
+
+         --  Suppress the tag assignment when VM_Target because VM tags are
+         --  represented implicitly in objects.
+
+         if VM_Target /= No_VM then
+            null;
+
+         --  Ada 2005 (AI-251): Suppress the tag assignment with class-wide
+         --  interface objects because in this case the tag does not change.
+
+         elsif Is_Interface (Directly_Designated_Type (Etype (N))) then
+            pragma Assert (Is_Class_Wide_Type
+                            (Directly_Designated_Type (Etype (N))));
+            null;
+
+         elsif Is_Tagged_Type (T) and then not Is_Class_Wide_Type (T) then
+            TagT := T;
+            TagR := New_Reference_To (Temp, Loc);
+
+         elsif Is_Private_Type (T)
+           and then Is_Tagged_Type (Underlying_Type (T))
+         then
+            TagT := Underlying_Type (T);
+            TagR :=
+              Unchecked_Convert_To (Underlying_Type (T),
+                Make_Explicit_Dereference (Loc,
+                  Prefix => New_Reference_To (Temp, Loc)));
+         end if;
+
+         if Present (TagT) then
+            Tag_Assign :=
+              Make_Assignment_Statement (Loc,
+                Name =>
+                  Make_Selected_Component (Loc,
+                    Prefix => TagR,
+                    Selector_Name =>
+                      New_Reference_To (First_Tag_Component (TagT), Loc)),
+
+                Expression =>
+                  Unchecked_Convert_To (RTE (RE_Tag),
+                    New_Reference_To
+                      (Elists.Node (First_Elmt (Access_Disp_Table (TagT))),
+                       Loc)));
+
+            --  The previous assignment has to be done in any case
+
+            Set_Assignment_OK (Name (Tag_Assign));
+            Insert_Action (N, Tag_Assign);
+         end if;
+
+         if Controlled_Type (DesigT)
+            and then Controlled_Type (T)
+         then
+            declare
+               Attach : Node_Id;
+               Apool  : constant Entity_Id :=
+                          Associated_Storage_Pool (PtrT);
+
+            begin
+               --  If it is an allocation on the secondary stack
+               --  (i.e. a value returned from a function), the object
+               --  is attached on the caller side as soon as the call
+               --  is completed (see Expand_Ctrl_Function_Call)
+
+               if Is_RTE (Apool, RE_SS_Pool) then
+                  declare
+                     F : constant Entity_Id :=
+                           Make_Defining_Identifier (Loc,
+                             New_Internal_Name ('F'));
+                  begin
+                     Insert_Action (N,
+                       Make_Object_Declaration (Loc,
+                         Defining_Identifier => F,
+                         Object_Definition   => New_Reference_To (RTE
+                          (RE_Finalizable_Ptr), Loc)));
+
+                     Flist := New_Reference_To (F, Loc);
+                     Attach :=  Make_Integer_Literal (Loc, 1);
+                  end;
+
+               --  Normal case, not a secondary stack allocation
+
+               else
+                  if Controlled_Type (T)
+                    and then Ekind (PtrT) = E_Anonymous_Access_Type
+                  then
+                     --  Create local finalization list for access parameter
+
+                     Flist :=
+                       Get_Allocator_Final_List (N, Base_Type (T), PtrT);
+                  else
+                     Flist := Find_Final_List (PtrT);
+                  end if;
+
+                  Attach :=  Make_Integer_Literal (Loc, 2);
+               end if;
+
+               --  Generate an Adjust call if the object will be moved. In Ada
+               --  2005, the object may be inherently limited, in which case
+               --  there is no Adjust procedure, and the object is built in
+               --  place. In Ada 95, the object can be limited but not
+               --  inherently limited if this allocator came from a return
+               --  statement (we're allocating the result on the secondary
+               --  stack). In that case, the object will be moved, so we _do_
+               --  want to Adjust.
+
+               if not Aggr_In_Place
+                 and then not Is_Inherently_Limited_Type (T)
+               then
+                  Insert_Actions (N,
+                    Make_Adjust_Call (
+                      Ref          =>
+
+                     --  An unchecked conversion is needed in the
+                     --  classwide case because the designated type
+                     --  can be an ancestor of the subtype mark of
+                     --  the allocator.
+
+                      Unchecked_Convert_To (T,
+                        Make_Explicit_Dereference (Loc,
+                          Prefix => New_Reference_To (Temp, Loc))),
+
+                      Typ          => T,
+                      Flist_Ref    => Flist,
+                      With_Attach  => Attach,
+                      Allocator    => True));
+               end if;
+            end;
+         end if;
+
+         Rewrite (N, New_Reference_To (Temp, Loc));
+         Analyze_And_Resolve (N, PtrT);
+
+         --  Ada 2005 (AI-251): Displace the pointer to reference the
+         --  record component containing the secondary dispatch table
+         --  of the interface type.
+
+         if Is_Interface (Directly_Designated_Type (PtrT)) then
+            Displace_Allocator_Pointer (N);
+         end if;
+
+      elsif Aggr_In_Place then
+         Temp :=
+           Make_Defining_Identifier (Loc, New_Internal_Name ('P'));
+         Tmp_Node :=
+           Make_Object_Declaration (Loc,
+             Defining_Identifier => Temp,
+             Object_Definition   => New_Reference_To (PtrT, Loc),
+             Expression          => Make_Allocator (Loc,
+                 New_Reference_To (Etype (Exp), Loc)));
+
+         Set_Comes_From_Source
+           (Expression (Tmp_Node), Comes_From_Source (N));
+
+         Set_No_Initialization (Expression (Tmp_Node));
+         Insert_Action (N, Tmp_Node);
+         Convert_Aggr_In_Allocator (N, Tmp_Node, Exp);
+         Rewrite (N, New_Reference_To (Temp, Loc));
+         Analyze_And_Resolve (N, PtrT);
+
+      elsif Is_Access_Type (DesigT)
+        and then Nkind (Exp) = N_Allocator
+        and then Nkind (Expression (Exp)) /= N_Qualified_Expression
+      then
+         --  Apply constraint to designated subtype indication
+
+         Apply_Constraint_Check (Expression (Exp),
+           Designated_Type (DesigT),
+           No_Sliding => True);
+
+         if Nkind (Expression (Exp)) = N_Raise_Constraint_Error then
+
+            --  Propagate constraint_error to enclosing allocator
+
+            Rewrite (Exp, New_Copy (Expression (Exp)));
+         end if;
+      else
+         --  First check against the type of the qualified expression
+         --
+         --  NOTE: The commented call should be correct, but for
+         --  some reason causes the compiler to bomb (sigsegv) on
+         --  ACVC test c34007g, so for now we just perform the old
+         --  (incorrect) test against the designated subtype with
+         --  no sliding in the else part of the if statement below.
+         --  ???
+         --
+         --  Apply_Constraint_Check (Exp, T, No_Sliding => True);
+
+         --  A check is also needed in cases where the designated
+         --  subtype is constrained and differs from the subtype
+         --  given in the qualified expression. Note that the check
+         --  on the qualified expression does not allow sliding,
+         --  but this check does (a relaxation from Ada 83).
+
+         if Is_Constrained (DesigT)
+           and then not Subtypes_Statically_Match
+                          (T, DesigT)
+         then
+            Apply_Constraint_Check
+              (Exp, DesigT, No_Sliding => False);
+
+         --  The nonsliding check should really be performed
+         --  (unconditionally) against the subtype of the
+         --  qualified expression, but that causes a problem
+         --  with c34007g (see above), so for now we retain this.
+
+         else
+            Apply_Constraint_Check
+              (Exp, DesigT, No_Sliding => True);
+         end if;
+
+         --  For an access to unconstrained packed array, GIGI needs
+         --  to see an expression with a constrained subtype in order
+         --  to compute the proper size for the allocator.
+
+         if Is_Array_Type (T)
+           and then not Is_Constrained (T)
+           and then Is_Packed (T)
+         then
+            declare
+               ConstrT      : constant Entity_Id :=
+                                Make_Defining_Identifier (Loc,
+                                  Chars => New_Internal_Name ('A'));
+               Internal_Exp : constant Node_Id   := Relocate_Node (Exp);
+            begin
+               Insert_Action (Exp,
+                 Make_Subtype_Declaration (Loc,
+                   Defining_Identifier => ConstrT,
+                   Subtype_Indication  =>
+                     Make_Subtype_From_Expr (Exp, T)));
+               Freeze_Itype (ConstrT, Exp);
+               Rewrite (Exp, OK_Convert_To (ConstrT, Internal_Exp));
+            end;
+         end if;
+
+         --  Ada 2005 (AI-318-02): If the initialization expression is a
+         --  call to a build-in-place function, then access to the allocated
+         --  object must be passed to the function. Currently we limit such
+         --  functions to those with constrained limited result subtypes,
+         --  but eventually we plan to expand the allowed forms of funtions
+         --  that are treated as build-in-place.
+
+         if Ada_Version >= Ada_05
+           and then Is_Build_In_Place_Function_Call (Exp)
+         then
+            Make_Build_In_Place_Call_In_Allocator (N, Exp);
+         end if;
+      end if;
+
+   exception
+      when RE_Not_Available =>
+         return;
+   end Expand_Allocator_Expression;
+
    -----------------------------
    -- Expand_Array_Comparison --
    -----------------------------
 
-   --  Expansion is only required in the case of array types. The form of
-   --  the expansion is:
+   --  Expansion is only required in the case of array types. For the
+   --  unpacked case, an appropriate runtime routine is called. For
+   --  packed cases, and also in some other cases where a runtime
+   --  routine cannot be called, the form of the expansion is:
 
    --     [body for greater_nn; boolean_expression]
 
@@ -204,12 +1059,157 @@ package body Exp_Ch4 is
       Op1  : Node_Id             := Left_Opnd (N);
       Op2  : Node_Id             := Right_Opnd (N);
       Typ1 : constant Entity_Id  := Base_Type (Etype (Op1));
+      Ctyp : constant Entity_Id  := Component_Type (Typ1);
 
       Expr      : Node_Id;
       Func_Body : Node_Id;
       Func_Name : Entity_Id;
 
+      Comp : RE_Id;
+
+      Byte_Addressable : constant Boolean := System_Storage_Unit = Byte'Size;
+      --  True for byte addressable target
+
+      function Length_Less_Than_4 (Opnd : Node_Id) return Boolean;
+      --  Returns True if the length of the given operand is known to be
+      --  less than 4. Returns False if this length is known to be four
+      --  or greater or is not known at compile time.
+
+      ------------------------
+      -- Length_Less_Than_4 --
+      ------------------------
+
+      function Length_Less_Than_4 (Opnd : Node_Id) return Boolean is
+         Otyp : constant Entity_Id := Etype (Opnd);
+
+      begin
+         if Ekind (Otyp) = E_String_Literal_Subtype then
+            return String_Literal_Length (Otyp) < 4;
+
+         else
+            declare
+               Ityp : constant Entity_Id := Etype (First_Index (Otyp));
+               Lo   : constant Node_Id   := Type_Low_Bound (Ityp);
+               Hi   : constant Node_Id   := Type_High_Bound (Ityp);
+               Lov  : Uint;
+               Hiv  : Uint;
+
+            begin
+               if Compile_Time_Known_Value (Lo) then
+                  Lov := Expr_Value (Lo);
+               else
+                  return False;
+               end if;
+
+               if Compile_Time_Known_Value (Hi) then
+                  Hiv := Expr_Value (Hi);
+               else
+                  return False;
+               end if;
+
+               return Hiv < Lov + 3;
+            end;
+         end if;
+      end Length_Less_Than_4;
+
+   --  Start of processing for Expand_Array_Comparison
+
    begin
+      --  Deal first with unpacked case, where we can call a runtime routine
+      --  except that we avoid this for targets for which are not addressable
+      --  by bytes, and for the JVM/CIL, since they do not support direct
+      --  addressing of array components.
+
+      if not Is_Bit_Packed_Array (Typ1)
+        and then Byte_Addressable
+        and then VM_Target = No_VM
+      then
+         --  The call we generate is:
+
+         --  Compare_Array_xn[_Unaligned]
+         --    (left'address, right'address, left'length, right'length) <op> 0
+
+         --  x = U for unsigned, S for signed
+         --  n = 8,16,32,64 for component size
+         --  Add _Unaligned if length < 4 and component size is 8.
+         --  <op> is the standard comparison operator
+
+         if Component_Size (Typ1) = 8 then
+            if Length_Less_Than_4 (Op1)
+                 or else
+               Length_Less_Than_4 (Op2)
+            then
+               if Is_Unsigned_Type (Ctyp) then
+                  Comp := RE_Compare_Array_U8_Unaligned;
+               else
+                  Comp := RE_Compare_Array_S8_Unaligned;
+               end if;
+
+            else
+               if Is_Unsigned_Type (Ctyp) then
+                  Comp := RE_Compare_Array_U8;
+               else
+                  Comp := RE_Compare_Array_S8;
+               end if;
+            end if;
+
+         elsif Component_Size (Typ1) = 16 then
+            if Is_Unsigned_Type (Ctyp) then
+               Comp := RE_Compare_Array_U16;
+            else
+               Comp := RE_Compare_Array_S16;
+            end if;
+
+         elsif Component_Size (Typ1) = 32 then
+            if Is_Unsigned_Type (Ctyp) then
+               Comp := RE_Compare_Array_U32;
+            else
+               Comp := RE_Compare_Array_S32;
+            end if;
+
+         else pragma Assert (Component_Size (Typ1) = 64);
+            if Is_Unsigned_Type (Ctyp) then
+               Comp := RE_Compare_Array_U64;
+            else
+               Comp := RE_Compare_Array_S64;
+            end if;
+         end if;
+
+         Remove_Side_Effects (Op1, Name_Req => True);
+         Remove_Side_Effects (Op2, Name_Req => True);
+
+         Rewrite (Op1,
+           Make_Function_Call (Sloc (Op1),
+             Name => New_Occurrence_Of (RTE (Comp), Loc),
+
+             Parameter_Associations => New_List (
+               Make_Attribute_Reference (Loc,
+                 Prefix         => Relocate_Node (Op1),
+                 Attribute_Name => Name_Address),
+
+               Make_Attribute_Reference (Loc,
+                 Prefix         => Relocate_Node (Op2),
+                 Attribute_Name => Name_Address),
+
+               Make_Attribute_Reference (Loc,
+                 Prefix         => Relocate_Node (Op1),
+                 Attribute_Name => Name_Length),
+
+               Make_Attribute_Reference (Loc,
+                 Prefix         => Relocate_Node (Op2),
+                 Attribute_Name => Name_Length))));
+
+         Rewrite (Op2,
+           Make_Integer_Literal (Sloc (Op2),
+             Intval => Uint_0));
+
+         Analyze_And_Resolve (Op1, Standard_Integer);
+         Analyze_And_Resolve (Op2, Standard_Integer);
+         return;
+      end if;
+
+      --  Cases where we cannot make runtime call
+
       --  For (a <= b) we convert to not (a > b)
 
       if Chars (N) = Name_Op_Le then
@@ -263,6 +1263,9 @@ package body Exp_Ch4 is
       Rewrite (N, Expr);
       Analyze_And_Resolve (N, Standard_Boolean);
 
+   exception
+      when RE_Not_Available =>
+         return;
    end Expand_Array_Comparison;
 
    ---------------------------
@@ -272,66 +1275,163 @@ package body Exp_Ch4 is
    --  Expand an equality function for multi-dimensional arrays. Here is
    --  an example of such a function for Nb_Dimension = 2
 
-   --  function Enn (A : arr; B : arr) return boolean is
-   --     J1 : integer;
-   --     J2 : integer;
-   --
+   --  function Enn (A : atyp; B : btyp) return boolean is
    --  begin
-   --     if A'length (1) /= B'length (1) then
-   --        return false;
-   --     else
-   --        J1 := B'first (1);
-   --        for I1 in A'first (1) .. A'last (1) loop
-   --           if A'length (2) /= B'length (2) then
-   --              return false;
-   --           else
-   --              J2 := B'first (2);
-   --              for I2 in A'first (2) .. A'last (2) loop
-   --                 if A (I1, I2) /=  B (J1, J2) then
-   --                    return false;
+   --     if (A'length (1) = 0 or else A'length (2) = 0)
+   --          and then
+   --        (B'length (1) = 0 or else B'length (2) = 0)
+   --     then
+   --        return True;    -- RM 4.5.2(22)
+   --     end if;
+
+   --     if A'length (1) /= B'length (1)
+   --               or else
+   --           A'length (2) /= B'length (2)
+   --     then
+   --        return False;   -- RM 4.5.2(23)
+   --     end if;
+
+   --     declare
+   --        A1 : Index_T1 := A'first (1);
+   --        B1 : Index_T1 := B'first (1);
+   --     begin
+   --        loop
+   --           declare
+   --              A2 : Index_T2 := A'first (2);
+   --              B2 : Index_T2 := B'first (2);
+   --           begin
+   --              loop
+   --                 if A (A1, A2) /= B (B1, B2) then
+   --                    return False;
    --                 end if;
-   --                 J2 := Integer'succ (J2);
+
+   --                 exit when A2 = A'last (2);
+   --                 A2 := Index_T2'succ (A2);
+   --                 B2 := Index_T2'succ (B2);
    --              end loop;
-   --           end if;
-   --           J1 := Integer'succ (J1);
+   --           end;
+
+   --           exit when A1 = A'last (1);
+   --           A1 := Index_T1'succ (A1);
+   --           B1 := Index_T1'succ (B1);
    --        end loop;
-   --     end if;
+   --     end;
+
    --     return true;
    --  end Enn;
 
+   --  Note on the formal types used (atyp and btyp). If either of the
+   --  arrays is of a private type, we use the underlying type, and
+   --  do an unchecked conversion of the actual. If either of the arrays
+   --  has a bound depending on a discriminant, then we use the base type
+   --  since otherwise we have an escaped discriminant in the function.
+
+   --  If both arrays are constrained and have the same bounds, we can
+   --  generate a loop with an explicit iteration scheme using a 'Range
+   --  attribute over the first array.
+
    function Expand_Array_Equality
      (Nod    : Node_Id;
-      Typ    : Entity_Id;
-      A_Typ  : Entity_Id;
       Lhs    : Node_Id;
       Rhs    : Node_Id;
-      Bodies : List_Id)
-      return   Node_Id
+      Bodies : List_Id;
+      Typ    : Entity_Id) return Node_Id
    is
       Loc         : constant Source_Ptr := Sloc (Nod);
-      Actuals     : List_Id;
-      Decls       : List_Id := New_List;
-      Index_List1 : List_Id := New_List;
-      Index_List2 : List_Id := New_List;
-      Formals     : List_Id;
-      Stats       : Node_Id;
-      Func_Name   : Entity_Id;
-      Func_Body   : Node_Id;
+      Decls       : constant List_Id    := New_List;
+      Index_List1 : constant List_Id    := New_List;
+      Index_List2 : constant List_Id    := New_List;
+
+      Actuals   : List_Id;
+      Formals   : List_Id;
+      Func_Name : Entity_Id;
+      Func_Body : Node_Id;
 
       A : constant Entity_Id := Make_Defining_Identifier (Loc, Name_uA);
       B : constant Entity_Id := Make_Defining_Identifier (Loc, Name_uB);
 
+      Ltyp : Entity_Id;
+      Rtyp : Entity_Id;
+      --  The parameter types to be used for the formals
+
+      function Arr_Attr
+        (Arr : Entity_Id;
+         Nam : Name_Id;
+         Num : Int) return Node_Id;
+      --  This builds the attribute reference Arr'Nam (Expr)
+
       function Component_Equality (Typ : Entity_Id) return Node_Id;
-      --  Create one statement to compare corresponding components, designated
-      --  by a full set of indices.
+      --  Create one statement to compare corresponding components,
+      --  designated by a full set of indices.
 
-      function Loop_One_Dimension
+      function Get_Arg_Type (N : Node_Id) return Entity_Id;
+      --  Given one of the arguments, computes the appropriate type to
+      --  be used for that argument in the corresponding function formal
+
+      function Handle_One_Dimension
         (N     : Int;
-         Index : Node_Id)
-         return Node_Id;
-      --  Loop over the n'th dimension of the arrays. The single statement
-      --  in the body of the loop is a loop over the next dimension, or
-      --  the comparison of corresponding components.
+         Index : Node_Id) return Node_Id;
+      --  This procedure returns the following code
+      --
+      --    declare
+      --       Bn : Index_T := B'First (N);
+      --    begin
+      --       loop
+      --          xxx
+      --          exit when An = A'Last (N);
+      --          An := Index_T'Succ (An)
+      --          Bn := Index_T'Succ (Bn)
+      --       end loop;
+      --    end;
+      --
+      --  If both indices are constrained and identical, the procedure
+      --  returns a simpler loop:
+      --
+      --      for An in A'Range (N) loop
+      --         xxx
+      --      end loop
+      --
+      --  N is the dimension for which we are generating a loop. Index is the
+      --  N'th index node, whose Etype is Index_Type_n in the above code.
+      --  The xxx statement is either the loop or declare for the next
+      --  dimension or if this is the last dimension the comparison
+      --  of corresponding components of the arrays.
+      --
+      --  The actual way the code works is to return the comparison
+      --  of corresponding components for the N+1 call. That's neater!
+
+      function Test_Empty_Arrays return Node_Id;
+      --  This function constructs the test for both arrays being empty
+      --    (A'length (1) = 0 or else A'length (2) = 0 or else ...)
+      --      and then
+      --    (B'length (1) = 0 or else B'length (2) = 0 or else ...)
+
+      function Test_Lengths_Correspond return Node_Id;
+      --  This function constructs the test for arrays having different
+      --  lengths in at least one index position, in which case resull
+
+      --     A'length (1) /= B'length (1)
+      --       or else
+      --     A'length (2) /= B'length (2)
+      --       or else
+      --       ...
+
+      --------------
+      -- Arr_Attr --
+      --------------
+
+      function Arr_Attr
+        (Arr : Entity_Id;
+         Nam : Name_Id;
+         Num : Int) return Node_Id
+      is
+      begin
+         return
+           Make_Attribute_Reference (Loc,
+            Attribute_Name => Nam,
+            Prefix => New_Reference_To (Arr, Loc),
+            Expressions => New_List (Make_Integer_Literal (Loc, Num)));
+      end Arr_Attr;
 
       ------------------------
       -- Component_Equality --
@@ -357,141 +1457,287 @@ package body Exp_Ch4 is
          Test := Expand_Composite_Equality
                    (Nod, Component_Type (Typ), L, R, Decls);
 
-         return
-           Make_Implicit_If_Statement (Nod,
-             Condition => Make_Op_Not (Loc, Right_Opnd => Test),
-             Then_Statements => New_List (
-               Make_Return_Statement (Loc,
-                 Expression => New_Occurrence_Of (Standard_False, Loc))));
+         --  If some (sub)component is an unchecked_union, the whole operation
+         --  will raise program error.
+
+         if Nkind (Test) = N_Raise_Program_Error then
+
+            --  This node is going to be inserted at a location where a
+            --  statement is expected: clear its Etype so analysis will
+            --  set it to the expected Standard_Void_Type.
 
+            Set_Etype (Test, Empty);
+            return Test;
+
+         else
+            return
+              Make_Implicit_If_Statement (Nod,
+                Condition => Make_Op_Not (Loc, Right_Opnd => Test),
+                Then_Statements => New_List (
+                  Make_Simple_Return_Statement (Loc,
+                    Expression => New_Occurrence_Of (Standard_False, Loc))));
+         end if;
       end Component_Equality;
 
-      ------------------------
-      -- Loop_One_Dimension --
-      ------------------------
+      ------------------
+      -- Get_Arg_Type --
+      ------------------
+
+      function Get_Arg_Type (N : Node_Id) return Entity_Id is
+         T : Entity_Id;
+         X : Node_Id;
+
+      begin
+         T := Etype (N);
+
+         if No (T) then
+            return Typ;
 
-      function Loop_One_Dimension
+         else
+            T := Underlying_Type (T);
+
+            X := First_Index (T);
+            while Present (X) loop
+               if Denotes_Discriminant (Type_Low_Bound (Etype (X)))
+                 or else
+                   Denotes_Discriminant (Type_High_Bound (Etype (X)))
+               then
+                  T := Base_Type (T);
+                  exit;
+               end if;
+
+               Next_Index (X);
+            end loop;
+
+            return T;
+         end if;
+      end Get_Arg_Type;
+
+      --------------------------
+      -- Handle_One_Dimension --
+      ---------------------------
+
+      function Handle_One_Dimension
         (N     : Int;
-         Index : Node_Id)
-         return Node_Id
+         Index : Node_Id) return Node_Id
       is
-         I : constant Entity_Id := Make_Defining_Identifier (Loc,
-                                                  New_Internal_Name ('I'));
-         J : constant Entity_Id := Make_Defining_Identifier (Loc,
-                                                  New_Internal_Name ('J'));
-         Index_Type  : Entity_Id;
-         Stats : Node_Id;
+         Need_Separate_Indexes : constant Boolean :=
+                                   Ltyp /= Rtyp
+                                     or else not Is_Constrained (Ltyp);
+         --  If the index types are identical, and we are working with
+         --  constrained types, then we can use the same index for both of
+         --  the arrays.
+
+         An : constant Entity_Id := Make_Defining_Identifier (Loc,
+                                      Chars => New_Internal_Name ('A'));
+
+         Bn       : Entity_Id;
+         Index_T  : Entity_Id;
+         Stm_List : List_Id;
+         Loop_Stm : Node_Id;
 
       begin
-         if N > Number_Dimensions (Typ) then
-            return Component_Equality (Typ);
+         if N > Number_Dimensions (Ltyp) then
+            return Component_Equality (Ltyp);
+         end if;
+
+         --  Case where we generate a loop
 
+         Index_T := Base_Type (Etype (Index));
+
+         if Need_Separate_Indexes then
+            Bn :=
+              Make_Defining_Identifier (Loc,
+                Chars => New_Internal_Name ('B'));
          else
-            --  Generate the following:
+            Bn := An;
+         end if;
 
-            --  j: index_type;
-            --  ...
+         Append (New_Reference_To (An, Loc), Index_List1);
+         Append (New_Reference_To (Bn, Loc), Index_List2);
 
-            --  if a'length (n) /= b'length (n) then
-            --    return false;
-            --  else
-            --     j := b'first (n);
-            --     for i in a'range (n) loop
-            --        --  loop over remaining dimensions.
-            --        j := index_type'succ (j);
-            --     end loop;
-            --  end if;
+         Stm_List := New_List (
+           Handle_One_Dimension (N + 1, Next_Index (Index)));
 
-            --  retrieve index type for current dimension.
+         if Need_Separate_Indexes then
 
-            Index_Type := Base_Type (Etype (Index));
-            Append (New_Reference_To (I, Loc), Index_List1);
-            Append (New_Reference_To (J, Loc), Index_List2);
+            --  Generate guard for loop, followed by increments of indices
 
-            --  Declare index for j as a local variable to the function.
-            --  Index i is a loop variable.
+            Append_To (Stm_List,
+               Make_Exit_Statement (Loc,
+                 Condition =>
+                   Make_Op_Eq (Loc,
+                      Left_Opnd => New_Reference_To (An, Loc),
+                      Right_Opnd => Arr_Attr (A, Name_Last, N))));
 
-            Append_To (Decls,
-              Make_Object_Declaration (Loc,
-                Defining_Identifier => J,
-                Object_Definition   => New_Reference_To (Index_Type, Loc)));
+            Append_To (Stm_List,
+              Make_Assignment_Statement (Loc,
+                Name       => New_Reference_To (An, Loc),
+                Expression =>
+                  Make_Attribute_Reference (Loc,
+                    Prefix         => New_Reference_To (Index_T, Loc),
+                    Attribute_Name => Name_Succ,
+                    Expressions    => New_List (New_Reference_To (An, Loc)))));
 
-            Stats :=
-              Make_Implicit_If_Statement (Nod,
-                Condition =>
-                  Make_Op_Ne (Loc,
-                    Left_Opnd =>
-                      Make_Attribute_Reference (Loc,
-                        Prefix => New_Reference_To (A, Loc),
-                        Attribute_Name => Name_Length,
-                        Expressions => New_List (
-                          Make_Integer_Literal (Loc, N))),
-                    Right_Opnd =>
-                      Make_Attribute_Reference (Loc,
-                        Prefix => New_Reference_To (B, Loc),
-                        Attribute_Name => Name_Length,
-                        Expressions => New_List (
-                          Make_Integer_Literal (Loc, N)))),
+            Append_To (Stm_List,
+              Make_Assignment_Statement (Loc,
+                Name       => New_Reference_To (Bn, Loc),
+                Expression =>
+                  Make_Attribute_Reference (Loc,
+                    Prefix         => New_Reference_To (Index_T, Loc),
+                    Attribute_Name => Name_Succ,
+                    Expressions    => New_List (New_Reference_To (Bn, Loc)))));
+         end if;
 
-                Then_Statements => New_List (
-                  Make_Return_Statement (Loc,
-                    Expression => New_Occurrence_Of (Standard_False, Loc))),
+         --  If separate indexes, we need a declare block for An and Bn, and a
+         --  loop without an iteration scheme.
 
-                Else_Statements => New_List (
+         if Need_Separate_Indexes then
+            Loop_Stm :=
+              Make_Implicit_Loop_Statement (Nod, Statements => Stm_List);
 
-                  Make_Assignment_Statement (Loc,
-                    Name       => New_Reference_To (J, Loc),
-                    Expression =>
-                      Make_Attribute_Reference (Loc,
-                        Prefix => New_Reference_To (B, Loc),
-                        Attribute_Name => Name_First,
-                        Expressions => New_List (
-                          Make_Integer_Literal (Loc, N)))),
-
-                  Make_Implicit_Loop_Statement (Nod,
-                    Identifier => Empty,
-                    Iteration_Scheme =>
-                      Make_Iteration_Scheme (Loc,
-                        Loop_Parameter_Specification =>
-                          Make_Loop_Parameter_Specification (Loc,
-                            Defining_Identifier => I,
-                            Discrete_Subtype_Definition =>
-                              Make_Attribute_Reference (Loc,
-                                Prefix => New_Reference_To (A, Loc),
-                                Attribute_Name => Name_Range,
-                                Expressions => New_List (
-                                  Make_Integer_Literal (Loc, N))))),
-
-                    Statements => New_List (
-                      Loop_One_Dimension (N + 1, Next_Index (Index)),
-                      Make_Assignment_Statement (Loc,
-                        Name => New_Reference_To (J, Loc),
-                        Expression =>
-                          Make_Attribute_Reference (Loc,
-                            Prefix => New_Reference_To (Index_Type, Loc),
-                            Attribute_Name => Name_Succ,
-                            Expressions => New_List (
-                              New_Reference_To (J, Loc))))))));
+            return
+              Make_Block_Statement (Loc,
+                Declarations => New_List (
+                  Make_Object_Declaration (Loc,
+                    Defining_Identifier => An,
+                    Object_Definition   => New_Reference_To (Index_T, Loc),
+                    Expression          => Arr_Attr (A, Name_First, N)),
+
+                  Make_Object_Declaration (Loc,
+                    Defining_Identifier => Bn,
+                    Object_Definition   => New_Reference_To (Index_T, Loc),
+                    Expression          => Arr_Attr (B, Name_First, N))),
 
-            return Stats;
+                Handled_Statement_Sequence =>
+                  Make_Handled_Sequence_Of_Statements (Loc,
+                    Statements => New_List (Loop_Stm)));
+
+         --  If no separate indexes, return loop statement with explicit
+         --  iteration scheme on its own
+
+         else
+            Loop_Stm :=
+              Make_Implicit_Loop_Statement (Nod,
+                Statements       => Stm_List,
+                Iteration_Scheme =>
+                  Make_Iteration_Scheme (Loc,
+                    Loop_Parameter_Specification =>
+                      Make_Loop_Parameter_Specification (Loc,
+                        Defining_Identifier         => An,
+                        Discrete_Subtype_Definition =>
+                          Arr_Attr (A, Name_Range, N))));
+            return Loop_Stm;
          end if;
-      end Loop_One_Dimension;
+      end Handle_One_Dimension;
+
+      -----------------------
+      -- Test_Empty_Arrays --
+      -----------------------
+
+      function Test_Empty_Arrays return Node_Id is
+         Alist : Node_Id;
+         Blist : Node_Id;
+
+         Atest : Node_Id;
+         Btest : Node_Id;
+
+      begin
+         Alist := Empty;
+         Blist := Empty;
+         for J in 1 .. Number_Dimensions (Ltyp) loop
+            Atest :=
+              Make_Op_Eq (Loc,
+                Left_Opnd  => Arr_Attr (A, Name_Length, J),
+                Right_Opnd => Make_Integer_Literal (Loc, 0));
+
+            Btest :=
+              Make_Op_Eq (Loc,
+                Left_Opnd  => Arr_Attr (B, Name_Length, J),
+                Right_Opnd => Make_Integer_Literal (Loc, 0));
+
+            if No (Alist) then
+               Alist := Atest;
+               Blist := Btest;
+
+            else
+               Alist :=
+                 Make_Or_Else (Loc,
+                   Left_Opnd  => Relocate_Node (Alist),
+                   Right_Opnd => Atest);
+
+               Blist :=
+                 Make_Or_Else (Loc,
+                   Left_Opnd  => Relocate_Node (Blist),
+                   Right_Opnd => Btest);
+            end if;
+         end loop;
+
+         return
+           Make_And_Then (Loc,
+             Left_Opnd  => Alist,
+             Right_Opnd => Blist);
+      end Test_Empty_Arrays;
+
+      -----------------------------
+      -- Test_Lengths_Correspond --
+      -----------------------------
+
+      function Test_Lengths_Correspond return Node_Id is
+         Result : Node_Id;
+         Rtest  : Node_Id;
+
+      begin
+         Result := Empty;
+         for J in 1 .. Number_Dimensions (Ltyp) loop
+            Rtest :=
+              Make_Op_Ne (Loc,
+                Left_Opnd  => Arr_Attr (A, Name_Length, J),
+                Right_Opnd => Arr_Attr (B, Name_Length, J));
+
+            if No (Result) then
+               Result := Rtest;
+            else
+               Result :=
+                 Make_Or_Else (Loc,
+                   Left_Opnd  => Relocate_Node (Result),
+                   Right_Opnd => Rtest);
+            end if;
+         end loop;
+
+         return Result;
+      end Test_Lengths_Correspond;
 
    --  Start of processing for Expand_Array_Equality
 
    begin
+      Ltyp := Get_Arg_Type (Lhs);
+      Rtyp := Get_Arg_Type (Rhs);
+
+      --  For now, if the argument types are not the same, go to the
+      --  base type, since the code assumes that the formals have the
+      --  same type. This is fixable in future ???
+
+      if Ltyp /= Rtyp then
+         Ltyp := Base_Type (Ltyp);
+         Rtyp := Base_Type (Rtyp);
+         pragma Assert (Ltyp = Rtyp);
+      end if;
+
+      --  Build list of formals for function
+
       Formals := New_List (
         Make_Parameter_Specification (Loc,
           Defining_Identifier => A,
-          Parameter_Type      => New_Reference_To (Typ, Loc)),
+          Parameter_Type      => New_Reference_To (Ltyp, Loc)),
 
         Make_Parameter_Specification (Loc,
           Defining_Identifier => B,
-          Parameter_Type      => New_Reference_To (Typ, Loc)));
+          Parameter_Type      => New_Reference_To (Rtyp, Loc)));
 
       Func_Name := Make_Defining_Identifier (Loc,  New_Internal_Name ('E'));
 
-      Stats := Loop_One_Dimension (1, First_Index (Typ));
+      --  Build statement sequence for function
 
       Func_Body :=
         Make_Subprogram_Body (Loc,
@@ -499,34 +1745,67 @@ package body Exp_Ch4 is
             Make_Function_Specification (Loc,
               Defining_Unit_Name       => Func_Name,
               Parameter_Specifications => Formals,
-              Subtype_Mark => New_Reference_To (Standard_Boolean, Loc)),
-          Declarations               =>  Decls,
+              Result_Definition => New_Reference_To (Standard_Boolean, Loc)),
+
+          Declarations =>  Decls,
+
           Handled_Statement_Sequence =>
             Make_Handled_Sequence_Of_Statements (Loc,
               Statements => New_List (
-                Stats,
-                Make_Return_Statement (Loc,
+
+                Make_Implicit_If_Statement (Nod,
+                  Condition => Test_Empty_Arrays,
+                  Then_Statements => New_List (
+                    Make_Simple_Return_Statement (Loc,
+                      Expression =>
+                        New_Occurrence_Of (Standard_True, Loc)))),
+
+                Make_Implicit_If_Statement (Nod,
+                  Condition => Test_Lengths_Correspond,
+                  Then_Statements => New_List (
+                    Make_Simple_Return_Statement (Loc,
+                      Expression =>
+                        New_Occurrence_Of (Standard_False, Loc)))),
+
+                Handle_One_Dimension (1, First_Index (Ltyp)),
+
+                Make_Simple_Return_Statement (Loc,
                   Expression => New_Occurrence_Of (Standard_True, Loc)))));
 
          Set_Has_Completion (Func_Name, True);
+         Set_Is_Inlined (Func_Name);
 
          --  If the array type is distinct from the type of the arguments,
          --  it is the full view of a private type. Apply an unchecked
          --  conversion to insure that analysis of the call succeeds.
 
-         if Base_Type (A_Typ) /= Base_Type (Typ) then
-            Actuals := New_List (
-              OK_Convert_To (Typ, Lhs),
-              OK_Convert_To (Typ, Rhs));
-         else
-            Actuals := New_List (Lhs, Rhs);
-         end if;
+         declare
+            L, R : Node_Id;
+
+         begin
+            L := Lhs;
+            R := Rhs;
+
+            if No (Etype (Lhs))
+              or else Base_Type (Etype (Lhs)) /= Base_Type (Ltyp)
+            then
+               L := OK_Convert_To (Ltyp, Lhs);
+            end if;
+
+            if No (Etype (Rhs))
+              or else Base_Type (Etype (Rhs)) /= Base_Type (Rtyp)
+            then
+               R := OK_Convert_To (Rtyp, Rhs);
+            end if;
+
+            Actuals := New_List (L, R);
+         end;
 
          Append_To (Bodies, Func_Body);
 
          return
            Make_Function_Call (Loc,
-             Name => New_Reference_To (Func_Name, Loc),
+             Name                   => New_Reference_To (Func_Name, Loc),
              Parameter_Associations => Actuals);
    end Expand_Array_Equality;
 
@@ -538,32 +1817,53 @@ package body Exp_Ch4 is
    --  since we always want to deal with types that have bounds.
 
    procedure Expand_Boolean_Operator (N : Node_Id) is
-      Typ       : constant Entity_Id  := Etype (N);
+      Typ : constant Entity_Id  := Etype (N);
 
    begin
-      if Is_Bit_Packed_Array (Typ) then
+      --  Special case of bit packed array where both operands are known
+      --  to be properly aligned. In this case we use an efficient run time
+      --  routine to carry out the operation (see System.Bit_Ops).
+
+      if Is_Bit_Packed_Array (Typ)
+        and then not Is_Possibly_Unaligned_Object (Left_Opnd (N))
+        and then not Is_Possibly_Unaligned_Object (Right_Opnd (N))
+      then
          Expand_Packed_Boolean_Operator (N);
+         return;
+      end if;
 
-      else
+      --  For the normal non-packed case, the general expansion is to build
+      --  function for carrying out the comparison (use Make_Boolean_Array_Op)
+      --  and then inserting it into the tree. The original operator node is
+      --  then rewritten as a call to this function. We also use this in the
+      --  packed case if either operand is a possibly unaligned object.
 
-         --  For the normal non-packed case, the expansion is
-         --  to build a function for carrying out the comparison
-         --  (using Make_Boolean_Array_Op) and then inserting it
-         --  into the tree. The original operator node is then
-         --  rewritten as a call to this function.
+      declare
+         Loc       : constant Source_Ptr := Sloc (N);
+         L         : constant Node_Id    := Relocate_Node (Left_Opnd  (N));
+         R         : constant Node_Id    := Relocate_Node (Right_Opnd (N));
+         Func_Body : Node_Id;
+         Func_Name : Entity_Id;
 
-         declare
-            Loc       : constant Source_Ptr := Sloc (N);
-            L         : constant Node_Id    := Relocate_Node (Left_Opnd  (N));
-            R         : constant Node_Id    := Relocate_Node (Right_Opnd (N));
-            Func_Body : Node_Id;
-            Func_Name : Entity_Id;
-         begin
-            Convert_To_Actual_Subtype (L);
-            Convert_To_Actual_Subtype (R);
-            Ensure_Defined (Etype (L), N);
-            Ensure_Defined (Etype (R), N);
-            Apply_Length_Check (R, Etype (L));
+      begin
+         Convert_To_Actual_Subtype (L);
+         Convert_To_Actual_Subtype (R);
+         Ensure_Defined (Etype (L), N);
+         Ensure_Defined (Etype (R), N);
+         Apply_Length_Check (R, Etype (L));
+
+         if Nkind (Parent (N)) = N_Assignment_Statement
+           and then Safe_In_Place_Array_Op (Name (Parent (N)), L, R)
+         then
+            Build_Boolean_Array_Proc_Call (Parent (N), L, R);
+
+         elsif Nkind (Parent (N)) = N_Op_Not
+           and then Nkind (N) = N_Op_And
+           and then
+         Safe_In_Place_Array_Op (Name (Parent (Parent (N))), L, R)
+         then
+            return;
+         else
 
             Func_Body := Make_Boolean_Array_Op (Etype (L), N);
             Func_Name := Defining_Unit_Name (Specification (Func_Body));
@@ -573,15 +1873,16 @@ package body Exp_Ch4 is
 
             Rewrite (N,
               Make_Function_Call (Loc,
-                Name => New_Reference_To (Func_Name, Loc),
+                Name                   => New_Reference_To (Func_Name, Loc),
                 Parameter_Associations =>
-                  New_List
-                    (L, Make_Type_Conversion
-                          (Loc, New_Reference_To (Etype (L), Loc), R))));
+                  New_List (
+                    L,
+                    Make_Type_Conversion
+                      (Loc, New_Reference_To (Etype (L), Loc), R))));
 
             Analyze_And_Resolve (N, Typ);
-         end;
-      end if;
+         end if;
+      end;
    end Expand_Boolean_Operator;
 
    -------------------------------
@@ -597,8 +1898,7 @@ package body Exp_Ch4 is
       Typ    : Entity_Id;
       Lhs    : Node_Id;
       Rhs    : Node_Id;
-      Bodies : List_Id)
-      return   Node_Id
+      Bodies : List_Id) return Node_Id
    is
       Loc       : constant Source_Ptr := Sloc (Nod);
       Full_Type : Entity_Id;
@@ -640,8 +1940,7 @@ package body Exp_Ch4 is
          --  case of any composite type recursively containing such fields.
 
          else
-            return Expand_Array_Equality
-                     (Nod, Full_Type, Typ, Lhs, Rhs, Bodies);
+            return Expand_Array_Equality (Nod, Lhs, Rhs, Bodies, Full_Type);
          end if;
 
       elsif Is_Tagged_Type (Full_Type) then
@@ -673,7 +1972,8 @@ package body Exp_Ch4 is
             Eq_Op := Node (Prim);
             exit when Chars (Eq_Op) = Name_Op_Eq
               and then Etype (First_Formal (Eq_Op)) =
-                       Etype (Next_Formal (First_Formal (Eq_Op)));
+                       Etype (Next_Formal (First_Formal (Eq_Op)))
+              and then Base_Type (Etype (Eq_Op)) = Standard_Boolean;
             Next_Elmt (Prim);
             pragma Assert (Present (Prim));
          end loop;
@@ -689,7 +1989,7 @@ package body Exp_Ch4 is
                   Unchecked_Convert_To (Etype (First_Formal (Eq_Op)), Rhs)));
 
       elsif Is_Record_Type (Full_Type) then
-         Eq_Op := TSS (Full_Type, Name_uEquality);
+         Eq_Op := TSS (Full_Type, TSS_Composite_Equality);
 
          if Present (Eq_Op) then
             if Etype (First_Formal (Eq_Op)) /= Full_Type then
@@ -698,7 +1998,7 @@ package body Exp_Ch4 is
                --  to match signature of operation.
 
                declare
-                  T : Entity_Id := Etype (First_Formal (Eq_Op));
+                  T : constant Entity_Id := Etype (First_Formal (Eq_Op));
 
                begin
                   return
@@ -710,6 +2010,116 @@ package body Exp_Ch4 is
                end;
 
             else
+               --  Comparison between Unchecked_Union components
+
+               if Is_Unchecked_Union (Full_Type) then
+                  declare
+                     Lhs_Type      : Node_Id := Full_Type;
+                     Rhs_Type      : Node_Id := Full_Type;
+                     Lhs_Discr_Val : Node_Id;
+                     Rhs_Discr_Val : Node_Id;
+
+                  begin
+                     --  Lhs subtype
+
+                     if Nkind (Lhs) = N_Selected_Component then
+                        Lhs_Type := Etype (Entity (Selector_Name (Lhs)));
+                     end if;
+
+                     --  Rhs subtype
+
+                     if Nkind (Rhs) = N_Selected_Component then
+                        Rhs_Type := Etype (Entity (Selector_Name (Rhs)));
+                     end if;
+
+                     --  Lhs of the composite equality
+
+                     if Is_Constrained (Lhs_Type) then
+
+                        --  Since the enclosing record can never be an
+                        --  Unchecked_Union (this code is executed for records
+                        --  that do not have variants), we may reference its
+                        --  discriminant(s).
+
+                        if Nkind (Lhs) = N_Selected_Component
+                          and then Has_Per_Object_Constraint (
+                                     Entity (Selector_Name (Lhs)))
+                        then
+                           Lhs_Discr_Val :=
+                             Make_Selected_Component (Loc,
+                               Prefix => Prefix (Lhs),
+                               Selector_Name =>
+                                 New_Copy (
+                                   Get_Discriminant_Value (
+                                     First_Discriminant (Lhs_Type),
+                                     Lhs_Type,
+                                     Stored_Constraint (Lhs_Type))));
+
+                        else
+                           Lhs_Discr_Val := New_Copy (
+                             Get_Discriminant_Value (
+                               First_Discriminant (Lhs_Type),
+                               Lhs_Type,
+                               Stored_Constraint (Lhs_Type)));
+
+                        end if;
+                     else
+                        --  It is not possible to infer the discriminant since
+                        --  the subtype is not constrained.
+
+                        return
+                          Make_Raise_Program_Error (Loc,
+                            Reason => PE_Unchecked_Union_Restriction);
+                     end if;
+
+                     --  Rhs of the composite equality
+
+                     if Is_Constrained (Rhs_Type) then
+                        if Nkind (Rhs) = N_Selected_Component
+                          and then Has_Per_Object_Constraint (
+                                     Entity (Selector_Name (Rhs)))
+                        then
+                           Rhs_Discr_Val :=
+                             Make_Selected_Component (Loc,
+                               Prefix => Prefix (Rhs),
+                               Selector_Name =>
+                                 New_Copy (
+                                   Get_Discriminant_Value (
+                                     First_Discriminant (Rhs_Type),
+                                     Rhs_Type,
+                                     Stored_Constraint (Rhs_Type))));
+
+                        else
+                           Rhs_Discr_Val := New_Copy (
+                             Get_Discriminant_Value (
+                               First_Discriminant (Rhs_Type),
+                               Rhs_Type,
+                               Stored_Constraint (Rhs_Type)));
+
+                        end if;
+                     else
+                        return
+                          Make_Raise_Program_Error (Loc,
+                            Reason => PE_Unchecked_Union_Restriction);
+                     end if;
+
+                     --  Call the TSS equality function with the inferred
+                     --  discriminant values.
+
+                     return
+                       Make_Function_Call (Loc,
+                         Name => New_Reference_To (Eq_Op, Loc),
+                         Parameter_Associations => New_List (
+                           Lhs,
+                           Rhs,
+                           Lhs_Discr_Val,
+                           Rhs_Discr_Val));
+                  end;
+               end if;
+
+               --  Shouldn't this be an else, we can't fall through
+               --  the above IF, right???
+
                return
                  Make_Function_Call (Loc,
                    Name => New_Reference_To (Eq_Op, Loc),
@@ -735,7 +2145,7 @@ package body Exp_Ch4 is
    --  their base type, Ind_Typ their index type, and Arr_Typ the original
    --  array type to which the concatenantion operator applies, then the
    --  following subprogram is constructed:
-   --
+
    --  [function Cnn (S1 : Base_Typ; ...; Sn : Base_Typ) return Base_Typ is
    --      L : Ind_Typ;
    --   begin
@@ -752,7 +2162,7 @@ package body Exp_Ch4 is
    --      else
    --         return Sn;
    --      end if;
-   --
+
    --      declare
    --         P : Ind_Typ;
    --         H : Ind_Typ :=
@@ -779,9 +2189,9 @@ package body Exp_Ch4 is
    --               P := Ind_Typ'Succ (P);
    --            end loop;
    --         end if;
-   --
+
    --         ...
-   --
+
    --         if Sn'Length /= 0 then
    --            P := Sn'First;
    --            loop
@@ -791,7 +2201,7 @@ package body Exp_Ch4 is
    --               P := Ind_Typ'Succ (P);
    --            end loop;
    --         end if;
-   --
+
    --         return R;
    --      end;
    --   end Cnn;]
@@ -831,7 +2241,7 @@ package body Exp_Ch4 is
       Params  : List_Id;
       Operand : Node_Id;
 
-      function Copy_Into_R_S (I : Nat) return List_Id;
+      function Copy_Into_R_S (I : Nat; Last : Boolean) return List_Id;
       --  Builds the sequence of statement:
       --    P := Si'First;
       --    loop
@@ -842,6 +2252,9 @@ package body Exp_Ch4 is
       --    end loop;
       --
       --  where i is the input parameter I given.
+      --  If the flag Last is true, the exit statement is emitted before
+      --  incrementing the lower bound, to prevent the creation out of
+      --  bound values.
 
       function Init_L (I : Nat) return Node_Id;
       --  Builds the statement:
@@ -849,53 +2262,55 @@ package body Exp_Ch4 is
       --    L := Si'First;       otherwise (where I is the input param given)
 
       function H return Node_Id;
-      --  Builds reference to identifier H.
+      --  Builds reference to identifier H
 
       function Ind_Val (E : Node_Id) return Node_Id;
       --  Builds expression Ind_Typ'Val (E);
 
       function L return Node_Id;
-      --  Builds reference to identifier L.
+      --  Builds reference to identifier L
 
       function L_Pos return Node_Id;
-      --  Builds expression Ind_Typ'Pos (L).
+      --  Builds expression Integer_Type'(Ind_Typ'Pos (L)). We qualify the
+      --  expression to avoid universal_integer computations whenever possible,
+      --  in the expression for the upper bound H.
 
       function L_Succ return Node_Id;
-      --  Builds expression Ind_Typ'Succ (L).
+      --  Builds expression Ind_Typ'Succ (L)
 
       function One return Node_Id;
-      --  Builds integer literal one.
+      --  Builds integer literal one
 
       function P return Node_Id;
-      --  Builds reference to identifier P.
+      --  Builds reference to identifier P
 
       function P_Succ return Node_Id;
-      --  Builds expression Ind_Typ'Succ (P).
+      --  Builds expression Ind_Typ'Succ (P)
 
       function R return Node_Id;
-      --  Builds reference to identifier R.
+      --  Builds reference to identifier R
 
       function S (I : Nat) return Node_Id;
-      --  Builds reference to identifier Si, where I is the value given.
+      --  Builds reference to identifier Si, where I is the value given
 
       function S_First (I : Nat) return Node_Id;
-      --  Builds expression Si'First, where I is the value given.
+      --  Builds expression Si'First, where I is the value given
 
       function S_Last (I : Nat) return Node_Id;
-      --  Builds expression Si'Last, where I is the value given.
+      --  Builds expression Si'Last, where I is the value given
 
       function S_Length (I : Nat) return Node_Id;
-      --  Builds expression Si'Length, where I is the value given.
+      --  Builds expression Si'Length, where I is the value given
 
       function S_Length_Test (I : Nat) return Node_Id;
-      --  Builds expression Si'Length /= 0, where I is the value given.
+      --  Builds expression Si'Length /= 0, where I is the value given
 
       -------------------
       -- Copy_Into_R_S --
       -------------------
 
-      function Copy_Into_R_S (I : Nat) return List_Id is
-         Stmts     : List_Id := New_List;
+      function Copy_Into_R_S (I : Nat; Last : Boolean) return List_Id is
+         Stmts     : constant List_Id := New_List;
          P_Start   : Node_Id;
          Loop_Stmt : Node_Id;
          R_Copy    : Node_Id;
@@ -932,9 +2347,15 @@ package body Exp_Ch4 is
                     Name       => P,
                     Expression => P_Succ);
 
-         Loop_Stmt :=
-           Make_Implicit_Loop_Statement (Cnode,
-             Statements => New_List (R_Copy, L_Inc, Exit_Stmt, P_Inc));
+         if Last then
+            Loop_Stmt :=
+              Make_Implicit_Loop_Statement (Cnode,
+                Statements => New_List (R_Copy, Exit_Stmt, L_Inc, P_Inc));
+         else
+            Loop_Stmt :=
+              Make_Implicit_Loop_Statement (Cnode,
+                Statements => New_List (R_Copy, L_Inc, Exit_Stmt, P_Inc));
+         end if;
 
          Append_To (Stmts, Loop_Stmt);
 
@@ -997,12 +2418,31 @@ package body Exp_Ch4 is
       -----------
 
       function L_Pos return Node_Id is
+         Target_Type : Entity_Id;
+
       begin
+         --  If the index type is an enumeration type, the computation
+         --  can be done in standard integer. Otherwise, choose a large
+         --  enough integer type.
+
+         if Is_Enumeration_Type (Ind_Typ)
+           or else Root_Type (Ind_Typ) = Standard_Integer
+           or else Root_Type (Ind_Typ) = Standard_Short_Integer
+           or else Root_Type (Ind_Typ) = Standard_Short_Short_Integer
+         then
+            Target_Type := Standard_Integer;
+         else
+            Target_Type := Root_Type (Ind_Typ);
+         end if;
+
          return
-           Make_Attribute_Reference (Loc,
-             Prefix         => New_Reference_To (Ind_Typ, Loc),
-             Attribute_Name => Name_Pos,
-             Expressions    => New_List (L));
+           Make_Qualified_Expression (Loc,
+              Subtype_Mark => New_Reference_To (Target_Type, Loc),
+              Expression   =>
+                Make_Attribute_Reference (Loc,
+                  Prefix         => New_Reference_To (Ind_Typ, Loc),
+                  Attribute_Name => Name_Pos,
+                  Expressions    => New_List (L)));
       end L_Pos;
 
       ------------
@@ -1132,7 +2572,7 @@ package body Exp_Ch4 is
         Make_Function_Specification (Loc,
           Defining_Unit_Name       => Func_Id,
           Parameter_Specifications => Param_Specs,
-          Subtype_Mark             => New_Reference_To (Base_Typ, Loc));
+          Result_Definition        => New_Reference_To (Base_Typ, Loc));
 
       --  Construct L's object declaration
 
@@ -1157,7 +2597,7 @@ package body Exp_Ch4 is
           Condition       => S_Length_Test (1),
           Then_Statements => New_List (Init_L (1)),
           Elsif_Parts     => Elsif_List,
-          Else_Statements => New_List (Make_Return_Statement (Loc,
+          Else_Statements => New_List (Make_Simple_Return_Statement (Loc,
                                          Expression => S (Nb_Opnds))));
 
       --  Construct the declaration for H
@@ -1205,10 +2645,11 @@ package body Exp_Ch4 is
          Append_To (Declare_Stmts,
                     Make_Implicit_If_Statement (Cnode,
                       Condition       => S_Length_Test (I),
-                      Then_Statements => Copy_Into_R_S (I)));
+                      Then_Statements => Copy_Into_R_S (I, I = Nb_Opnds)));
       end loop;
 
-      Append_To (Declare_Stmts, Make_Return_Statement (Loc, Expression => R));
+      Append_To
+        (Declare_Stmts, Make_Simple_Return_Statement (Loc, Expression => R));
 
       --  Construct the declare block
 
@@ -1235,7 +2676,7 @@ package body Exp_Ch4 is
 
       --  Note that this does *not* fix the array concatenation bug when the
       --  low bound is Integer'first sibce that bug comes from the pointer
-      --  derefencing an unconstrained array. An there we need a constraint
+      --  dereferencing an unconstrained array. An there we need a constraint
       --  check to make sure the length of the concatenated array is ok. ???
 
       Insert_Action (Cnode, Func_Body, Suppress => All_Checks);
@@ -1298,11 +2739,11 @@ package body Exp_Ch4 is
                end if;
 
             --  If we have anything other than Standard_Character or
-            --  Standard_String, then we must have had an error earlier.
-            --  So we just abandon the attempt at expansion.
+            --  Standard_String, then we must have had a serious error
+            --  earlier, so we just abandon the attempt at expansion.
 
             else
-               pragma Assert (Errors_Detected > 0);
+               pragma Assert (Serious_Errors_Detected > 0);
                return;
             end if;
 
@@ -1328,6 +2769,10 @@ package body Exp_Ch4 is
           Parameter_Associations => Opnds));
 
       Analyze_And_Resolve (Cnode, Standard_String);
+
+   exception
+      when RE_Not_Available =>
+         return;
    end Expand_Concatenate_String;
 
    ------------------------
@@ -1336,649 +2781,690 @@ package body Exp_Ch4 is
 
    procedure Expand_N_Allocator (N : Node_Id) is
       PtrT  : constant Entity_Id  := Etype (N);
-      Desig : Entity_Id;
+      Dtyp  : constant Entity_Id  := Designated_Type (PtrT);
+      Etyp  : constant Entity_Id  := Etype (Expression (N));
       Loc   : constant Source_Ptr := Sloc (N);
+      Desig : Entity_Id;
       Temp  : Entity_Id;
-      Node  : Node_Id;
+      Nod   : Node_Id;
 
-   begin
-      --  RM E.2.3(22). We enforce that the expected type of an allocator
-      --  shall not be a remote access-to-class-wide-limited-private type
+      procedure Complete_Coextension_Finalization;
+      --  Generate finalization calls for all nested coextensions of N. This
+      --  routine may allocate list controllers if necessary.
 
-      --  Why is this being done at expansion time, seems clearly wrong ???
+      procedure Rewrite_Coextension (N : Node_Id);
+      --  Static coextensions have the same lifetime as the entity they
+      --  constrain. Such occurences can be rewritten as aliased objects
+      --  and their unrestricted access used instead of the coextension.
 
-      Validate_Remote_Access_To_Class_Wide_Type (N);
+      ---------------------------------------
+      -- Complete_Coextension_Finalization --
+      ---------------------------------------
 
-      --  Set the Storage Pool
+      procedure Complete_Coextension_Finalization is
+         Coext      : Node_Id;
+         Coext_Elmt : Elmt_Id;
+         Flist      : Node_Id;
+         Ref        : Node_Id;
 
-      Set_Storage_Pool (N, Associated_Storage_Pool (Root_Type (PtrT)));
+         function Inside_A_Return_Statement (N : Node_Id) return Boolean;
+         --  Determine whether node N is part of a return statement
 
-      if Present (Storage_Pool (N)) then
-         if Is_RTE (Storage_Pool (N), RE_SS_Pool) then
-            if not Java_VM then
-               Set_Procedure_To_Call (N, RTE (RE_SS_Allocate));
+         function Needs_Initialization_Call (N : Node_Id) return Boolean;
+         --  Determine whether node N is a subtype indicator allocator which
+         --  asts a coextension. Such coextensions need initialization.
+
+         -------------------------------
+         -- Inside_A_Return_Statement --
+         -------------------------------
+
+         function Inside_A_Return_Statement (N : Node_Id) return Boolean is
+            P : Node_Id;
+
+         begin
+            P := Parent (N);
+            while Present (P) loop
+               if Nkind_In
+                   (P, N_Extended_Return_Statement, N_Simple_Return_Statement)
+               then
+                  return True;
+
+               --  Stop the traversal when we reach a subprogram body
+
+               elsif Nkind (P) = N_Subprogram_Body then
+                  return False;
+               end if;
+
+               P := Parent (P);
+            end loop;
+
+            return False;
+         end Inside_A_Return_Statement;
+
+         -------------------------------
+         -- Needs_Initialization_Call --
+         -------------------------------
+
+         function Needs_Initialization_Call (N : Node_Id) return Boolean is
+            Obj_Decl : Node_Id;
+
+         begin
+            if Nkind (N) = N_Explicit_Dereference
+              and then Nkind (Prefix (N)) = N_Identifier
+              and then Nkind (Parent (Entity (Prefix (N)))) =
+                         N_Object_Declaration
+            then
+               Obj_Decl := Parent (Entity (Prefix (N)));
+
+               return
+                 Present (Expression (Obj_Decl))
+                   and then Nkind (Expression (Obj_Decl)) = N_Allocator
+                   and then Nkind (Expression (Expression (Obj_Decl))) /=
+                              N_Qualified_Expression;
             end if;
-         else
-            Set_Procedure_To_Call (N,
-              Find_Prim_Op (Etype (Storage_Pool (N)), Name_Allocate));
+
+            return False;
+         end Needs_Initialization_Call;
+
+      --  Start of processing for Complete_Coextension_Finalization
+
+      begin
+         --  When a coextension root is inside a return statement, we need to
+         --  use the finalization chain of the function's scope. This does not
+         --  apply for controlled named access types because in those cases we
+         --  can use the finalization chain of the type itself.
+
+         if Inside_A_Return_Statement (N)
+           and then
+             (Ekind (PtrT) = E_Anonymous_Access_Type
+                or else
+                  (Ekind (PtrT) = E_Access_Type
+                     and then No (Associated_Final_Chain (PtrT))))
+         then
+            declare
+               Decl    : Node_Id;
+               Outer_S : Entity_Id;
+               S       : Entity_Id := Current_Scope;
+
+            begin
+               while Present (S) and then S /= Standard_Standard loop
+                  if Ekind (S) = E_Function then
+                     Outer_S := Scope (S);
+
+                     --  Retrieve the declaration of the body
+
+                     Decl := Parent (Parent (
+                               Corresponding_Body (Parent (Parent (S)))));
+                     exit;
+                  end if;
+
+                  S := Scope (S);
+               end loop;
+
+               --  Push the scope of the function body since we are inserting
+               --  the list before the body, but we are currently in the body
+               --  itself. Override the finalization list of PtrT since the
+               --  finalization context is now different.
+
+               Push_Scope (Outer_S);
+               Build_Final_List (Decl, PtrT);
+               Pop_Scope;
+            end;
+
+         --  The root allocator may not be controlled, but it still needs a
+         --  finalization list for all nested coextensions.
+
+         elsif No (Associated_Final_Chain (PtrT)) then
+            Build_Final_List (N, PtrT);
          end if;
-      end if;
 
-      --  Under certain circumstances we can replace an allocator by an
-      --  access to statically allocated storage. The conditions, as noted
-      --  in AARM 3.10 (10c) are as follows:
+         Flist :=
+           Make_Selected_Component (Loc,
+             Prefix =>
+               New_Reference_To (Associated_Final_Chain (PtrT), Loc),
+             Selector_Name =>
+               Make_Identifier (Loc, Name_F));
 
-      --    Size and initial value is known at compile time
-      --    Access type is access-to-constant
+         Coext_Elmt := First_Elmt (Coextensions (N));
+         while Present (Coext_Elmt) loop
+            Coext := Node (Coext_Elmt);
 
-      if Is_Access_Constant (PtrT)
-        and then Nkind (Expression (N)) = N_Qualified_Expression
-        and then Compile_Time_Known_Value (Expression (Expression (N)))
-        and then Size_Known_At_Compile_Time (Etype (Expression
-                                                    (Expression (N))))
-      then
-         --  Here we can do the optimization. For the allocator
+            --  Generate:
+            --    typ! (coext.all)
 
-         --    new x'(y)
+            if Nkind (Coext) = N_Identifier then
+               Ref := Make_Unchecked_Type_Conversion (Loc,
+                        Subtype_Mark =>
+                          New_Reference_To (Etype (Coext), Loc),
+                        Expression =>
+                          Make_Explicit_Dereference (Loc,
+                            New_Copy_Tree (Coext)));
+            else
+               Ref := New_Copy_Tree (Coext);
+            end if;
 
-         --  We insert an object declaration
+            --  Generate:
+            --    initialize (Ref)
+            --    attach_to_final_list (Ref, Flist, 2)
 
-         --    Tnn : aliased x := y;
+            if Needs_Initialization_Call (Coext) then
+               Insert_Actions (N,
+                 Make_Init_Call (
+                   Ref         => Ref,
+                   Typ         => Etype (Coext),
+                   Flist_Ref   => Flist,
+                   With_Attach => Make_Integer_Literal (Loc, Uint_2)));
 
-         --  and replace the allocator by Tnn'Unrestricted_Access.
-         --  Tnn is marked as requiring static allocation.
+            --  Generate:
+            --    attach_to_final_list (Ref, Flist, 2)
 
-         Temp :=
-           Make_Defining_Identifier (Loc, New_Internal_Name ('T'));
+            else
+               Insert_Action (N,
+                 Make_Attach_Call (
+                   Obj_Ref     => Ref,
+                   Flist_Ref   => New_Copy_Tree (Flist),
+                   With_Attach => Make_Integer_Literal (Loc, Uint_2)));
+            end if;
 
-         Desig := Subtype_Mark (Expression (N));
+            Next_Elmt (Coext_Elmt);
+         end loop;
+      end Complete_Coextension_Finalization;
 
-         --  If context is constrained, use constrained subtype directly,
-         --  so that the constant is not labelled as having a nomimally
-         --  unconstrained subtype.
+      -------------------------
+      -- Rewrite_Coextension --
+      -------------------------
 
-         if Entity (Desig) = Base_Type (Designated_Type (PtrT)) then
-            Desig := New_Occurrence_Of (Designated_Type (PtrT), Loc);
+      procedure Rewrite_Coextension (N : Node_Id) is
+         Temp : constant Node_Id :=
+                  Make_Defining_Identifier (Loc,
+                    New_Internal_Name ('C'));
+
+         --  Generate:
+         --    Cnn : aliased Etyp;
+
+         Decl : constant Node_Id :=
+                  Make_Object_Declaration (Loc,
+                    Defining_Identifier => Temp,
+                    Aliased_Present     => True,
+                    Object_Definition   =>
+                      New_Occurrence_Of (Etyp, Loc));
+         Nod  : Node_Id;
+
+      begin
+         if Nkind (Expression (N)) = N_Qualified_Expression then
+            Set_Expression (Decl, Expression (Expression (N)));
          end if;
 
-         Insert_Action (N,
-           Make_Object_Declaration (Loc,
-             Defining_Identifier => Temp,
-             Aliased_Present     => True,
-             Constant_Present    => Is_Access_Constant (PtrT),
-             Object_Definition   => Desig,
-             Expression          => Expression (Expression (N))));
+         --  Find the proper insertion node for the declaration
+
+         Nod := Parent (N);
+         while Present (Nod) loop
+            exit when Nkind (Nod) in N_Statement_Other_Than_Procedure_Call
+              or else Nkind (Nod) = N_Procedure_Call_Statement
+              or else Nkind (Nod) in N_Declaration;
+            Nod := Parent (Nod);
+         end loop;
+
+         Insert_Before (Nod, Decl);
+         Analyze (Decl);
 
          Rewrite (N,
            Make_Attribute_Reference (Loc,
-             Prefix => New_Occurrence_Of (Temp, Loc),
+             Prefix         => New_Occurrence_Of (Temp, Loc),
              Attribute_Name => Name_Unrestricted_Access));
 
          Analyze_And_Resolve (N, PtrT);
+      end Rewrite_Coextension;
 
-         --  We set the variable as statically allocated, since we don't
-         --  want it going on the stack of the current procedure!
-
-         Set_Is_Statically_Allocated (Temp);
-         return;
-      end if;
-
-      --  If the allocator is for a type which requires initialization, and
-      --  there is no initial value (i.e. the operand is a subtype indication
-      --  rather than a qualifed expression), then we must generate a call to
-      --  the initialization routine. This is done using an expression actions
-      --  node:
-      --
-      --     [Pnnn : constant ptr_T := new (T); Init (Pnnn.all,...); Pnnn]
-      --
-      --  Here ptr_T is the pointer type for the allocator, and T is the
-      --  subtype of the allocator. A special case arises if the designated
-      --  type of the access type is a task or contains tasks. In this case
-      --  the call to Init (Temp.all ...) is replaced by code that ensures
-      --  that the tasks get activated (see Exp_Ch9.Build_Task_Allocate_Block
-      --  for details). In addition, if the type T is a task T, then the first
-      --  argument to Init must be converted to the task record type.
-
-      if Nkind (Expression (N)) = N_Qualified_Expression then
-         declare
-            Indic : constant Node_Id   := Subtype_Mark (Expression (N));
-            T     : constant Entity_Id := Entity (Indic);
-            Exp   : constant Node_Id   := Expression (Expression (N));
+   --  Start of processing for Expand_N_Allocator
 
-            Aggr_In_Place : constant Boolean := Is_Delayed_Aggregate (Exp);
+   begin
+      --  RM E.2.3(22). We enforce that the expected type of an allocator
+      --  shall not be a remote access-to-class-wide-limited-private type
 
-            Tag_Assign : Node_Id;
-            Tmp_Node   : Node_Id;
+      --  Why is this being done at expansion time, seems clearly wrong ???
 
-         begin
-            if Is_Tagged_Type (T) or else Controlled_Type (T) then
-
-               --    Actions inserted before:
-               --              Temp : constant ptr_T := new T'(Expression);
-               --   <no CW>    Temp._tag := T'tag;
-               --   <CTRL>     Adjust (Finalizable (Temp.all));
-               --   <CTRL>     Attach_To_Final_List (Finalizable (Temp.all));
-
-               --  We analyze by hand the new internal allocator to avoid
-               --  any recursion and inappropriate call to Initialize
-               if not Aggr_In_Place then
-                  Remove_Side_Effects (Exp);
-               end if;
+      Validate_Remote_Access_To_Class_Wide_Type (N);
 
-               Temp :=
-                 Make_Defining_Identifier (Loc, New_Internal_Name ('P'));
+      --  Set the Storage Pool
 
-               --  For a class wide allocation generate the following code:
+      Set_Storage_Pool (N, Associated_Storage_Pool (Root_Type (PtrT)));
 
-               --    type Equiv_Record is record ... end record;
-               --    implicit subtype CW is <Class_Wide_Subytpe>;
-               --    temp : PtrT := new CW'(CW!(expr));
+      if Present (Storage_Pool (N)) then
+         if Is_RTE (Storage_Pool (N), RE_SS_Pool) then
+            if VM_Target = No_VM then
+               Set_Procedure_To_Call (N, RTE (RE_SS_Allocate));
+            end if;
 
-               if Is_Class_Wide_Type (T) then
-                  Expand_Subtype_From_Expr (Empty, T, Indic, Exp);
+         elsif Is_Class_Wide_Type (Etype (Storage_Pool (N))) then
+            Set_Procedure_To_Call (N, RTE (RE_Allocate_Any));
 
-                  Set_Expression (Expression (N),
-                    Unchecked_Convert_To (Entity (Indic), Exp));
+         else
+            Set_Procedure_To_Call (N,
+              Find_Prim_Op (Etype (Storage_Pool (N)), Name_Allocate));
+         end if;
+      end if;
 
-                  Analyze_And_Resolve (Expression (N), Entity (Indic));
-               end if;
+      --  Under certain circumstances we can replace an allocator by an
+      --  access to statically allocated storage. The conditions, as noted
+      --  in AARM 3.10 (10c) are as follows:
 
-               if Aggr_In_Place then
-                  Tmp_Node :=
-                    Make_Object_Declaration (Loc,
-                      Defining_Identifier => Temp,
-                      Object_Definition   => New_Reference_To (PtrT, Loc),
-                      Expression          => Make_Allocator (Loc,
-                          New_Reference_To (Etype (Exp), Loc)));
+      --    Size and initial value is known at compile time
+      --    Access type is access-to-constant
 
-                  Set_No_Initialization (Expression (Tmp_Node));
-                  Insert_Action (N, Tmp_Node);
-                  Convert_Aggr_In_Allocator (Tmp_Node, Exp);
-               else
-                  Node := Relocate_Node (N);
-                  Set_Analyzed (Node);
-                  Insert_Action (N,
-                    Make_Object_Declaration (Loc,
-                      Defining_Identifier => Temp,
-                      Constant_Present    => True,
-                      Object_Definition   => New_Reference_To (PtrT, Loc),
-                      Expression          => Node));
-               end if;
+      --  The allocator is not part of a constraint on a record component,
+      --  because in that case the inserted actions are delayed until the
+      --  record declaration is fully analyzed, which is too late for the
+      --  analysis of the rewritten allocator.
 
-               --  Suppress the tag assignment when Java_VM because JVM tags
-               --  are represented implicitly in objects.
+      if Is_Access_Constant (PtrT)
+        and then Nkind (Expression (N)) = N_Qualified_Expression
+        and then Compile_Time_Known_Value (Expression (Expression (N)))
+        and then Size_Known_At_Compile_Time (Etype (Expression
+                                                    (Expression (N))))
+        and then not Is_Record_Type (Current_Scope)
+      then
+         --  Here we can do the optimization. For the allocator
 
-               if Is_Tagged_Type (T)
-                 and then not Is_Class_Wide_Type (T)
-                 and then not Java_VM
-               then
-                  Tag_Assign :=
-                    Make_Assignment_Statement (Loc,
-                      Name =>
-                        Make_Selected_Component (Loc,
-                          Prefix => New_Reference_To (Temp, Loc),
-                          Selector_Name =>
-                            New_Reference_To (Tag_Component (T), Loc)),
+         --    new x'(y)
 
-                      Expression =>
-                        Unchecked_Convert_To (RTE (RE_Tag),
-                          New_Reference_To (Access_Disp_Table (T), Loc)));
+         --  We insert an object declaration
 
-                  --  The previous assignment has to be done in any case
+         --    Tnn : aliased x := y;
 
-                  Set_Assignment_OK (Name (Tag_Assign));
-                  Insert_Action (N, Tag_Assign);
+         --  and replace the allocator by Tnn'Unrestricted_Access.
+         --  Tnn is marked as requiring static allocation.
 
-               elsif Is_Private_Type (T)
-                 and then Is_Tagged_Type (Underlying_Type (T))
-                 and then not Java_VM
-               then
-                  declare
-                     Utyp : constant Entity_Id := Underlying_Type (T);
-                     Ref  : constant Node_Id :=
-                              Unchecked_Convert_To (Utyp,
-                                Make_Explicit_Dereference (Loc,
-                                  New_Reference_To (Temp, Loc)));
+         Temp :=
+           Make_Defining_Identifier (Loc, New_Internal_Name ('T'));
 
-                  begin
-                     Tag_Assign :=
-                       Make_Assignment_Statement (Loc,
-                         Name =>
-                           Make_Selected_Component (Loc,
-                             Prefix => Ref,
-                             Selector_Name =>
-                               New_Reference_To (Tag_Component (Utyp), Loc)),
-
-                         Expression =>
-                           Unchecked_Convert_To (RTE (RE_Tag),
-                             New_Reference_To (
-                               Access_Disp_Table (Utyp), Loc)));
-
-                     Set_Assignment_OK (Name (Tag_Assign));
-                     Insert_Action (N, Tag_Assign);
-                  end;
-               end if;
+         Desig := Subtype_Mark (Expression (N));
 
-               if Controlled_Type (Designated_Type (PtrT))
-                  and then Controlled_Type (T)
-               then
-                  declare
-                     Flist  : Node_Id;
-                     Attach : Node_Id;
-                     Apool  : constant Entity_Id :=
-                                Associated_Storage_Pool (PtrT);
+         --  If context is constrained, use constrained subtype directly,
+         --  so that the constant is not labelled as having a nomimally
+         --  unconstrained subtype.
 
-                  begin
-                     --  If it is an allocation on the secondary stack
-                     --  (i.e. a value returned from a function), the object
-                     --  is attached on the caller side as soon as the call
-                     --  is completed (see Expand_Ctrl_Function_Call)
-
-                     if Is_RTE (Apool, RE_SS_Pool) then
-                        declare
-                           F : constant Entity_Id :=
-                                 Make_Defining_Identifier (Loc,
-                                   New_Internal_Name ('F'));
-                        begin
-                           Insert_Action (N,
-                             Make_Object_Declaration (Loc,
-                               Defining_Identifier => F,
-                               Object_Definition   => New_Reference_To (RTE
-                                (RE_Finalizable_Ptr), Loc)));
-
-                           Flist := New_Reference_To (F, Loc);
-                           Attach :=  Make_Integer_Literal (Loc, 1);
-                        end;
-
-                     --  Normal case, not a secondary stack allocation
+         if Entity (Desig) = Base_Type (Dtyp) then
+            Desig := New_Occurrence_Of (Dtyp, Loc);
+         end if;
 
-                     else
-                        Flist := Find_Final_List (PtrT);
-                        Attach :=  Make_Integer_Literal (Loc, 2);
-                     end if;
+         Insert_Action (N,
+           Make_Object_Declaration (Loc,
+             Defining_Identifier => Temp,
+             Aliased_Present     => True,
+             Constant_Present    => Is_Access_Constant (PtrT),
+             Object_Definition   => Desig,
+             Expression          => Expression (Expression (N))));
 
-                     if not Aggr_In_Place then
-                        Insert_Actions (N,
-                          Make_Adjust_Call (
-                            Ref          =>
+         Rewrite (N,
+           Make_Attribute_Reference (Loc,
+             Prefix => New_Occurrence_Of (Temp, Loc),
+             Attribute_Name => Name_Unrestricted_Access));
 
-                           --  An unchecked conversion is needed in the
-                           --  classwide case because the designated type
-                           --  can be an ancestor of the subtype mark of
-                           --  the allocator.
+         Analyze_And_Resolve (N, PtrT);
 
-                            Unchecked_Convert_To (T,
-                              Make_Explicit_Dereference (Loc,
-                                New_Reference_To (Temp, Loc))),
+         --  We set the variable as statically allocated, since we don't
+         --  want it going on the stack of the current procedure!
 
-                            Typ          => T,
-                            Flist_Ref    => Flist,
-                            With_Attach  => Attach));
-                     end if;
-                  end;
-               end if;
+         Set_Is_Statically_Allocated (Temp);
+         return;
+      end if;
 
-               Rewrite (N, New_Reference_To (Temp, Loc));
-               Analyze_And_Resolve (N, PtrT);
+      --  Same if the allocator is an access discriminant for a local object:
+      --  instead of an allocator we create a local value and constrain the
+      --  the enclosing object with the corresponding access attribute.
 
-            elsif Aggr_In_Place then
-               Temp :=
-                 Make_Defining_Identifier (Loc, New_Internal_Name ('P'));
-               Tmp_Node :=
-                 Make_Object_Declaration (Loc,
-                   Defining_Identifier => Temp,
-                   Object_Definition   => New_Reference_To (PtrT, Loc),
-                   Expression          => Make_Allocator (Loc,
-                       New_Reference_To (Etype (Exp), Loc)));
+      if Is_Static_Coextension (N) then
+         Rewrite_Coextension (N);
+         return;
+      end if;
 
-               Set_No_Initialization (Expression (Tmp_Node));
-               Insert_Action (N, Tmp_Node);
-               Convert_Aggr_In_Allocator (Tmp_Node, Exp);
-               Rewrite (N, New_Reference_To (Temp, Loc));
-               Analyze_And_Resolve (N, PtrT);
+      --  The current allocator creates an object which may contain nested
+      --  coextensions. Use the current allocator's finalization list to
+      --  generate finalization call for all nested coextensions.
 
-            elsif Is_Access_Type (Designated_Type (PtrT))
-              and then Nkind (Exp) = N_Allocator
-              and then Nkind (Expression (Exp)) /= N_Qualified_Expression
-            then
-               --  Apply constraint to designated subtype indication.
+      if Is_Coextension_Root (N) then
+         Complete_Coextension_Finalization;
+      end if;
 
-               Apply_Constraint_Check (Expression (Exp),
-                 Designated_Type (Designated_Type (PtrT)),
-                 No_Sliding => True);
+      --  Handle case of qualified expression (other than optimization above)
 
-               if Nkind (Expression (Exp)) = N_Raise_Constraint_Error then
+      if Nkind (Expression (N)) = N_Qualified_Expression then
+         Expand_Allocator_Expression (N);
+         return;
+      end if;
 
-                  --  Propagate constraint_error to enclosing allocator.
+      --  If the allocator is for a type which requires initialization, and
+      --  there is no initial value (i.e. operand is a subtype indication
+      --  rather than a qualifed expression), then we must generate a call
+      --  to the initialization routine. This is done using an expression
+      --  actions node:
 
-                  Rewrite
-                    (Exp, New_Copy (Expression (Exp)));
-               end if;
-            else
-               --  First check against the type of the qualified expression
-               --
-               --  NOTE: The commented call should be correct, but for
-               --  some reason causes the compiler to bomb (sigsegv) on
-               --  ACVC test c34007g, so for now we just perform the old
-               --  (incorrect) test against the designated subtype with
-               --  no sliding in the else part of the if statement below.
-               --  ???
-               --
-               --  Apply_Constraint_Check (Exp, T, No_Sliding => True);
-
-               --  A check is also needed in cases where the designated
-               --  subtype is constrained and differs from the subtype
-               --  given in the qualified expression. Note that the check
-               --  on the qualified expression does not allow sliding,
-               --  but this check does (a relaxation from Ada 83).
-
-               if Is_Constrained (Designated_Type (PtrT))
-                 and then not Subtypes_Statically_Match
-                                (T, Designated_Type (PtrT))
-               then
-                  Apply_Constraint_Check
-                    (Exp, Designated_Type (PtrT), No_Sliding => False);
+      --     [Pnnn : constant ptr_T := new (T); Init (Pnnn.all,...); Pnnn]
 
-               --  The nonsliding check should really be performed
-               --  (unconditionally) against the subtype of the
-               --  qualified expression, but that causes a problem
-               --  with c34007g (see above), so for now we retain this.
+      --  Here ptr_T is the pointer type for the allocator, and T is the
+      --  subtype of the allocator. A special case arises if the designated
+      --  type of the access type is a task or contains tasks. In this case
+      --  the call to Init (Temp.all ...) is replaced by code that ensures
+      --  that tasks get activated (see Exp_Ch9.Build_Task_Allocate_Block
+      --  for details). In addition, if the type T is a task T, then the
+      --  first argument to Init must be converted to the task record type.
 
-               else
-                  Apply_Constraint_Check
-                    (Exp, Designated_Type (PtrT), No_Sliding => True);
-               end if;
-            end if;
-         end;
+      declare
+         T            : constant Entity_Id := Entity (Expression (N));
+         Init         : Entity_Id;
+         Arg1         : Node_Id;
+         Args         : List_Id;
+         Decls        : List_Id;
+         Decl         : Node_Id;
+         Discr        : Elmt_Id;
+         Flist        : Node_Id;
+         Temp_Decl    : Node_Id;
+         Temp_Type    : Entity_Id;
+         Attach_Level : Uint;
 
-      --  Here if not qualified expression case.
-      --  In this case, an initialization routine may be required
+      begin
+         if No_Initialization (N) then
+            null;
 
-      else
-         declare
-            T         : constant Entity_Id  := Entity (Expression (N));
-            Init      : Entity_Id;
-            Arg1      : Node_Id;
-            Args      : List_Id;
-            Decls     : List_Id;
-            Decl      : Node_Id;
-            Discr     : Elmt_Id;
-            Flist     : Node_Id;
-            Temp_Decl : Node_Id;
-            Temp_Type : Entity_Id;
+         --  Case of no initialization procedure present
 
-         begin
+         elsif not Has_Non_Null_Base_Init_Proc (T) then
 
-            if No_Initialization (N) then
-               null;
+            --  Case of simple initialization required
 
-            --  Case of no initialization procedure present
+            if Needs_Simple_Initialization (T) then
+               Rewrite (Expression (N),
+                 Make_Qualified_Expression (Loc,
+                   Subtype_Mark => New_Occurrence_Of (T, Loc),
+                   Expression   => Get_Simple_Init_Val (T, Loc)));
 
-            elsif not Has_Non_Null_Base_Init_Proc (T) then
+               Analyze_And_Resolve (Expression (Expression (N)), T);
+               Analyze_And_Resolve (Expression (N), T);
+               Set_Paren_Count     (Expression (Expression (N)), 1);
+               Expand_N_Allocator  (N);
 
-               --  Case of simple initialization required
+            --  No initialization required
 
-               if Needs_Simple_Initialization (T) then
-                  Rewrite (Expression (N),
-                    Make_Qualified_Expression (Loc,
-                      Subtype_Mark => New_Occurrence_Of (T, Loc),
-                      Expression   => Get_Simple_Init_Val (T, Loc)));
+            else
+               null;
+            end if;
 
-                  Analyze_And_Resolve (Expression (Expression (N)), T);
-                  Analyze_And_Resolve (Expression (N), T);
-                  Set_Paren_Count (Expression (Expression (N)), 1);
-                  Expand_N_Allocator (N);
+         --  Case of initialization procedure present, must be called
 
-               --  No initialization required
+         else
+            Init := Base_Init_Proc (T);
+            Nod  := N;
+            Temp := Make_Defining_Identifier (Loc, New_Internal_Name ('P'));
 
-               else
-                  null;
-               end if;
+            --  Construct argument list for the initialization routine call
 
-            --  Case of initialization procedure present, must be called
+            Arg1 :=
+              Make_Explicit_Dereference (Loc,
+                Prefix => New_Reference_To (Temp, Loc));
+            Set_Assignment_OK (Arg1);
+            Temp_Type := PtrT;
 
-            else
-               Init := Base_Init_Proc (T);
-               Node := N;
-               Temp :=
-                 Make_Defining_Identifier (Loc, New_Internal_Name ('P'));
+            --  The initialization procedure expects a specific type. if the
+            --  context is access to class wide, indicate that the object being
+            --  allocated has the right specific type.
 
-               --  Construct argument list for the initialization routine call
-               --  The CPP constructor needs the address directly
+            if Is_Class_Wide_Type (Dtyp) then
+               Arg1 := Unchecked_Convert_To (T, Arg1);
+            end if;
 
-               if Is_CPP_Class (T) then
-                  Arg1 := New_Reference_To (Temp, Loc);
-                  Temp_Type := T;
+            --  If designated type is a concurrent type or if it is private
+            --  type whose definition is a concurrent type, the first argument
+            --  in the Init routine has to be unchecked conversion to the
+            --  corresponding record type. If the designated type is a derived
+            --  type, we also convert the argument to its root type.
 
-               else
-                  Arg1 :=
-                    Make_Explicit_Dereference (Loc,
-                      Prefix => New_Reference_To (Temp, Loc));
-                  Set_Assignment_OK (Arg1);
-                  Temp_Type := PtrT;
+            if Is_Concurrent_Type (T) then
+               Arg1 :=
+                 Unchecked_Convert_To (Corresponding_Record_Type (T), Arg1);
 
-                  --  The initialization procedure expects a specific type.
-                  --  if the context is access to class wide, indicate that
-                  --  the object being allocated has the right specific type.
+            elsif Is_Private_Type (T)
+              and then Present (Full_View (T))
+              and then Is_Concurrent_Type (Full_View (T))
+            then
+               Arg1 :=
+                 Unchecked_Convert_To
+                   (Corresponding_Record_Type (Full_View (T)), Arg1);
 
-                  if Is_Class_Wide_Type (Designated_Type (PtrT)) then
-                     Arg1 := Unchecked_Convert_To (T, Arg1);
-                  end if;
-               end if;
+            elsif Etype (First_Formal (Init)) /= Base_Type (T) then
+               declare
+                  Ftyp : constant Entity_Id := Etype (First_Formal (Init));
 
-               --  If designated type is a concurrent type or if it is a
-               --  private type whose definition is a concurrent type,
-               --  the first argument in the Init routine has to be
-               --  unchecked conversion to the corresponding record type.
-               --  If the designated type is a derived type, we also
-               --  convert the argument to its root type.
+               begin
+                  Arg1 := OK_Convert_To (Etype (Ftyp), Arg1);
+                  Set_Etype (Arg1, Ftyp);
+               end;
+            end if;
 
-               if Is_Concurrent_Type (T) then
-                  Arg1 :=
-                    Unchecked_Convert_To (Corresponding_Record_Type (T), Arg1);
+            Args := New_List (Arg1);
 
-               elsif Is_Private_Type (T)
-                 and then Present (Full_View (T))
-                 and then Is_Concurrent_Type (Full_View (T))
-               then
-                  Arg1 :=
-                    Unchecked_Convert_To
-                      (Corresponding_Record_Type (Full_View (T)), Arg1);
+            --  For the task case, pass the Master_Id of the access type as
+            --  the value of the _Master parameter, and _Chain as the value
+            --  of the _Chain parameter (_Chain will be defined as part of
+            --  the generated code for the allocator).
 
-               elsif Etype (First_Formal (Init)) /= Base_Type (T) then
+            --  In Ada 2005, the context may be a function that returns an
+            --  anonymous access type. In that case the Master_Id has been
+            --  created when expanding the function declaration.
 
-                  declare
-                     Ftyp : constant Entity_Id := Etype (First_Formal (Init));
+            if Has_Task (T) then
+               if No (Master_Id (Base_Type (PtrT))) then
 
-                  begin
-                     Arg1 := OK_Convert_To (Etype (Ftyp), Arg1);
-                     Set_Etype (Arg1, Ftyp);
-                  end;
-               end if;
+                  --  If we have a non-library level task with the restriction
+                  --  No_Task_Hierarchy set, then no point in expanding.
 
-               Args := New_List (Arg1);
+                  if not Is_Library_Level_Entity (T)
+                    and then Restriction_Active (No_Task_Hierarchy)
+                  then
+                     return;
+                  end if;
 
-               --  For the task case, pass the Master_Id of the access type
-               --  as the value of the _Master parameter, and _Chain as the
-               --  value of the _Chain parameter (_Chain will be defined as
-               --  part of the generated code for the allocator).
+                  --  The designated type was an incomplete type, and the
+                  --  access type did not get expanded. Salvage it now.
 
-               if Has_Task (T) then
+                  pragma Assert (Present (Parent (Base_Type (PtrT))));
+                  Expand_N_Full_Type_Declaration (Parent (Base_Type (PtrT)));
+               end if;
 
-                  if No (Master_Id (Base_Type (PtrT))) then
+               --  If the context of the allocator is a declaration or an
+               --  assignment, we can generate a meaningful image for it,
+               --  even though subsequent assignments might remove the
+               --  connection between task and entity. We build this image
+               --  when the left-hand side is a simple variable, a simple
+               --  indexed assignment or a simple selected component.
 
-                     --  The designated type was an incomplete type, and
-                     --  the access type did not get expanded. Salvage
-                     --  it now.
+               if Nkind (Parent (N)) = N_Assignment_Statement then
+                  declare
+                     Nam : constant Node_Id := Name (Parent (N));
 
-                     Expand_N_Full_Type_Declaration
-                       (Parent (Base_Type (PtrT)));
-                  end if;
+                  begin
+                     if Is_Entity_Name (Nam) then
+                        Decls :=
+                          Build_Task_Image_Decls (
+                            Loc,
+                              New_Occurrence_Of
+                                (Entity (Nam), Sloc (Nam)), T);
+
+                     elsif Nkind_In
+                             (Nam, N_Indexed_Component, N_Selected_Component)
+                       and then Is_Entity_Name (Prefix (Nam))
+                     then
+                        Decls :=
+                          Build_Task_Image_Decls
+                            (Loc, Nam, Etype (Prefix (Nam)));
+                     else
+                        Decls := Build_Task_Image_Decls (Loc, T, T);
+                     end if;
+                  end;
 
-                  --  If the context of the allocator is a declaration or
-                  --  an assignment, we can generate a meaningful image for
-                  --  it, even though subsequent assignments might remove
-                  --  the connection between task and entity. We build this
-                  --  image when the left-hand side is a simple variable,
-                  --  a simple indexed assignment or a simple selected
-                  --  component.
-
-                  if Nkind (Parent (N)) = N_Assignment_Statement then
-                     declare
-                        Nam : constant Node_Id := Name (Parent (N));
-
-                     begin
-                        if Is_Entity_Name (Nam) then
-                           Decls :=
-                             Build_Task_Image_Decls (
-                               Loc,
-                                 New_Occurrence_Of
-                                   (Entity (Nam), Sloc (Nam)), T);
-
-                        elsif (Nkind (Nam) = N_Indexed_Component
-                                or else Nkind (Nam) = N_Selected_Component)
-                          and then Is_Entity_Name (Prefix (Nam))
-                        then
-                           Decls :=
-                             Build_Task_Image_Decls (
-                             Loc, Nam, Etype (Prefix (Nam)));
-                        else
-                           Decls := Build_Task_Image_Decls (Loc, T, T);
-                        end if;
-                     end;
+               elsif Nkind (Parent (N)) = N_Object_Declaration then
+                  Decls :=
+                    Build_Task_Image_Decls (
+                       Loc, Defining_Identifier (Parent (N)), T);
 
-                  elsif Nkind (Parent (N)) = N_Object_Declaration then
-                     Decls :=
-                       Build_Task_Image_Decls (
-                          Loc, Defining_Identifier (Parent (N)), T);
+               else
+                  Decls := Build_Task_Image_Decls (Loc, T, T);
+               end if;
 
-                  else
-                     Decls := Build_Task_Image_Decls (Loc, T, T);
-                  end if;
+               Append_To (Args,
+                 New_Reference_To
+                   (Master_Id (Base_Type (Root_Type (PtrT))), Loc));
+               Append_To (Args, Make_Identifier (Loc, Name_uChain));
 
-                  Append_To (Args,
-                    New_Reference_To
-                      (Master_Id (Base_Type (Root_Type (PtrT))), Loc));
-                  Append_To (Args, Make_Identifier (Loc, Name_uChain));
+               Decl := Last (Decls);
+               Append_To (Args,
+                 New_Occurrence_Of (Defining_Identifier (Decl), Loc));
 
-                  Decl := Last (Decls);
-                  Append_To (Args,
-                    New_Occurrence_Of (Defining_Identifier (Decl), Loc));
+            --  Has_Task is false, Decls not used
 
-               --  Has_Task is false, Decls not used
+            else
+               Decls := No_List;
+            end if;
 
-               else
-                  Decls := No_List;
-               end if;
+            --  Add discriminants if discriminated type
 
-               --  Add discriminants if discriminated type
+            declare
+               Dis : Boolean := False;
+               Typ : Entity_Id;
 
+            begin
                if Has_Discriminants (T) then
-                  Discr := First_Elmt (Discriminant_Constraint (T));
-
-                  while Present (Discr) loop
-                     Append (New_Copy (Elists.Node (Discr)), Args);
-                     Next_Elmt (Discr);
-                  end loop;
+                  Dis := True;
+                  Typ := T;
 
                elsif Is_Private_Type (T)
                  and then Present (Full_View (T))
                  and then Has_Discriminants (Full_View (T))
                then
-                  Discr :=
-                    First_Elmt (Discriminant_Constraint (Full_View (T)));
+                  Dis := True;
+                  Typ := Full_View (T);
+               end if;
+
+               if Dis then
+                  --  If the allocated object will be constrained by the
+                  --  default values for discriminants, then build a
+                  --  subtype with those defaults, and change the allocated
+                  --  subtype to that. Note that this happens in fewer
+                  --  cases in Ada 2005 (AI-363).
+
+                  if not Is_Constrained (Typ)
+                    and then Present (Discriminant_Default_Value
+                                       (First_Discriminant (Typ)))
+                    and then (Ada_Version < Ada_05
+                               or else not Has_Constrained_Partial_View (Typ))
+                  then
+                     Typ := Build_Default_Subtype (Typ, N);
+                     Set_Expression (N, New_Reference_To (Typ, Loc));
+                  end if;
 
+                  Discr := First_Elmt (Discriminant_Constraint (Typ));
                   while Present (Discr) loop
-                     Append (New_Copy (Elists.Node (Discr)), Args);
+                     Nod := Node (Discr);
+                     Append (New_Copy_Tree (Node (Discr)), Args);
+
+                     --  AI-416: when the discriminant constraint is an
+                     --  anonymous access type make sure an accessibility
+                     --  check is inserted if necessary (3.10.2(22.q/2))
+
+                     if Ada_Version >= Ada_05
+                       and then Ekind (Etype (Nod)) = E_Anonymous_Access_Type
+                     then
+                        Apply_Accessibility_Check (Nod, Typ);
+                     end if;
+
                      Next_Elmt (Discr);
                   end loop;
                end if;
+            end;
 
-               --  We set the allocator as analyzed so that when we analyze the
-               --  expression actions node, we do not get an unwanted recursive
-               --  expansion of the allocator expression.
-
-               Set_Analyzed (N, True);
-               Node := Relocate_Node (N);
+            --  We set the allocator as analyzed so that when we analyze the
+            --  expression actions node, we do not get an unwanted recursive
+            --  expansion of the allocator expression.
 
-               --  Here is the transformation:
-               --    input:  new T
-               --    output: Temp : constant ptr_T := new T;
-               --            Init (Temp.all, ...);
-               --    <CTRL>  Attach_To_Final_List (Finalizable (Temp.all));
-               --    <CTRL>  Initialize (Finalizable (Temp.all));
+            Set_Analyzed (N, True);
+            Nod := Relocate_Node (N);
 
-               --  Here ptr_T is the pointer type for the allocator, and T
-               --  is the subtype of the allocator.
+            --  Here is the transformation:
+            --    input:  new T
+            --    output: Temp : constant ptr_T := new T;
+            --            Init (Temp.all, ...);
+            --    <CTRL>  Attach_To_Final_List (Finalizable (Temp.all));
+            --    <CTRL>  Initialize (Finalizable (Temp.all));
 
-               Temp_Decl :=
-                 Make_Object_Declaration (Loc,
-                   Defining_Identifier => Temp,
-                   Constant_Present    => True,
-                   Object_Definition   => New_Reference_To (Temp_Type, Loc),
-                   Expression          => Node);
+            --  Here ptr_T is the pointer type for the allocator, and is the
+            --  subtype of the allocator.
 
-               Set_Assignment_OK (Temp_Decl);
+            Temp_Decl :=
+              Make_Object_Declaration (Loc,
+                Defining_Identifier => Temp,
+                Constant_Present    => True,
+                Object_Definition   => New_Reference_To (Temp_Type, Loc),
+                Expression          => Nod);
 
-               if Is_CPP_Class (T) then
-                  Set_Aliased_Present (Temp_Decl);
-               end if;
+            Set_Assignment_OK (Temp_Decl);
+            Insert_Action (N, Temp_Decl, Suppress => All_Checks);
 
-               Insert_Action (N, Temp_Decl, Suppress => All_Checks);
+            --  If the designated type is a task type or contains tasks,
+            --  create block to activate created tasks, and insert
+            --  declaration for Task_Image variable ahead of call.
 
-               --  Case of designated type is task or contains task
-               --  Create block to activate created tasks, and insert
-               --  declaration for Task_Image variable ahead of call.
+            if Has_Task (T) then
+               declare
+                  L   : constant List_Id := New_List;
+                  Blk : Node_Id;
 
-               if Has_Task (T) then
-                  declare
-                     L   : List_Id := New_List;
-                     Blk : Node_Id;
+               begin
+                  Build_Task_Allocate_Block (L, Nod, Args);
+                  Blk := Last (L);
 
-                  begin
-                     Build_Task_Allocate_Block (L, Node, Args);
-                     Blk := Last (L);
+                  Insert_List_Before (First (Declarations (Blk)), Decls);
+                  Insert_Actions (N, L);
+               end;
 
-                     Insert_List_Before (First (Declarations (Blk)), Decls);
-                     Insert_Actions (N, L);
-                  end;
+            else
+               Insert_Action (N,
+                 Make_Procedure_Call_Statement (Loc,
+                   Name => New_Reference_To (Init, Loc),
+                   Parameter_Associations => Args));
+            end if;
 
-               else
-                  Insert_Action (N,
-                    Make_Procedure_Call_Statement (Loc,
-                      Name => New_Reference_To (Init, Loc),
-                      Parameter_Associations => Args));
-               end if;
+            if Controlled_Type (T) then
 
-               if Controlled_Type (T) then
+               --  Postpone the generation of a finalization call for the
+               --  current allocator if it acts as a coextension.
 
-                  --  If the context is an access parameter, we need to create
-                  --  a non-anonymous access type in order to have a usable
-                  --  final list, because there is otherwise no pool to which
-                  --  the allocated object can belong. We create both the type
-                  --  and the finalization chain here, because freezing an
-                  --  internal type does not create such a chain.
+               if Is_Dynamic_Coextension (N) then
+                  if No (Coextensions (N)) then
+                     Set_Coextensions (N, New_Elmt_List);
+                  end if;
 
-                  if Ekind (PtrT) = E_Anonymous_Access_Type then
-                     declare
-                        Acc : Entity_Id :=
-                                Make_Defining_Identifier (Loc,
-                                  New_Internal_Name ('I'));
-                     begin
-                        Insert_Action (N,
-                          Make_Full_Type_Declaration (Loc,
-                            Defining_Identifier => Acc,
-                            Type_Definition =>
-                               Make_Access_To_Object_Definition (Loc,
-                                 Subtype_Indication =>
-                                   New_Occurrence_Of (T, Loc))));
-
-                        Build_Final_List (N, Acc);
-                        Flist := Find_Final_List (Acc);
-                     end;
+                  Append_Elmt (New_Copy_Tree (Arg1), Coextensions (N));
 
+               else
+                  Flist := Get_Allocator_Final_List (N, Base_Type (T), PtrT);
+
+                  --  Anonymous access types created for access parameters
+                  --  are attached to an explicitly constructed controller,
+                  --  which ensures that they can be finalized properly, even
+                  --  if their deallocation might not happen. The list
+                  --  associated with the controller is doubly-linked. For
+                  --  other anonymous access types, the object may end up
+                  --  on the global final list which is singly-linked.
+                  --  Work needed for access discriminants in Ada 2005 ???
+
+                  if Ekind (PtrT) = E_Anonymous_Access_Type
+                       and then
+                         Nkind (Associated_Node_For_Itype (PtrT))
+                           not in N_Subprogram_Specification
+                  then
+                     Attach_Level := Uint_1;
                   else
-                     Flist := Find_Final_List (PtrT);
+                     Attach_Level := Uint_2;
                   end if;
 
                   Insert_Actions (N,
@@ -1986,30 +3472,37 @@ package body Exp_Ch4 is
                       Ref          => New_Copy_Tree (Arg1),
                       Typ          => T,
                       Flist_Ref    => Flist,
-                      With_Attach  => Make_Integer_Literal (Loc, 2)));
+                      With_Attach  => Make_Integer_Literal
+                                        (Loc, Attach_Level)));
                end if;
+            end if;
 
-               if Is_CPP_Class (T) then
-                  Rewrite (N,
-                    Make_Attribute_Reference (Loc,
-                      Prefix => New_Reference_To (Temp, Loc),
-                      Attribute_Name => Name_Unchecked_Access));
-               else
-                  Rewrite (N, New_Reference_To (Temp, Loc));
-               end if;
+            Rewrite (N, New_Reference_To (Temp, Loc));
+            Analyze_And_Resolve (N, PtrT);
+         end if;
+      end;
 
-               Analyze_And_Resolve (N, PtrT);
-            end if;
-         end;
+      --  Ada 2005 (AI-251): If the allocator is for a class-wide interface
+      --  object that has been rewritten as a reference, we displace "this"
+      --  to reference properly its secondary dispatch table.
+
+      if Nkind (N) = N_Identifier
+        and then Is_Interface (Dtyp)
+      then
+         Displace_Allocator_Pointer (N);
       end if;
+
+   exception
+      when RE_Not_Available =>
+         return;
    end Expand_N_Allocator;
 
    -----------------------
    -- Expand_N_And_Then --
    -----------------------
 
-   --  Expand into conditional expression if Actions present, and also
-   --  deal with optimizing case of arguments being True or False.
+   --  Expand into conditional expression if Actions present, and also deal
+   --  with optimizing case of arguments being True or False.
 
    procedure Expand_N_And_Then (N : Node_Id) is
       Loc     : constant Source_Ptr := Sloc (N);
@@ -2044,9 +3537,9 @@ package body Exp_Ch4 is
             Adjust_Result_Type (N, Typ);
             return;
 
-         --  If left argument is False, change (False and then Right) to
-         --  False. In this case we can forget the actions associated with
-         --  Right, since they will never be executed.
+         --  If left argument is False, change (False and then Right) to False.
+         --  In this case we can forget the actions associated with Right,
+         --  since they will never be executed.
 
          elsif Entity (Left) = Standard_False then
             Kill_Dead_Code (Right);
@@ -2139,7 +3632,7 @@ package body Exp_Ch4 is
       --         Cnn := else-expr
       --      end if;
 
-      --  and replace the conditional expression by a reference to Cnn.
+      --  and replace the conditional expression by a reference to Cnn
 
       if Present (Then_Actions (N)) or else Present (Else_Actions (N)) then
          Cnn := Make_Defining_Identifier (Loc, New_Internal_Name ('C'));
@@ -2158,6 +3651,9 @@ package body Exp_Ch4 is
                  Name => New_Occurrence_Of (Cnn, Sloc (Elsex)),
                  Expression => Relocate_Node (Elsex))));
 
+         Set_Assignment_OK (Name (First (Then_Statements (New_If))));
+         Set_Assignment_OK (Name (First (Else_Statements (New_If))));
+
          if Present (Then_Actions (N)) then
             Insert_List_Before
               (First (Then_Statements (New_If)), Then_Actions (N));
@@ -2186,34 +3682,222 @@ package body Exp_Ch4 is
 
    procedure Expand_N_Explicit_Dereference (N : Node_Id) is
    begin
-      --  The only processing required is an insertion of an explicit
-      --  dereference call for the checked storage pool case.
+      --  Insert explicit dereference call for the checked storage pool case
 
       Insert_Dereference_Action (Prefix (N));
    end Expand_N_Explicit_Dereference;
 
-   -----------------
-   -- Expand_N_In --
-   -----------------
+   -----------------
+   -- Expand_N_In --
+   -----------------
+
+   procedure Expand_N_In (N : Node_Id) is
+      Loc    : constant Source_Ptr := Sloc (N);
+      Rtyp   : constant Entity_Id  := Etype (N);
+      Lop    : constant Node_Id    := Left_Opnd (N);
+      Rop    : constant Node_Id    := Right_Opnd (N);
+      Static : constant Boolean    := Is_OK_Static_Expression (N);
+
+      procedure Substitute_Valid_Check;
+      --  Replaces node N by Lop'Valid. This is done when we have an explicit
+      --  test for the left operand being in range of its subtype.
+
+      ----------------------------
+      -- Substitute_Valid_Check --
+      ----------------------------
+
+      procedure Substitute_Valid_Check is
+      begin
+         Rewrite (N,
+           Make_Attribute_Reference (Loc,
+             Prefix         => Relocate_Node (Lop),
+             Attribute_Name => Name_Valid));
+
+         Analyze_And_Resolve (N, Rtyp);
+
+         Error_Msg_N ("?explicit membership test may be optimized away", N);
+         Error_Msg_N ("\?use ''Valid attribute instead", N);
+         return;
+      end Substitute_Valid_Check;
+
+   --  Start of processing for Expand_N_In
+
+   begin
+      --  Check case of explicit test for an expression in range of its
+      --  subtype. This is suspicious usage and we replace it with a 'Valid
+      --  test and give a warning.
+
+      if Is_Scalar_Type (Etype (Lop))
+        and then Nkind (Rop) in N_Has_Entity
+        and then Etype (Lop) = Entity (Rop)
+        and then Comes_From_Source (N)
+        and then VM_Target = No_VM
+      then
+         Substitute_Valid_Check;
+         return;
+      end if;
+
+      --  Do validity check on operands
+
+      if Validity_Checks_On and Validity_Check_Operands then
+         Ensure_Valid (Left_Opnd (N));
+         Validity_Check_Range (Right_Opnd (N));
+      end if;
+
+      --  Case of explicit range
+
+      if Nkind (Rop) = N_Range then
+         declare
+            Lo : constant Node_Id := Low_Bound (Rop);
+            Hi : constant Node_Id := High_Bound (Rop);
+
+            Ltyp : constant Entity_Id := Etype (Lop);
+
+            Lo_Orig : constant Node_Id := Original_Node (Lo);
+            Hi_Orig : constant Node_Id := Original_Node (Hi);
+
+            Lcheck : constant Compare_Result := Compile_Time_Compare (Lop, Lo);
+            Ucheck : constant Compare_Result := Compile_Time_Compare (Lop, Hi);
+
+            Warn1 : constant Boolean :=
+                      Constant_Condition_Warnings
+                        and then Comes_From_Source (N);
+            --  This must be true for any of the optimization warnings, we
+            --  clearly want to give them only for source with the flag on.
+
+            Warn2 : constant Boolean :=
+                      Warn1
+                        and then Nkind (Original_Node (Rop)) = N_Range
+                        and then Is_Integer_Type (Etype (Lo));
+            --  For the case where only one bound warning is elided, we also
+            --  insist on an explicit range and an integer type. The reason is
+            --  that the use of enumeration ranges including an end point is
+            --  common, as is the use of a subtype name, one of whose bounds
+            --  is the same as the type of the expression.
+
+         begin
+            --  If test is explicit x'first .. x'last, replace by valid check
+
+            if Is_Scalar_Type (Ltyp)
+              and then Nkind (Lo_Orig) = N_Attribute_Reference
+              and then Attribute_Name (Lo_Orig) = Name_First
+              and then Nkind (Prefix (Lo_Orig)) in N_Has_Entity
+              and then Entity (Prefix (Lo_Orig)) = Ltyp
+              and then Nkind (Hi_Orig) = N_Attribute_Reference
+              and then Attribute_Name (Hi_Orig) = Name_Last
+              and then Nkind (Prefix (Hi_Orig)) in N_Has_Entity
+              and then Entity (Prefix (Hi_Orig)) = Ltyp
+              and then Comes_From_Source (N)
+              and then VM_Target = No_VM
+            then
+               Substitute_Valid_Check;
+               return;
+            end if;
+
+            --  If bounds of type are known at compile time, and the end points
+            --  are known at compile time and identical, this is another case
+            --  for substituting a valid test. We only do this for discrete
+            --  types, since it won't arise in practice for float types.
+
+            if Comes_From_Source (N)
+              and then Is_Discrete_Type (Ltyp)
+              and then Compile_Time_Known_Value (Type_High_Bound (Ltyp))
+              and then Compile_Time_Known_Value (Type_Low_Bound  (Ltyp))
+              and then Compile_Time_Known_Value (Lo)
+              and then Compile_Time_Known_Value (Hi)
+              and then Expr_Value (Type_High_Bound (Ltyp)) = Expr_Value (Hi)
+              and then Expr_Value (Type_Low_Bound  (Ltyp)) = Expr_Value (Lo)
+            then
+               Substitute_Valid_Check;
+               return;
+            end if;
+
+            --  If we have an explicit range, do a bit of optimization based
+            --  on range analysis (we may be able to kill one or both checks).
+
+            --  If either check is known to fail, replace result by False since
+            --  the other check does not matter. Preserve the static flag for
+            --  legality checks, because we are constant-folding beyond RM 4.9.
+
+            if Lcheck = LT or else Ucheck = GT then
+               if Warn1 then
+                  Error_Msg_N ("?range test optimized away", N);
+                  Error_Msg_N ("\?value is known to be out of range", N);
+               end if;
+
+               Rewrite (N,
+                 New_Reference_To (Standard_False, Loc));
+               Analyze_And_Resolve (N, Rtyp);
+               Set_Is_Static_Expression (N, Static);
+
+               return;
+
+            --  If both checks are known to succeed, replace result
+            --  by True, since we know we are in range.
+
+            elsif Lcheck in Compare_GE and then Ucheck in Compare_LE then
+               if Warn1 then
+                  Error_Msg_N ("?range test optimized away", N);
+                  Error_Msg_N ("\?value is known to be in range", N);
+               end if;
+
+               Rewrite (N,
+                 New_Reference_To (Standard_True, Loc));
+               Analyze_And_Resolve (N, Rtyp);
+               Set_Is_Static_Expression (N, Static);
+
+               return;
+
+            --  If lower bound check succeeds and upper bound check is not
+            --  known to succeed or fail, then replace the range check with
+            --  a comparison against the upper bound.
+
+            elsif Lcheck in Compare_GE then
+               if Warn2 then
+                  Error_Msg_N ("?lower bound test optimized away", Lo);
+                  Error_Msg_N ("\?value is known to be in range", Lo);
+               end if;
+
+               Rewrite (N,
+                 Make_Op_Le (Loc,
+                   Left_Opnd  => Lop,
+                   Right_Opnd => High_Bound (Rop)));
+               Analyze_And_Resolve (N, Rtyp);
+
+               return;
+
+            --  If upper bound check succeeds and lower bound check is not
+            --  known to succeed or fail, then replace the range check with
+            --  a comparison against the lower bound.
+
+            elsif Ucheck in Compare_LE then
+               if Warn2 then
+                  Error_Msg_N ("?upper bound test optimized away", Hi);
+                  Error_Msg_N ("\?value is known to be in range", Hi);
+               end if;
+
+               Rewrite (N,
+                 Make_Op_Ge (Loc,
+                   Left_Opnd  => Lop,
+                   Right_Opnd => Low_Bound (Rop)));
+               Analyze_And_Resolve (N, Rtyp);
 
-   procedure Expand_N_In (N : Node_Id) is
-      Loc  : constant Source_Ptr := Sloc (N);
-      Rtyp : constant Entity_Id  := Etype (N);
+               return;
+            end if;
+         end;
 
-   begin
-      --  No expansion is required if we have an explicit range
+         --  For all other cases of an explicit range, nothing to be done
 
-      if Nkind (Right_Opnd (N)) = N_Range then
          return;
 
       --  Here right operand is a subtype mark
 
       else
          declare
-            Typ    : Entity_Id := Etype (Right_Opnd (N));
-            Obj    : Node_Id   := Left_Opnd (N);
-            Cond   : Node_Id := Empty;
-            Is_Acc : Boolean := Is_Access_Type (Typ);
+            Typ    : Entity_Id        := Etype (Rop);
+            Is_Acc : constant Boolean := Is_Access_Type (Typ);
+            Obj    : Node_Id          := Lop;
+            Cond   : Node_Id          := Empty;
 
          begin
             Remove_Side_Effects (Obj);
@@ -2221,25 +3905,25 @@ package body Exp_Ch4 is
             --  For tagged type, do tagged membership operation
 
             if Is_Tagged_Type (Typ) then
-               --  No expansion will be performed when Java_VM, as the
-               --  JVM back end will handle the membership tests directly
-               --  (tags are not explicitly represented in Java objects,
-               --  so the normal tagged membership expansion is not what
-               --  we want).
 
-               if not Java_VM then
+               --  No expansion will be performed when VM_Target, as the VM
+               --  back-ends will handle the membership tests directly (tags
+               --  are not explicitly represented in Java objects, so the
+               --  normal tagged membership expansion is not what we want).
+
+               if VM_Target = No_VM then
                   Rewrite (N, Tagged_Membership (N));
                   Analyze_And_Resolve (N, Rtyp);
                end if;
 
                return;
 
-            --  If type is scalar type, rewrite as x in t'first .. t'last
+            --  If type is scalar type, rewrite as x in t'first .. t'last.
             --  This reason we do this is that the bounds may have the wrong
             --  type if they come from the original type definition.
 
             elsif Is_Scalar_Type (Typ) then
-               Rewrite (Right_Opnd (N),
+               Rewrite (Rop,
                  Make_Range (Loc,
                    Low_Bound =>
                      Make_Attribute_Reference (Loc,
@@ -2252,8 +3936,31 @@ package body Exp_Ch4 is
                        Prefix => New_Reference_To (Typ, Loc))));
                Analyze_And_Resolve (N, Rtyp);
                return;
+
+            --  Ada 2005 (AI-216): Program_Error is raised when evaluating
+            --  a membership test if the subtype mark denotes a constrained
+            --  Unchecked_Union subtype and the expression lacks inferable
+            --  discriminants.
+
+            elsif Is_Unchecked_Union (Base_Type (Typ))
+              and then Is_Constrained (Typ)
+              and then not Has_Inferable_Discriminants (Lop)
+            then
+               Insert_Action (N,
+                 Make_Raise_Program_Error (Loc,
+                   Reason => PE_Unchecked_Union_Restriction));
+
+               --  Prevent Gigi from generating incorrect code by rewriting
+               --  the test as a standard False.
+
+               Rewrite (N,
+                 New_Occurrence_Of (Standard_False, Loc));
+
+               return;
             end if;
 
+            --  Here we have a non-scalar type
+
             if Is_Acc then
                Typ := Designated_Type (Typ);
             end if;
@@ -2269,19 +3976,21 @@ package body Exp_Ch4 is
 
             elsif Is_Array_Type (Typ) then
 
-               declare
+               Check_Subscripts : declare
                   function Construct_Attribute_Reference
-                    (E    : Node_Id;
-                     Nam  : Name_Id;
-                     Dim  : Nat)
-                     return Node_Id;
+                    (E   : Node_Id;
+                     Nam : Name_Id;
+                     Dim : Nat) return Node_Id;
                   --  Build attribute reference E'Nam(Dim)
 
+                  -----------------------------------
+                  -- Construct_Attribute_Reference --
+                  -----------------------------------
+
                   function Construct_Attribute_Reference
-                    (E    : Node_Id;
-                     Nam  : Name_Id;
-                     Dim  : Nat)
-                     return Node_Id
+                    (E   : Node_Id;
+                     Nam : Name_Id;
+                     Dim : Nat) return Node_Id
                   is
                   begin
                      return
@@ -2292,13 +4001,16 @@ package body Exp_Ch4 is
                            Make_Integer_Literal (Loc, Dim)));
                   end Construct_Attribute_Reference;
 
+               --  Start processing for Check_Subscripts
+
                begin
                   for J in 1 .. Number_Dimensions (Typ) loop
                      Evolve_And_Then (Cond,
                        Make_Op_Eq (Loc,
                          Left_Opnd  =>
                            Construct_Attribute_Reference
-                             (Duplicate_Subexpr (Obj), Name_First, J),
+                             (Duplicate_Subexpr_No_Checks (Obj),
+                              Name_First, J),
                          Right_Opnd =>
                            Construct_Attribute_Reference
                              (New_Occurrence_Of (Typ, Loc), Name_First, J)));
@@ -2307,24 +4019,26 @@ package body Exp_Ch4 is
                        Make_Op_Eq (Loc,
                          Left_Opnd  =>
                            Construct_Attribute_Reference
-                             (Duplicate_Subexpr (Obj), Name_Last, J),
+                             (Duplicate_Subexpr_No_Checks (Obj),
+                              Name_Last, J),
                          Right_Opnd =>
                            Construct_Attribute_Reference
                              (New_Occurrence_Of (Typ, Loc), Name_Last, J)));
                   end loop;
 
                   if Is_Acc then
-                     Cond := Make_Or_Else (Loc,
-                       Left_Opnd =>
-                         Make_Op_Eq (Loc,
-                           Left_Opnd  => Obj,
-                           Right_Opnd => Make_Null (Loc)),
-                       Right_Opnd => Cond);
+                     Cond :=
+                       Make_Or_Else (Loc,
+                         Left_Opnd =>
+                           Make_Op_Eq (Loc,
+                             Left_Opnd  => Obj,
+                             Right_Opnd => Make_Null (Loc)),
+                         Right_Opnd => Cond);
                   end if;
 
                   Rewrite (N, Cond);
                   Analyze_And_Resolve (N, Rtyp);
-               end;
+               end Check_Subscripts;
 
             --  These are the cases where constraint checks may be
             --  required, e.g. records with possible discriminants
@@ -2403,12 +4117,14 @@ package body Exp_Ch4 is
       --  was necessary, but it cleans up the code to do it all the time.
 
       if Is_Access_Type (T) then
-         Rewrite (P,
-           Make_Explicit_Dereference (Sloc (N),
-             Prefix => Relocate_Node (P)));
+         Insert_Explicit_Dereference (P);
          Analyze_And_Resolve (P, Designated_Type (T));
       end if;
 
+      --  Generate index and validity checks
+
+      Generate_Index_Checks (N);
+
       if Validity_Checks_On and then Validity_Check_Subscripts then
          Apply_Subscript_Validity_Checks (N);
       end if;
@@ -2432,7 +4148,8 @@ package body Exp_Ch4 is
       --  convert it to a reference to the corresponding Packed_Array_Type.
       --  We only want to do this for simple references, and not for:
 
-      --    Left side of assignment (or prefix of left side of assignment)
+      --    Left side of assignment, or prefix of left side of assignment,
+      --    or prefix of the prefix, to handle packed arrays of packed arrays,
       --      This case is handled in Exp_Ch5.Expand_N_Assignment_Statement
 
       --    Renaming objects in renaming associations
@@ -2456,8 +4173,8 @@ package body Exp_Ch4 is
             if Nkind (Parnt) = N_Unchecked_Expression then
                null;
 
-            elsif Nkind (Parnt) = N_Object_Renaming_Declaration
-              or else Nkind (Parnt) = N_Procedure_Call_Statement
+            elsif Nkind_In (Parnt, N_Object_Renaming_Declaration,
+                                   N_Procedure_Call_Statement)
               or else (Nkind (Parnt) = N_Parameter_Association
                         and then
                           Nkind (Parent (Parnt)) =  N_Procedure_Call_Statement)
@@ -2477,14 +4194,27 @@ package body Exp_Ch4 is
             then
                return;
 
+            --  If the expression is an index of an indexed component,
+            --  it must be expanded regardless of context.
+
+            elsif Nkind (Parnt) = N_Indexed_Component
+              and then Child /= Prefix (Parnt)
+            then
+               Expand_Packed_Element_Reference (N);
+               return;
+
+            elsif Nkind (Parent (Parnt)) = N_Assignment_Statement
+              and then Name (Parent (Parnt)) = Parnt
+            then
+               return;
+
             elsif Nkind (Parnt) = N_Attribute_Reference
               and then Attribute_Name (Parnt) = Name_Read
               and then Next (First (Expressions (Parnt))) = Child
             then
                return;
 
-            elsif (Nkind (Parnt) = N_Indexed_Component
-                    or else Nkind (Parnt) = N_Selected_Component)
+            elsif Nkind_In (Parnt, N_Indexed_Component, N_Selected_Component)
                and then Prefix (Parnt) = Child
             then
                null;
@@ -2501,7 +4231,6 @@ package body Exp_Ch4 is
             Parnt := Parent (Child);
          end loop;
       end;
-
    end Expand_N_Indexed_Component;
 
    ---------------------
@@ -2512,8 +4241,9 @@ package body Exp_Ch4 is
    --  can be done. This avoids needing to duplicate this expansion code.
 
    procedure Expand_N_Not_In (N : Node_Id) is
-      Loc  : constant Source_Ptr := Sloc (N);
-      Typ  : constant Entity_Id  := Etype (N);
+      Loc : constant Source_Ptr := Sloc (N);
+      Typ : constant Entity_Id  := Etype (N);
+      Cfs : constant Boolean    := Comes_From_Source (N);
 
    begin
       Rewrite (N,
@@ -2522,6 +4252,15 @@ package body Exp_Ch4 is
             Make_In (Loc,
               Left_Opnd  => Left_Opnd (N),
               Right_Opnd => Right_Opnd (N))));
+
+      --  We want this to appear as coming from source if original does (see
+      --  tranformations in Expand_N_In).
+
+      Set_Comes_From_Source (N, Cfs);
+      Set_Comes_From_Source (Right_Opnd (N), Cfs);
+
+      --  Now analyze tranformed node
+
       Analyze_And_Resolve (N, Typ);
    end Expand_N_Not_In;
 
@@ -2541,7 +4280,7 @@ package body Exp_Ch4 is
       Agg : Node_Id;
 
    begin
-      if Ekind (Typ) = E_Access_Protected_Subprogram_Type then
+      if Is_Access_Protected_Subprogram_Type (Typ) then
          Agg :=
            Make_Aggregate (Loc,
              Expressions => New_List (
@@ -2557,6 +4296,10 @@ package body Exp_Ch4 is
 
          Set_Etype (N, Typ);
       end if;
+
+   exception
+      when RE_Not_Available =>
+         return;
    end Expand_N_Null;
 
    ---------------------
@@ -2572,33 +4315,34 @@ package body Exp_Ch4 is
 
       --  Deal with software overflow checking
 
-      if Software_Overflow_Checking
+      if not Backend_Overflow_Checks_On_Target
          and then Is_Signed_Integer_Type (Etype (N))
          and then Do_Overflow_Check (N)
       then
-         --  Software overflow checking expands abs (expr) into
+         --  The only case to worry about is when the argument is
+         --  equal to the largest negative number, so what we do is
+         --  to insert the check:
 
-         --    (if expr >= 0 then expr else -expr)
+         --     [constraint_error when Expr = typ'Base'First]
 
          --  with the usual Duplicate_Subexpr use coding for expr
 
-         Rewrite (N,
-           Make_Conditional_Expression (Loc,
-             Expressions => New_List (
-               Make_Op_Ge (Loc,
+         Insert_Action (N,
+           Make_Raise_Constraint_Error (Loc,
+             Condition =>
+               Make_Op_Eq (Loc,
                  Left_Opnd  => Duplicate_Subexpr (Expr),
-                 Right_Opnd => Make_Integer_Literal (Loc, 0)),
-
-               Duplicate_Subexpr (Expr),
-
-               Make_Op_Minus (Loc,
-                 Right_Opnd  => Duplicate_Subexpr (Expr)))));
-
-         Analyze_And_Resolve (N);
+                 Right_Opnd =>
+                   Make_Attribute_Reference (Loc,
+                     Prefix =>
+                       New_Occurrence_Of (Base_Type (Etype (Expr)), Loc),
+                     Attribute_Name => Name_First)),
+             Reason => CE_Overflow_Check_Failed));
+      end if;
 
       --  Vax floating-point types case
 
-      elsif Vax_Float (Etype (N)) then
+      if Vax_Float (Etype (N)) then
          Expand_Vax_Arith (N);
       end if;
    end Expand_N_Op_Abs;
@@ -2630,7 +4374,7 @@ package body Exp_Ch4 is
          end if;
       end if;
 
-      --  Arithemtic overflow checks for signed integer/fixed point types
+      --  Arithmetic overflow checks for signed integer/fixed point types
 
       if Is_Signed_Integer_Type (Typ)
         or else Is_Fixed_Point_Type (Typ)
@@ -2670,8 +4414,22 @@ package body Exp_Ch4 is
    -- Expand_N_Op_Concat --
    ------------------------
 
-   procedure Expand_N_Op_Concat (N : Node_Id) is
+   Max_Available_String_Operands : Int := -1;
+   --  This is initialized the first time this routine is called. It records
+   --  a value of 0,2,3,4,5 depending on what Str_Concat_n procedures are
+   --  available in the run-time:
+   --
+   --    0  None available
+   --    2  RE_Str_Concat available, RE_Str_Concat_3 not available
+   --    3  RE_Str_Concat/Concat_2 available, RE_Str_Concat_4 not available
+   --    4  RE_Str_Concat/Concat_2/3 available, RE_Str_Concat_5 not available
+   --    5  All routines including RE_Str_Concat_5 available
+
+   Char_Concat_Available : Boolean;
+   --  Records if the routines RE_Str_Concat_CC/CS/SC are available. True if
+   --  all three are available, False if any one of these is unavailable.
 
+   procedure Expand_N_Op_Concat (N : Node_Id) is
       Opnds : List_Id;
       --  List of operands to be concatenated
 
@@ -2689,6 +4447,49 @@ package body Exp_Ch4 is
       --  Component type of concatenation represented by Cnode
 
    begin
+      --  Initialize global variables showing run-time status
+
+      if Max_Available_String_Operands < 1 then
+
+         --  In No_Run_Time mode, consider that no entities are available
+
+         --  This seems wrong, RTE_Available should return False for any entity
+         --  that is not in the special No_Run_Time list of allowed entities???
+
+         if No_Run_Time_Mode then
+            Max_Available_String_Operands := 0;
+
+         --  Otherwise see what routines are available and set max operand
+         --  count according to the highest count available in the run-time.
+
+         elsif not RTE_Available (RE_Str_Concat) then
+            Max_Available_String_Operands := 0;
+
+         elsif not RTE_Available (RE_Str_Concat_3) then
+            Max_Available_String_Operands := 2;
+
+         elsif not RTE_Available (RE_Str_Concat_4) then
+            Max_Available_String_Operands := 3;
+
+         elsif not RTE_Available (RE_Str_Concat_5) then
+            Max_Available_String_Operands := 4;
+
+         else
+            Max_Available_String_Operands := 5;
+         end if;
+
+         Char_Concat_Available :=
+           not No_Run_Time_Mode
+             and then
+           RTE_Available (RE_Str_Concat_CC)
+             and then
+           RTE_Available (RE_Str_Concat_CS)
+             and then
+           RTE_Available (RE_Str_Concat_SC);
+      end if;
+
+      --  Ensure validity of both operands
+
       Binary_Op_Validity_Checks (N);
 
       --  If we are the left operand of a concatenation higher up the
@@ -2723,12 +4524,21 @@ package body Exp_Ch4 is
          Opnds := New_List (Left_Opnd (Cnode), Right_Opnd (Cnode));
          Set_Parent (Opnds, N);
 
-         --  The inner loop gathers concatenation operands
+         --  The inner loop gathers concatenation operands. We gather any
+         --  number of these in the non-string case, or if no concatenation
+         --  routines are available for string (since in that case we will
+         --  treat string like any other non-string case). Otherwise we only
+         --  gather as many operands as can be handled by the available
+         --  procedures in the run-time library (normally 5, but may be
+         --  less for the configurable run-time case).
 
          Inner : while Cnode /= N
                    and then (Base_Type (Etype (Cnode)) /= Standard_String
                                or else
-                             List_Length (Opnds) < 5)
+                             Max_Available_String_Operands = 0
+                               or else
+                             List_Length (Opnds) <
+                                               Max_Available_String_Operands)
                    and then Base_Type (Etype (Cnode)) =
                             Base_Type (Etype (Parent (Cnode)))
          loop
@@ -2744,7 +4554,9 @@ package body Exp_Ch4 is
          Atyp := Base_Type (Etype (Cnode));
          Ctyp := Base_Type (Component_Type (Etype (Cnode)));
 
-         if List_Length (Opnds) > 2 or else Atyp /= Standard_String then
+         if (List_Length (Opnds) > 2 or else Atyp /= Standard_String)
+           or else not Char_Concat_Available
+         then
             Opnd := First (Opnds);
             loop
                if Base_Type (Etype (Opnd)) = Ctyp then
@@ -2761,7 +4573,9 @@ package body Exp_Ch4 is
 
          --  Now call appropriate continuation routine
 
-         if Atyp = Standard_String then
+         if Atyp = Standard_String
+           and then Max_Available_String_Operands > 0
+         then
             Expand_Concatenate_String (Cnode, Opnds);
          else
             Expand_Concatenate_Other (Cnode, Opnds);
@@ -2777,28 +4591,28 @@ package body Exp_Ch4 is
    ------------------------
 
    procedure Expand_N_Op_Divide (N : Node_Id) is
-      Loc  : constant Source_Ptr := Sloc (N);
-      Ltyp : constant Entity_Id  := Etype (Left_Opnd (N));
-      Rtyp : constant Entity_Id  := Etype (Right_Opnd (N));
-      Typ  : Entity_Id           := Etype (N);
+      Loc   : constant Source_Ptr := Sloc (N);
+      Lopnd : constant Node_Id    := Left_Opnd (N);
+      Ropnd : constant Node_Id    := Right_Opnd (N);
+      Ltyp  : constant Entity_Id  := Etype (Lopnd);
+      Rtyp  : constant Entity_Id  := Etype (Ropnd);
+      Typ   : Entity_Id           := Etype (N);
+      Rknow : constant Boolean    := Is_Integer_Type (Typ)
+                                       and then
+                                         Compile_Time_Known_Value (Ropnd);
+      Rval  : Uint;
 
    begin
       Binary_Op_Validity_Checks (N);
 
-      --  Vax_Float is a special case
-
-      if Vax_Float (Typ) then
-         Expand_Vax_Arith (N);
-         return;
+      if Rknow then
+         Rval := Expr_Value (Ropnd);
       end if;
 
       --  N / 1 = N for integer types
 
-      if Is_Integer_Type (Typ)
-        and then Compile_Time_Known_Value (Right_Opnd (N))
-        and then Expr_Value (Right_Opnd (N)) = Uint_1
-      then
-         Rewrite (N, Left_Opnd (N));
+      if Rknow and then Rval = Uint_1 then
+         Rewrite (N, Lopnd);
          return;
       end if;
 
@@ -2806,14 +4620,21 @@ package body Exp_Ch4 is
       --  Is_Power_Of_2_For_Shift is set means that we know that our left
       --  operand is an unsigned integer, as required for this to work.
 
-      if Nkind (Right_Opnd (N)) = N_Op_Expon
-        and then Is_Power_Of_2_For_Shift (Right_Opnd (N))
+      if Nkind (Ropnd) = N_Op_Expon
+        and then Is_Power_Of_2_For_Shift (Ropnd)
+
+      --  We cannot do this transformation in configurable run time mode if we
+      --  have 64-bit --  integers and long shifts are not available.
+
+        and then
+          (Esize (Ltyp) <= 32
+             or else Support_Long_Shifts_On_Target)
       then
          Rewrite (N,
            Make_Op_Shift_Right (Loc,
-             Left_Opnd  => Left_Opnd (N),
+             Left_Opnd  => Lopnd,
              Right_Opnd =>
-               Convert_To (Standard_Natural, Right_Opnd (Right_Opnd (N)))));
+               Convert_To (Standard_Natural, Right_Opnd (Ropnd))));
          Analyze_And_Resolve (N, Typ);
          return;
       end if;
@@ -2855,77 +4676,407 @@ package body Exp_Ch4 is
             Expand_Divide_Fixed_By_Fixed_Giving_Float (N);
          end if;
 
-      --  Mixed-mode operations can appear in a non-static universal
-      --  context, in  which case the integer argument must be converted
-      --  explicitly.
+      --  Mixed-mode operations can appear in a non-static universal
+      --  context, in  which case the integer argument must be converted
+      --  explicitly.
+
+      elsif Typ = Universal_Real
+        and then Is_Integer_Type (Rtyp)
+      then
+         Rewrite (Ropnd,
+           Convert_To (Universal_Real, Relocate_Node (Ropnd)));
+
+         Analyze_And_Resolve (Ropnd, Universal_Real);
+
+      elsif Typ = Universal_Real
+        and then Is_Integer_Type (Ltyp)
+      then
+         Rewrite (Lopnd,
+           Convert_To (Universal_Real, Relocate_Node (Lopnd)));
+
+         Analyze_And_Resolve (Lopnd, Universal_Real);
+
+      --  Non-fixed point cases, do integer zero divide and overflow checks
+
+      elsif Is_Integer_Type (Typ) then
+         Apply_Divide_Check (N);
+
+         --  Check for 64-bit division available, or long shifts if the divisor
+         --  is a small power of 2 (since such divides will be converted into
+         --  long shifts.
+
+         if Esize (Ltyp) > 32
+           and then not Support_64_Bit_Divides_On_Target
+           and then
+             (not Rknow
+                or else not Support_Long_Shifts_On_Target
+                or else (Rval /= Uint_2  and then
+                         Rval /= Uint_4  and then
+                         Rval /= Uint_8  and then
+                         Rval /= Uint_16 and then
+                         Rval /= Uint_32 and then
+                         Rval /= Uint_64))
+         then
+            Error_Msg_CRT ("64-bit division", N);
+         end if;
+
+      --  Deal with Vax_Float
+
+      elsif Vax_Float (Typ) then
+         Expand_Vax_Arith (N);
+         return;
+      end if;
+   end Expand_N_Op_Divide;
+
+   --------------------
+   -- Expand_N_Op_Eq --
+   --------------------
+
+   procedure Expand_N_Op_Eq (N : Node_Id) is
+      Loc    : constant Source_Ptr := Sloc (N);
+      Typ    : constant Entity_Id  := Etype (N);
+      Lhs    : constant Node_Id    := Left_Opnd (N);
+      Rhs    : constant Node_Id    := Right_Opnd (N);
+      Bodies : constant List_Id    := New_List;
+      A_Typ  : constant Entity_Id  := Etype (Lhs);
+
+      Typl    : Entity_Id := A_Typ;
+      Op_Name : Entity_Id;
+      Prim    : Elmt_Id;
+
+      procedure Build_Equality_Call (Eq : Entity_Id);
+      --  If a constructed equality exists for the type or for its parent,
+      --  build and analyze call, adding conversions if the operation is
+      --  inherited.
+
+      function Has_Unconstrained_UU_Component (Typ : Node_Id) return Boolean;
+      --  Determines whether a type has a subcompoment of an unconstrained
+      --  Unchecked_Union subtype. Typ is a record type.
+
+      -------------------------
+      -- Build_Equality_Call --
+      -------------------------
+
+      procedure Build_Equality_Call (Eq : Entity_Id) is
+         Op_Type : constant Entity_Id := Etype (First_Formal (Eq));
+         L_Exp   : Node_Id := Relocate_Node (Lhs);
+         R_Exp   : Node_Id := Relocate_Node (Rhs);
+
+      begin
+         if Base_Type (Op_Type) /= Base_Type (A_Typ)
+           and then not Is_Class_Wide_Type (A_Typ)
+         then
+            L_Exp := OK_Convert_To (Op_Type, L_Exp);
+            R_Exp := OK_Convert_To (Op_Type, R_Exp);
+         end if;
+
+         --  If we have an Unchecked_Union, we need to add the inferred
+         --  discriminant values as actuals in the function call. At this
+         --  point, the expansion has determined that both operands have
+         --  inferable discriminants.
+
+         if Is_Unchecked_Union (Op_Type) then
+            declare
+               Lhs_Type      : constant Node_Id := Etype (L_Exp);
+               Rhs_Type      : constant Node_Id := Etype (R_Exp);
+               Lhs_Discr_Val : Node_Id;
+               Rhs_Discr_Val : Node_Id;
+
+            begin
+               --  Per-object constrained selected components require special
+               --  attention. If the enclosing scope of the component is an
+               --  Unchecked_Union, we cannot reference its discriminants
+               --  directly. This is why we use the two extra parameters of
+               --  the equality function of the enclosing Unchecked_Union.
+
+               --  type UU_Type (Discr : Integer := 0) is
+               --     . . .
+               --  end record;
+               --  pragma Unchecked_Union (UU_Type);
+
+               --  1. Unchecked_Union enclosing record:
+
+               --     type Enclosing_UU_Type (Discr : Integer := 0) is record
+               --        . . .
+               --        Comp : UU_Type (Discr);
+               --        . . .
+               --     end Enclosing_UU_Type;
+               --     pragma Unchecked_Union (Enclosing_UU_Type);
+
+               --     Obj1 : Enclosing_UU_Type;
+               --     Obj2 : Enclosing_UU_Type (1);
+
+               --     [. . .] Obj1 = Obj2 [. . .]
+
+               --     Generated code:
+
+               --     if not (uu_typeEQ (obj1.comp, obj2.comp, a, b)) then
+
+               --  A and B are the formal parameters of the equality function
+               --  of Enclosing_UU_Type. The function always has two extra
+               --  formals to capture the inferred discriminant values.
+
+               --  2. Non-Unchecked_Union enclosing record:
+
+               --     type
+               --       Enclosing_Non_UU_Type (Discr : Integer := 0)
+               --     is record
+               --        . . .
+               --        Comp : UU_Type (Discr);
+               --        . . .
+               --     end Enclosing_Non_UU_Type;
+
+               --     Obj1 : Enclosing_Non_UU_Type;
+               --     Obj2 : Enclosing_Non_UU_Type (1);
+
+               --     ...  Obj1 = Obj2 ...
+
+               --     Generated code:
+
+               --     if not (uu_typeEQ (obj1.comp, obj2.comp,
+               --                        obj1.discr, obj2.discr)) then
+
+               --  In this case we can directly reference the discriminants of
+               --  the enclosing record.
+
+               --  Lhs of equality
+
+               if Nkind (Lhs) = N_Selected_Component
+                 and then Has_Per_Object_Constraint
+                            (Entity (Selector_Name (Lhs)))
+               then
+                  --  Enclosing record is an Unchecked_Union, use formal A
+
+                  if Is_Unchecked_Union (Scope
+                       (Entity (Selector_Name (Lhs))))
+                  then
+                     Lhs_Discr_Val :=
+                       Make_Identifier (Loc,
+                         Chars => Name_A);
+
+                  --  Enclosing record is of a non-Unchecked_Union type, it is
+                  --  possible to reference the discriminant.
+
+                  else
+                     Lhs_Discr_Val :=
+                       Make_Selected_Component (Loc,
+                         Prefix => Prefix (Lhs),
+                         Selector_Name =>
+                           New_Copy
+                             (Get_Discriminant_Value
+                                (First_Discriminant (Lhs_Type),
+                                 Lhs_Type,
+                                 Stored_Constraint (Lhs_Type))));
+                  end if;
+
+               --  Comment needed here ???
+
+               else
+                  --  Infer the discriminant value
+
+                  Lhs_Discr_Val :=
+                    New_Copy
+                      (Get_Discriminant_Value
+                         (First_Discriminant (Lhs_Type),
+                          Lhs_Type,
+                          Stored_Constraint (Lhs_Type)));
+               end if;
+
+               --  Rhs of equality
+
+               if Nkind (Rhs) = N_Selected_Component
+                 and then Has_Per_Object_Constraint
+                            (Entity (Selector_Name (Rhs)))
+               then
+                  if Is_Unchecked_Union
+                       (Scope (Entity (Selector_Name (Rhs))))
+                  then
+                     Rhs_Discr_Val :=
+                       Make_Identifier (Loc,
+                         Chars => Name_B);
+
+                  else
+                     Rhs_Discr_Val :=
+                       Make_Selected_Component (Loc,
+                         Prefix => Prefix (Rhs),
+                         Selector_Name =>
+                           New_Copy (Get_Discriminant_Value (
+                             First_Discriminant (Rhs_Type),
+                             Rhs_Type,
+                             Stored_Constraint (Rhs_Type))));
+
+                  end if;
+               else
+                  Rhs_Discr_Val :=
+                    New_Copy (Get_Discriminant_Value (
+                      First_Discriminant (Rhs_Type),
+                      Rhs_Type,
+                      Stored_Constraint (Rhs_Type)));
+
+               end if;
+
+               Rewrite (N,
+                 Make_Function_Call (Loc,
+                   Name => New_Reference_To (Eq, Loc),
+                   Parameter_Associations => New_List (
+                     L_Exp,
+                     R_Exp,
+                     Lhs_Discr_Val,
+                     Rhs_Discr_Val)));
+            end;
+
+         --  Normal case, not an unchecked union
+
+         else
+            Rewrite (N,
+              Make_Function_Call (Loc,
+                Name => New_Reference_To (Eq, Loc),
+                Parameter_Associations => New_List (L_Exp, R_Exp)));
+         end if;
+
+         Analyze_And_Resolve (N, Standard_Boolean, Suppress => All_Checks);
+      end Build_Equality_Call;
+
+      ------------------------------------
+      -- Has_Unconstrained_UU_Component --
+      ------------------------------------
+
+      function Has_Unconstrained_UU_Component
+        (Typ : Node_Id) return Boolean
+      is
+         Tdef  : constant Node_Id :=
+                   Type_Definition (Declaration_Node (Base_Type (Typ)));
+         Clist : Node_Id;
+         Vpart : Node_Id;
+
+         function Component_Is_Unconstrained_UU
+           (Comp : Node_Id) return Boolean;
+         --  Determines whether the subtype of the component is an
+         --  unconstrained Unchecked_Union.
+
+         function Variant_Is_Unconstrained_UU
+           (Variant : Node_Id) return Boolean;
+         --  Determines whether a component of the variant has an unconstrained
+         --  Unchecked_Union subtype.
+
+         -----------------------------------
+         -- Component_Is_Unconstrained_UU --
+         -----------------------------------
+
+         function Component_Is_Unconstrained_UU
+           (Comp : Node_Id) return Boolean
+         is
+         begin
+            if Nkind (Comp) /= N_Component_Declaration then
+               return False;
+            end if;
+
+            declare
+               Sindic : constant Node_Id :=
+                          Subtype_Indication (Component_Definition (Comp));
+
+            begin
+               --  Unconstrained nominal type. In the case of a constraint
+               --  present, the node kind would have been N_Subtype_Indication.
+
+               if Nkind (Sindic) = N_Identifier then
+                  return Is_Unchecked_Union (Base_Type (Etype (Sindic)));
+               end if;
+
+               return False;
+            end;
+         end Component_Is_Unconstrained_UU;
+
+         ---------------------------------
+         -- Variant_Is_Unconstrained_UU --
+         ---------------------------------
+
+         function Variant_Is_Unconstrained_UU
+           (Variant : Node_Id) return Boolean
+         is
+            Clist : constant Node_Id := Component_List (Variant);
+
+         begin
+            if Is_Empty_List (Component_Items (Clist)) then
+               return False;
+            end if;
+
+            --  We only need to test one component
+
+            declare
+               Comp : Node_Id := First (Component_Items (Clist));
+
+            begin
+               while Present (Comp) loop
+                  if Component_Is_Unconstrained_UU (Comp) then
+                     return True;
+                  end if;
+
+                  Next (Comp);
+               end loop;
+            end;
+
+            --  None of the components withing the variant were of
+            --  unconstrained Unchecked_Union type.
+
+            return False;
+         end Variant_Is_Unconstrained_UU;
+
+      --  Start of processing for Has_Unconstrained_UU_Component
+
+      begin
+         if Null_Present (Tdef) then
+            return False;
+         end if;
+
+         Clist := Component_List (Tdef);
+         Vpart := Variant_Part (Clist);
 
-      elsif Typ = Universal_Real
-        and then Is_Integer_Type (Rtyp)
-      then
-         Rewrite (Right_Opnd (N),
-           Convert_To (Universal_Real, Relocate_Node (Right_Opnd (N))));
+         --  Inspect available components
 
-         Analyze_And_Resolve (Right_Opnd (N), Universal_Real);
+         if Present (Component_Items (Clist)) then
+            declare
+               Comp : Node_Id := First (Component_Items (Clist));
 
-      elsif Typ = Universal_Real
-        and then Is_Integer_Type (Ltyp)
-      then
-         Rewrite (Left_Opnd (N),
-           Convert_To (Universal_Real, Relocate_Node (Left_Opnd (N))));
+            begin
+               while Present (Comp) loop
 
-         Analyze_And_Resolve (Left_Opnd (N), Universal_Real);
+                  --  One component is sufficent
 
-      --  Non-fixed point cases, do zero divide and overflow checks
+                  if Component_Is_Unconstrained_UU (Comp) then
+                     return True;
+                  end if;
 
-      elsif Is_Integer_Type (Typ) then
-         Apply_Divide_Check (N);
-      end if;
-   end Expand_N_Op_Divide;
+                  Next (Comp);
+               end loop;
+            end;
+         end if;
 
-   --------------------
-   -- Expand_N_Op_Eq --
-   --------------------
+         --  Inspect available components withing variants
 
-   procedure Expand_N_Op_Eq (N : Node_Id) is
-      Loc     : constant Source_Ptr := Sloc (N);
-      Typ     : constant Entity_Id  := Etype (N);
-      Lhs     : constant Node_Id    := Left_Opnd (N);
-      Rhs     : constant Node_Id    := Right_Opnd (N);
-      A_Typ   : Entity_Id           := Etype (Lhs);
-      Typl    : Entity_Id := A_Typ;
-      Op_Name : Entity_Id;
-      Prim    : Elmt_Id;
-      Bodies  : List_Id := New_List;
+         if Present (Vpart) then
+            declare
+               Variant : Node_Id := First (Variants (Vpart));
 
-      procedure Build_Equality_Call (Eq : Entity_Id);
-      --  If a constructed equality exists for the type or for its parent,
-      --  build and analyze call, adding conversions if the operation is
-      --  inherited.
+            begin
+               while Present (Variant) loop
 
-      -------------------------
-      -- Build_Equality_Call --
-      -------------------------
+                  --  One component within a variant is sufficent
 
-      procedure Build_Equality_Call (Eq : Entity_Id) is
-         Op_Type : constant Entity_Id := Etype (First_Formal (Eq));
-         L_Exp   : Node_Id := Relocate_Node (Lhs);
-         R_Exp   : Node_Id := Relocate_Node (Rhs);
+                  if Variant_Is_Unconstrained_UU (Variant) then
+                     return True;
+                  end if;
 
-      begin
-         if Base_Type (Op_Type) /= Base_Type (A_Typ)
-           and then not Is_Class_Wide_Type (A_Typ)
-         then
-            L_Exp := OK_Convert_To (Op_Type, L_Exp);
-            R_Exp := OK_Convert_To (Op_Type, R_Exp);
+                  Next (Variant);
+               end loop;
+            end;
          end if;
 
-         Rewrite (N,
-           Make_Function_Call (Loc,
-             Name => New_Reference_To (Eq, Loc),
-             Parameter_Associations => New_List (L_Exp, R_Exp)));
+         --  Neither the available components, nor the components inside the
+         --  variant parts were of an unconstrained Unchecked_Union subtype.
 
-         Analyze_And_Resolve (N, Standard_Boolean, Suppress => All_Checks);
-      end Build_Equality_Call;
+         return False;
+      end Has_Unconstrained_UU_Component;
 
    --  Start of processing for Expand_N_Op_Eq
 
@@ -2934,9 +5085,10 @@ package body Exp_Ch4 is
 
       if Ekind (Typl) = E_Private_Type then
          Typl := Underlying_Type (Typl);
-
       elsif Ekind (Typl) = E_Private_Subtype then
          Typl := Underlying_Type (Base_Type (Typl));
+      else
+         null;
       end if;
 
       --  It may happen in error situations that the underlying type is not
@@ -2949,15 +5101,9 @@ package body Exp_Ch4 is
 
       Typl := Base_Type (Typl);
 
-      --  Vax float types
-
-      if Vax_Float (Typl) then
-         Expand_Vax_Comparison (N);
-         return;
-
       --  Boolean types (requiring handling of non-standard case)
 
-      elsif Is_Boolean_Type (Typl) then
+      if Is_Boolean_Type (Typl) then
          Adjust_Condition (Left_Opnd (N));
          Adjust_Condition (Right_Opnd (N));
          Set_Etype (N, Standard_Boolean);
@@ -2967,21 +5113,50 @@ package body Exp_Ch4 is
 
       elsif Is_Array_Type (Typl) then
 
-         --  Packed case
+         --  If we are doing full validity checking, and it is possible for the
+         --  array elements to be invalid then expand out array comparisons to
+         --  make sure that we check the array elements.
 
-         if Is_Bit_Packed_Array (Typl) then
-            Expand_Packed_Eq (N);
+         if Validity_Check_Operands
+           and then not Is_Known_Valid (Component_Type (Typl))
+         then
+            declare
+               Save_Force_Validity_Checks : constant Boolean :=
+                                              Force_Validity_Checks;
+            begin
+               Force_Validity_Checks := True;
+               Rewrite (N,
+                 Expand_Array_Equality
+                  (N,
+                   Relocate_Node (Lhs),
+                   Relocate_Node (Rhs),
+                   Bodies,
+                   Typl));
+               Insert_Actions (N, Bodies);
+               Analyze_And_Resolve (N, Standard_Boolean);
+               Force_Validity_Checks := Save_Force_Validity_Checks;
+            end;
+
+         --  Packed case where both operands are known aligned
 
-         --  For non-floating-point elementary types, the primitive equality
-         --  always applies, and block-bit comparison is fine. Floating-point
-         --  is an exception because of negative zeroes.
+         elsif Is_Bit_Packed_Array (Typl)
+           and then not Is_Possibly_Unaligned_Object (Lhs)
+           and then not Is_Possibly_Unaligned_Object (Rhs)
+         then
+            Expand_Packed_Eq (N);
 
-         --  However, we never use block bit comparison in No_Run_Time mode,
-         --  since this may result in a call to a run time routine
+         --  Where the component type is elementary we can use a block bit
+         --  comparison (if supported on the target) exception in the case
+         --  of floating-point (negative zero issues require element by
+         --  element comparison), and atomic types (where we must be sure
+         --  to load elements independently) and possibly unaligned arrays.
 
          elsif Is_Elementary_Type (Component_Type (Typl))
            and then not Is_Floating_Point_Type (Component_Type (Typl))
-           and then not No_Run_Time
+           and then not Is_Atomic (Component_Type (Typl))
+           and then not Is_Possibly_Unaligned_Object (Lhs)
+           and then not Is_Possibly_Unaligned_Object (Rhs)
+           and then Support_Composite_Compare_On_Target
          then
             null;
 
@@ -2991,9 +5166,12 @@ package body Exp_Ch4 is
 
          else
             Rewrite (N,
-              Expand_Array_Equality (N, Typl, A_Typ,
-                Relocate_Node (Lhs), Relocate_Node (Rhs), Bodies));
-
+              Expand_Array_Equality
+                (N,
+                 Relocate_Node (Lhs),
+                 Relocate_Node (Rhs),
+                 Bodies,
+                 Typl));
             Insert_Actions      (N, Bodies,           Suppress => All_Checks);
             Analyze_And_Resolve (N, Standard_Boolean, Suppress => All_Checks);
          end if;
@@ -3006,6 +5184,14 @@ package body Exp_Ch4 is
 
          if Is_Tagged_Type (Typl) then
 
+            --  No need to do anything else compiling under restriction
+            --  No_Dispatching_Calls. During the semantic analysis we
+            --  already notified such violation.
+
+            if Restriction_Active (No_Dispatching_Calls) then
+               return;
+            end if;
+
             --  If this is derived from an untagged private type completed
             --  with a tagged type, it does not have a full view, so we
             --  use the primitive operations of the private type.
@@ -3017,24 +5203,103 @@ package body Exp_Ch4 is
               and then Is_Derived_Type (A_Typ)
               and then No (Full_View (A_Typ))
             then
+               --  Search for equality operation, checking that the
+               --  operands have the same type. Note that we must find
+               --  a matching entry, or something is very wrong!
+
                Prim := First_Elmt (Collect_Primitive_Operations (A_Typ));
 
-               while Chars (Node (Prim)) /= Name_Op_Eq loop
+               while Present (Prim) loop
+                  exit when Chars (Node (Prim)) = Name_Op_Eq
+                    and then Etype (First_Formal (Node (Prim))) =
+                             Etype (Next_Formal (First_Formal (Node (Prim))))
+                    and then
+                      Base_Type (Etype (Node (Prim))) = Standard_Boolean;
+
                   Next_Elmt (Prim);
-                  pragma Assert (Present (Prim));
                end loop;
 
+               pragma Assert (Present (Prim));
                Op_Name := Node (Prim);
+
+            --  Find the type's predefined equality or an overriding
+            --  user-defined equality. The reason for not simply calling
+            --  Find_Prim_Op here is that there may be a user-defined
+            --  overloaded equality op that precedes the equality that
+            --  we want, so we have to explicitly search (e.g., there
+            --  could be an equality with two different parameter types).
+
             else
-               Op_Name := Find_Prim_Op (Typl, Name_Op_Eq);
+               if Is_Class_Wide_Type (Typl) then
+                  Typl := Root_Type (Typl);
+               end if;
+
+               Prim := First_Elmt (Primitive_Operations (Typl));
+               while Present (Prim) loop
+                  exit when Chars (Node (Prim)) = Name_Op_Eq
+                    and then Etype (First_Formal (Node (Prim))) =
+                             Etype (Next_Formal (First_Formal (Node (Prim))))
+                    and then
+                      Base_Type (Etype (Node (Prim))) = Standard_Boolean;
+
+                  Next_Elmt (Prim);
+               end loop;
+
+               pragma Assert (Present (Prim));
+               Op_Name := Node (Prim);
             end if;
 
             Build_Equality_Call (Op_Name);
 
+         --  Ada 2005 (AI-216): Program_Error is raised when evaluating the
+         --  predefined equality operator for a type which has a subcomponent
+         --  of an Unchecked_Union type whose nominal subtype is unconstrained.
+
+         elsif Has_Unconstrained_UU_Component (Typl) then
+            Insert_Action (N,
+              Make_Raise_Program_Error (Loc,
+                Reason => PE_Unchecked_Union_Restriction));
+
+            --  Prevent Gigi from generating incorrect code by rewriting the
+            --  equality as a standard False.
+
+            Rewrite (N,
+              New_Occurrence_Of (Standard_False, Loc));
+
+         elsif Is_Unchecked_Union (Typl) then
+
+            --  If we can infer the discriminants of the operands, we make a
+            --  call to the TSS equality function.
+
+            if Has_Inferable_Discriminants (Lhs)
+                 and then
+               Has_Inferable_Discriminants (Rhs)
+            then
+               Build_Equality_Call
+                 (TSS (Root_Type (Typl), TSS_Composite_Equality));
+
+            else
+               --  Ada 2005 (AI-216): Program_Error is raised when evaluating
+               --  the predefined equality operator for an Unchecked_Union type
+               --  if either of the operands lack inferable discriminants.
+
+               Insert_Action (N,
+                 Make_Raise_Program_Error (Loc,
+                   Reason => PE_Unchecked_Union_Restriction));
+
+               --  Prevent Gigi from generating incorrect code by rewriting
+               --  the equality as a standard False.
+
+               Rewrite (N,
+                 New_Occurrence_Of (Standard_False, Loc));
+
+            end if;
+
          --  If a type support function is present (for complex cases), use it
 
-         elsif Present (TSS (Root_Type (Typl), Name_uEquality)) then
-            Build_Equality_Call (TSS (Root_Type (Typl), Name_uEquality));
+         elsif Present (TSS (Root_Type (Typl), TSS_Composite_Equality)) then
+            Build_Equality_Call
+              (TSS (Root_Type (Typl), TSS_Composite_Equality));
 
          --  Otherwise expand the component by component equality. Note that
          --  we never use block-bit coparisons for records, because of the
@@ -3052,11 +5317,15 @@ package body Exp_Ch4 is
          end if;
       end if;
 
-      --  If we still have an equality comparison (i.e. it was not rewritten
-      --  in some way), then we can test if result is needed at compile time).
+      --  Test if result is known at compile time
 
-      if Nkind (N) = N_Op_Eq then
-         Rewrite_Comparison (N);
+      Rewrite_Comparison (N);
+
+      --  If we still have comparison for Vax_Float, process it
+
+      if Vax_Float (Typl) and then Nkind (N) in N_Op_Compare  then
+         Expand_Vax_Comparison (N);
+         return;
       end if;
    end Expand_N_Op_Eq;
 
@@ -3069,6 +5338,7 @@ package body Exp_Ch4 is
       Typ    : constant Entity_Id  := Etype (N);
       Rtyp   : constant Entity_Id  := Root_Type (Typ);
       Base   : constant Node_Id    := Relocate_Node (Left_Opnd (N));
+      Bastyp : constant Node_Id    := Etype (Base);
       Exp    : constant Node_Id    := Relocate_Node (Right_Opnd (N));
       Exptyp : constant Entity_Id  := Etype (Exp);
       Ovflo  : constant Boolean    := Do_Overflow_Check (N);
@@ -3077,14 +5347,42 @@ package body Exp_Ch4 is
       Temp   : Node_Id;
       Rent   : RE_Id;
       Ent    : Entity_Id;
+      Etyp   : Entity_Id;
 
    begin
       Binary_Op_Validity_Checks (N);
 
-      --  At this point the exponentiation must be dynamic since the static
-      --  case has already been folded after Resolve by Eval_Op_Expon.
+      --  If either operand is of a private type, then we have the use of
+      --  an intrinsic operator, and we get rid of the privateness, by using
+      --  root types of underlying types for the actual operation. Otherwise
+      --  the private types will cause trouble if we expand multiplications
+      --  or shifts etc. We also do this transformation if the result type
+      --  is different from the base type.
+
+      if Is_Private_Type (Etype (Base))
+           or else
+         Is_Private_Type (Typ)
+           or else
+         Is_Private_Type (Exptyp)
+           or else
+         Rtyp /= Root_Type (Bastyp)
+      then
+         declare
+            Bt : constant Entity_Id := Root_Type (Underlying_Type (Bastyp));
+            Et : constant Entity_Id := Root_Type (Underlying_Type (Exptyp));
+
+         begin
+            Rewrite (N,
+              Unchecked_Convert_To (Typ,
+                Make_Op_Expon (Loc,
+                  Left_Opnd  => Unchecked_Convert_To (Bt, Base),
+                  Right_Opnd => Unchecked_Convert_To (Et, Exp))));
+            Analyze_And_Resolve (N, Typ);
+            return;
+         end;
+      end if;
 
-      --  Test for case of literal right argument
+      --  Test for case of known right argument
 
       if Compile_Time_Known_Value (Exp) then
          Expv := Expr_Value (Exp);
@@ -3117,7 +5415,7 @@ package body Exp_Ch4 is
                Xnode :=
                  Make_Op_Multiply (Loc,
                    Left_Opnd  => Duplicate_Subexpr (Base),
-                   Right_Opnd => Duplicate_Subexpr (Base));
+                   Right_Opnd => Duplicate_Subexpr_No_Checks (Base));
 
             --  X ** 3 = X * X * X
 
@@ -3127,8 +5425,8 @@ package body Exp_Ch4 is
                    Left_Opnd =>
                      Make_Op_Multiply (Loc,
                        Left_Opnd  => Duplicate_Subexpr (Base),
-                       Right_Opnd => Duplicate_Subexpr (Base)),
-                   Right_Opnd  => Duplicate_Subexpr (Base));
+                       Right_Opnd => Duplicate_Subexpr_No_Checks (Base)),
+                   Right_Opnd  => Duplicate_Subexpr_No_Checks (Base));
 
             --  X ** 4  ->
             --    En : constant base'type := base * base;
@@ -3147,7 +5445,7 @@ package body Exp_Ch4 is
                    Expression =>
                      Make_Op_Multiply (Loc,
                        Left_Opnd  => Duplicate_Subexpr (Base),
-                       Right_Opnd => Duplicate_Subexpr (Base)))));
+                       Right_Opnd => Duplicate_Subexpr_No_Checks (Base)))));
 
                Xnode :=
                  Make_Op_Multiply (Loc,
@@ -3163,7 +5461,7 @@ package body Exp_Ch4 is
 
       --  Case of (2 ** expression) appearing as an argument of an integer
       --  multiplication, or as the right argument of a division of a non-
-      --  negative integer. In such cases we lave the node untouched, setting
+      --  negative integer. In such cases we leave the node untouched, setting
       --  the flag Is_Natural_Power_Of_2_for_Shift set, then the expansion
       --  of the higher level node converts it into a shift.
 
@@ -3201,9 +5499,9 @@ package body Exp_Ch4 is
          end;
       end if;
 
-      --  Fall through if exponentiation must be done using a runtime routine.
+      --  Fall through if exponentiation must be done using a runtime routine
 
-      --  First deal with modular case.
+      --  First deal with modular case
 
       if Is_Modular_Integer_Type (Rtyp) then
 
@@ -3213,7 +5511,6 @@ package body Exp_Ch4 is
          --  to the base type.
 
          if Non_Binary_Modulus (Rtyp) then
-
             Rewrite (N,
               Convert_To (Typ,
                 Make_Function_Call (Loc,
@@ -3253,83 +5550,54 @@ package body Exp_Ch4 is
          Analyze_And_Resolve (N, Typ);
          return;
 
-      --  Signed integer cases
-
-      elsif Rtyp = Base_Type (Standard_Integer) then
-         if Ovflo then
-            Rent := RE_Exp_Integer;
-         else
-            Rent := RE_Exn_Integer;
-         end if;
-
-      elsif Rtyp = Base_Type (Standard_Short_Integer) then
-         if Ovflo then
-            Rent := RE_Exp_Short_Integer;
-         else
-            Rent := RE_Exn_Short_Integer;
-         end if;
-
-      elsif Rtyp = Base_Type (Standard_Short_Short_Integer) then
-         if Ovflo then
-            Rent := RE_Exp_Short_Short_Integer;
-         else
-            Rent := RE_Exn_Short_Short_Integer;
-         end if;
+      --  Signed integer cases, done using either Integer or Long_Long_Integer.
+      --  It is not worth having routines for Short_[Short_]Integer, since for
+      --  most machines it would not help, and it would generate more code that
+      --  might need certification when a certified run time is required.
 
-      elsif Rtyp = Base_Type (Standard_Long_Integer) then
-         if Ovflo then
-            Rent := RE_Exp_Long_Integer;
-         else
-            Rent := RE_Exn_Long_Integer;
-         end if;
+      --  In the integer cases, we have two routines, one for when overflow
+      --  checks are required, and one when they are not required, since there
+      --  is a real gain in omitting checks on many machines.
 
-      elsif (Rtyp = Base_Type (Standard_Long_Long_Integer)
-        or else Rtyp = Universal_Integer)
+      elsif Rtyp = Base_Type (Standard_Long_Long_Integer)
+        or else (Rtyp = Base_Type (Standard_Long_Integer)
+                   and then
+                     Esize (Standard_Long_Integer) > Esize (Standard_Integer))
+        or else (Rtyp = Universal_Integer)
       then
+         Etyp := Standard_Long_Long_Integer;
+
          if Ovflo then
             Rent := RE_Exp_Long_Long_Integer;
          else
             Rent := RE_Exn_Long_Long_Integer;
          end if;
 
-      --  Floating-point cases
-
-      elsif Rtyp = Standard_Float then
-         if Ovflo then
-            Rent := RE_Exp_Float;
-         else
-            Rent := RE_Exn_Float;
-         end if;
+      elsif Is_Signed_Integer_Type (Rtyp) then
+         Etyp := Standard_Integer;
 
-      elsif Rtyp = Standard_Short_Float then
          if Ovflo then
-            Rent := RE_Exp_Short_Float;
+            Rent := RE_Exp_Integer;
          else
-            Rent := RE_Exn_Short_Float;
+            Rent := RE_Exn_Integer;
          end if;
 
-      elsif Rtyp = Standard_Long_Float then
-         if Ovflo then
-            Rent := RE_Exp_Long_Float;
-         else
-            Rent := RE_Exn_Long_Float;
-         end if;
+      --  Floating-point cases, always done using Long_Long_Float. We do not
+      --  need separate routines for the overflow case here, since in the case
+      --  of floating-point, we generate infinities anyway as a rule (either
+      --  that or we automatically trap overflow), and if there is an infinity
+      --  generated and a range check is required, the check will fail anyway.
 
       else
-         pragma Assert
-           (Rtyp = Standard_Long_Long_Float or else Rtyp = Universal_Real);
-
-         if Ovflo then
-            Rent := RE_Exp_Long_Long_Float;
-         else
-            Rent := RE_Exn_Long_Long_Float;
-         end if;
+         pragma Assert (Is_Floating_Point_Type (Rtyp));
+         Etyp := Standard_Long_Long_Float;
+         Rent := RE_Exn_Long_Long_Float;
       end if;
 
       --  Common processing for integer cases and floating-point cases.
-      --  If we are in the base type, we can call runtime routine directly
+      --  If we are in the right type, we can call runtime routine directly
 
-      if Typ = Rtyp
+      if Typ = Etyp
         and then Rtyp /= Universal_Integer
         and then Rtyp /= Universal_Real
       then
@@ -3339,8 +5607,8 @@ package body Exp_Ch4 is
              Parameter_Associations => New_List (Base, Exp)));
 
       --  Otherwise we have to introduce conversions (conversions are also
-      --  required in the universal cases, since the runtime routine was
-      --  typed using the largest integer or real case.
+      --  required in the universal cases, since the runtime routine is
+      --  typed using one of the standard types.
 
       else
          Rewrite (N,
@@ -3348,13 +5616,16 @@ package body Exp_Ch4 is
              Make_Function_Call (Loc,
                Name => New_Reference_To (RTE (Rent), Loc),
                Parameter_Associations => New_List (
-                 Convert_To (Rtyp, Base),
+                 Convert_To (Etyp, Base),
                  Exp))));
       end if;
 
       Analyze_And_Resolve (N, Typ);
       return;
 
+   exception
+      when RE_Not_Available =>
+         return;
    end Expand_N_Op_Expon;
 
    --------------------
@@ -3370,11 +5641,7 @@ package body Exp_Ch4 is
    begin
       Binary_Op_Validity_Checks (N);
 
-      if Vax_Float (Typ1) then
-         Expand_Vax_Comparison (N);
-         return;
-
-      elsif Is_Array_Type (Typ1) then
+      if Is_Array_Type (Typ1) then
          Expand_Array_Comparison (N);
          return;
       end if;
@@ -3387,6 +5654,13 @@ package body Exp_Ch4 is
       end if;
 
       Rewrite_Comparison (N);
+
+      --  If we still have comparison, and Vax_Float type, process it
+
+      if Vax_Float (Typ1) and then Nkind (N) in N_Op_Compare then
+         Expand_Vax_Comparison (N);
+         return;
+      end if;
    end Expand_N_Op_Ge;
 
    --------------------
@@ -3402,11 +5676,7 @@ package body Exp_Ch4 is
    begin
       Binary_Op_Validity_Checks (N);
 
-      if Vax_Float (Typ1) then
-         Expand_Vax_Comparison (N);
-         return;
-
-      elsif Is_Array_Type (Typ1) then
+      if Is_Array_Type (Typ1) then
          Expand_Array_Comparison (N);
          return;
       end if;
@@ -3419,6 +5689,13 @@ package body Exp_Ch4 is
       end if;
 
       Rewrite_Comparison (N);
+
+      --  If we still have comparison, and Vax_Float type, process it
+
+      if Vax_Float (Typ1) and then Nkind (N) in N_Op_Compare then
+         Expand_Vax_Comparison (N);
+         return;
+      end if;
    end Expand_N_Op_Gt;
 
    --------------------
@@ -3434,11 +5711,7 @@ package body Exp_Ch4 is
    begin
       Binary_Op_Validity_Checks (N);
 
-      if Vax_Float (Typ1) then
-         Expand_Vax_Comparison (N);
-         return;
-
-      elsif Is_Array_Type (Typ1) then
+      if Is_Array_Type (Typ1) then
          Expand_Array_Comparison (N);
          return;
       end if;
@@ -3451,6 +5724,13 @@ package body Exp_Ch4 is
       end if;
 
       Rewrite_Comparison (N);
+
+      --  If we still have comparison, and Vax_Float type, process it
+
+      if Vax_Float (Typ1) and then Nkind (N) in N_Op_Compare then
+         Expand_Vax_Comparison (N);
+         return;
+      end if;
    end Expand_N_Op_Le;
 
    --------------------
@@ -3466,11 +5746,7 @@ package body Exp_Ch4 is
    begin
       Binary_Op_Validity_Checks (N);
 
-      if Vax_Float (Typ1) then
-         Expand_Vax_Comparison (N);
-         return;
-
-      elsif Is_Array_Type (Typ1) then
+      if Is_Array_Type (Typ1) then
          Expand_Array_Comparison (N);
          return;
       end if;
@@ -3483,6 +5759,13 @@ package body Exp_Ch4 is
       end if;
 
       Rewrite_Comparison (N);
+
+      --  If we still have comparison, and Vax_Float type, process it
+
+      if Vax_Float (Typ1) and then Nkind (N) in N_Op_Compare then
+         Expand_Vax_Comparison (N);
+         return;
+      end if;
    end Expand_N_Op_Lt;
 
    -----------------------
@@ -3496,7 +5779,7 @@ package body Exp_Ch4 is
    begin
       Unary_Op_Validity_Checks (N);
 
-      if Software_Overflow_Checking
+      if not Backend_Overflow_Checks_On_Target
          and then Is_Signed_Integer_Type (Etype (N))
          and then Do_Overflow_Check (N)
       then
@@ -3522,7 +5805,7 @@ package body Exp_Ch4 is
 
    procedure Expand_N_Op_Mod (N : Node_Id) is
       Loc   : constant Source_Ptr := Sloc (N);
-      T     : constant Entity_Id  := Etype (N);
+      Typ   : constant Entity_Id  := Etype (N);
       Left  : constant Node_Id    := Left_Opnd (N);
       Right : constant Node_Id    := Right_Opnd (N);
       DOC   : constant Boolean    := Do_Overflow_Check (N);
@@ -3536,6 +5819,8 @@ package body Exp_Ch4 is
       Rhi : Uint;
       ROK : Boolean;
 
+      pragma Warnings (Off, Lhi);
+
    begin
       Binary_Op_Validity_Checks (N);
 
@@ -3561,7 +5846,7 @@ package body Exp_Ch4 is
          --  instance and is epsilon more efficient.
 
          Set_Entity            (N, Standard_Entity (S_Op_Rem));
-         Set_Etype             (N, T);
+         Set_Etype             (N, Typ);
          Set_Do_Overflow_Check (N, DOC);
          Set_Do_Division_Check (N, DDC);
          Expand_N_Op_Rem (N);
@@ -3574,6 +5859,19 @@ package body Exp_Ch4 is
             Apply_Divide_Check (N);
          end if;
 
+         --  Apply optimization x mod 1 = 0. We don't really need that with
+         --  gcc, but it is useful with other back ends (e.g. AAMP), and is
+         --  certainly harmless.
+
+         if Is_Integer_Type (Etype (N))
+           and then Compile_Time_Known_Value (Right)
+           and then Expr_Value (Right) = Uint_1
+         then
+            Rewrite (N, Make_Integer_Literal (Loc, 0));
+            Analyze_And_Resolve (N, Typ);
+            return;
+         end if;
+
          --  Deal with annoying case of largest negative number remainder
          --  minus one. Gigi does not handle this case correctly, because
          --  it generates a divide instruction which may trap in this case.
@@ -3582,7 +5880,13 @@ package body Exp_Ch4 is
          --  then the mod value is always 0, and we can just ignore the
          --  left operand completely in this case.
 
-         LLB := Expr_Value (Type_Low_Bound (Base_Type (Etype (Left))));
+         --  The operand type may be private (e.g. in the expansion of an
+         --  an intrinsic operation) so we must use the underlying type to
+         --  get the bounds, and convert the literals explicitly.
+
+         LLB :=
+           Expr_Value
+             (Type_Low_Bound (Base_Type (Underlying_Type (Etype (Left)))));
 
          if ((not ROK) or else (Rlo <= (-1) and then (-1) <= Rhi))
            and then
@@ -3594,12 +5898,14 @@ package body Exp_Ch4 is
                   Make_Op_Eq (Loc,
                     Left_Opnd => Duplicate_Subexpr (Right),
                     Right_Opnd =>
-                      Make_Integer_Literal (Loc, -1)),
-                  Make_Integer_Literal (Loc, Uint_0),
+                      Unchecked_Convert_To (Typ,
+                        Make_Integer_Literal (Loc, -1))),
+                  Unchecked_Convert_To (Typ,
+                    Make_Integer_Literal (Loc, Uint_0)),
                   Relocate_Node (N))));
 
             Set_Analyzed (Next (Next (First (Expressions (N)))));
-            Analyze_And_Resolve (N, T);
+            Analyze_And_Resolve (N, Typ);
          end if;
       end if;
    end Expand_N_Op_Mod;
@@ -3612,6 +5918,15 @@ package body Exp_Ch4 is
       Loc  : constant Source_Ptr := Sloc (N);
       Lop  : constant Node_Id    := Left_Opnd (N);
       Rop  : constant Node_Id    := Right_Opnd (N);
+
+      Lp2  : constant Boolean :=
+               Nkind (Lop) = N_Op_Expon
+                 and then Is_Power_Of_2_For_Shift (Lop);
+
+      Rp2  : constant Boolean :=
+               Nkind (Rop) = N_Op_Expon
+                 and then Is_Power_Of_2_For_Shift (Rop);
+
       Ltyp : constant Entity_Id  := Etype (Lop);
       Rtyp : constant Entity_Id  := Etype (Rop);
       Typ  : Entity_Id           := Etype (N);
@@ -3625,11 +5940,11 @@ package body Exp_Ch4 is
 
          --  N * 0 = 0 * N = 0 for integer types
 
-         if (Compile_Time_Known_Value (Right_Opnd (N))
-              and then Expr_Value (Right_Opnd (N)) = Uint_0)
+         if (Compile_Time_Known_Value (Rop)
+              and then Expr_Value (Rop) = Uint_0)
            or else
-            (Compile_Time_Known_Value (Left_Opnd (N))
-              and then Expr_Value (Left_Opnd (N)) = Uint_0)
+            (Compile_Time_Known_Value (Lop)
+              and then Expr_Value (Lop) = Uint_0)
          then
             Rewrite (N, Make_Integer_Literal (Loc, Uint_0));
             Analyze_And_Resolve (N, Typ);
@@ -3638,39 +5953,33 @@ package body Exp_Ch4 is
 
          --  N * 1 = 1 * N = N for integer types
 
-         if Compile_Time_Known_Value (Right_Opnd (N))
-           and then Expr_Value (Right_Opnd (N)) = Uint_1
+         --  This optimisation is not done if we are going to
+         --  rewrite the product 1 * 2 ** N to a shift.
+
+         if Compile_Time_Known_Value (Rop)
+           and then Expr_Value (Rop) = Uint_1
+           and then not Lp2
          then
-            Rewrite (N, Left_Opnd (N));
+            Rewrite (N, Lop);
             return;
 
-         elsif Compile_Time_Known_Value (Left_Opnd (N))
-           and then Expr_Value (Left_Opnd (N)) = Uint_1
+         elsif Compile_Time_Known_Value (Lop)
+           and then Expr_Value (Lop) = Uint_1
+           and then not Rp2
          then
-            Rewrite (N, Right_Opnd (N));
+            Rewrite (N, Rop);
             return;
          end if;
       end if;
 
-      --  Deal with VAX float case
-
-      if Vax_Float (Typ) then
-         Expand_Vax_Arith (N);
-         return;
-      end if;
-
       --  Convert x * 2 ** y to Shift_Left (x, y). Note that the fact that
       --  Is_Power_Of_2_For_Shift is set means that we know that our left
       --  operand is an integer, as required for this to work.
 
-      if Nkind (Rop) = N_Op_Expon
-        and then Is_Power_Of_2_For_Shift (Rop)
-      then
-         if Nkind (Lop) = N_Op_Expon
-           and then Is_Power_Of_2_For_Shift (Lop)
-         then
+      if Rp2 then
+         if Lp2 then
 
-            --  convert 2 ** A * 2 ** B into  2 ** (A + B)
+            --  Convert 2 ** A * 2 ** B into  2 ** (A + B)
 
             Rewrite (N,
               Make_Op_Expon (Loc,
@@ -3694,9 +6003,7 @@ package body Exp_Ch4 is
 
       --  Same processing for the operands the other way round
 
-      elsif Nkind (Lop) = N_Op_Expon
-        and then Is_Power_Of_2_For_Shift (Lop)
-      then
+      elsif Lp2 then
          Rewrite (N,
            Make_Op_Shift_Left (Loc,
              Left_Opnd  => Rop,
@@ -3773,42 +6080,89 @@ package body Exp_Ch4 is
 
       --  Non-fixed point cases, check software overflow checking required
 
-      elsif Is_Signed_Integer_Type (Etype (N)) then
-         Apply_Arithmetic_Overflow_Check (N);
-      end if;
-   end Expand_N_Op_Multiply;
+      elsif Is_Signed_Integer_Type (Etype (N)) then
+         Apply_Arithmetic_Overflow_Check (N);
+
+      --  Deal with VAX float case
+
+      elsif Vax_Float (Typ) then
+         Expand_Vax_Arith (N);
+         return;
+      end if;
+   end Expand_N_Op_Multiply;
+
+   --------------------
+   -- Expand_N_Op_Ne --
+   --------------------
+
+   procedure Expand_N_Op_Ne (N : Node_Id) is
+      Typ : constant Entity_Id := Etype (Left_Opnd (N));
+
+   begin
+      --  Case of elementary type with standard operator
+
+      if Is_Elementary_Type (Typ)
+        and then Sloc (Entity (N)) = Standard_Location
+      then
+         Binary_Op_Validity_Checks (N);
+
+         --  Boolean types (requiring handling of non-standard case)
+
+         if Is_Boolean_Type (Typ) then
+            Adjust_Condition (Left_Opnd (N));
+            Adjust_Condition (Right_Opnd (N));
+            Set_Etype (N, Standard_Boolean);
+            Adjust_Result_Type (N, Typ);
+         end if;
+
+         Rewrite_Comparison (N);
+
+         --  If we still have comparison for Vax_Float, process it
+
+         if Vax_Float (Typ) and then Nkind (N) in N_Op_Compare  then
+            Expand_Vax_Comparison (N);
+            return;
+         end if;
+
+      --  For all cases other than elementary types, we rewrite node as the
+      --  negation of an equality operation, and reanalyze. The equality to be
+      --  used is defined in the same scope and has the same signature. This
+      --  signature must be set explicitly since in an instance it may not have
+      --  the same visibility as in the generic unit. This avoids duplicating
+      --  or factoring the complex code for record/array equality tests etc.
 
-   --------------------
-   -- Expand_N_Op_Ne --
-   --------------------
+      else
+         declare
+            Loc : constant Source_Ptr := Sloc (N);
+            Neg : Node_Id;
+            Ne  : constant Entity_Id := Entity (N);
 
-   --  Rewrite node as the negation of an equality operation, and reanalyze.
-   --  The equality to be used is defined in the same scope and has the same
-   --  signature. It must be set explicitly because in an instance it may not
-   --  have the same visibility as in the generic unit.
+         begin
+            Binary_Op_Validity_Checks (N);
 
-   procedure Expand_N_Op_Ne (N : Node_Id) is
-      Loc : constant Source_Ptr := Sloc (N);
-      Neg : Node_Id;
-      Ne  : constant Entity_Id := Entity (N);
+            Neg :=
+              Make_Op_Not (Loc,
+                Right_Opnd =>
+                  Make_Op_Eq (Loc,
+                    Left_Opnd =>  Left_Opnd (N),
+                    Right_Opnd => Right_Opnd (N)));
+            Set_Paren_Count (Right_Opnd (Neg), 1);
 
-   begin
-      Binary_Op_Validity_Checks (N);
+            if Scope (Ne) /= Standard_Standard then
+               Set_Entity (Right_Opnd (Neg), Corresponding_Equality (Ne));
+            end if;
 
-      Neg :=
-        Make_Op_Not (Loc,
-          Right_Opnd =>
-            Make_Op_Eq (Loc,
-              Left_Opnd =>  Left_Opnd (N),
-              Right_Opnd => Right_Opnd (N)));
-      Set_Paren_Count (Right_Opnd (Neg), 1);
+            --  For navigation purposes, the inequality is treated as an
+            --  implicit reference to the corresponding equality. Preserve the
+            --  Comes_From_ source flag so that the proper Xref entry is
+            --  generated.
 
-      if Scope (Ne) /= Standard_Standard then
-         Set_Entity (Right_Opnd (Neg), Corresponding_Equality (Ne));
+            Preserve_Comes_From_Source (Neg, N);
+            Preserve_Comes_From_Source (Right_Opnd (Neg), N);
+            Rewrite (N, Neg);
+            Analyze_And_Resolve (N, Standard_Boolean);
+         end;
       end if;
-
-      Rewrite (N, Neg);
-      Analyze_And_Resolve (N, Standard_Boolean);
    end Expand_N_Op_Ne;
 
    ---------------------
@@ -3872,20 +6226,70 @@ package body Exp_Ch4 is
          return;
       end if;
 
-      --  Case of array operand. If bit packed, handle it in Exp_Pakd
+      --  Case of array operand. If bit packed with a component size of 1,
+      --  handle it in Exp_Pakd if the operand is known to be aligned.
 
-      if Is_Bit_Packed_Array (Typ) and then Component_Size (Typ) = 1 then
+      if Is_Bit_Packed_Array (Typ)
+        and then Component_Size (Typ) = 1
+        and then not Is_Possibly_Unaligned_Object (Right_Opnd (N))
+      then
          Expand_Packed_Not (N);
          return;
       end if;
 
-      --  Case of array operand which is not bit-packed
+      --  Case of array operand which is not bit-packed. If the context is
+      --  a safe assignment, call in-place operation, If context is a larger
+      --  boolean expression in the context of a safe assignment, expansion is
+      --  done by enclosing operation.
 
       Opnd := Relocate_Node (Right_Opnd (N));
       Convert_To_Actual_Subtype (Opnd);
       Arr := Etype (Opnd);
       Ensure_Defined (Arr, N);
 
+      if Nkind (Parent (N)) = N_Assignment_Statement then
+         if Safe_In_Place_Array_Op (Name (Parent (N)), N, Empty) then
+            Build_Boolean_Array_Proc_Call (Parent (N), Opnd, Empty);
+            return;
+
+         --  Special case the negation of a binary operation
+
+         elsif Nkind_In (Opnd, N_Op_And, N_Op_Or, N_Op_Xor)
+           and then Safe_In_Place_Array_Op
+                      (Name (Parent (N)), Left_Opnd (Opnd), Right_Opnd (Opnd))
+         then
+            Build_Boolean_Array_Proc_Call (Parent (N), Opnd, Empty);
+            return;
+         end if;
+
+      elsif Nkind (Parent (N)) in N_Binary_Op
+        and then Nkind (Parent (Parent (N))) = N_Assignment_Statement
+      then
+         declare
+            Op1 : constant Node_Id := Left_Opnd  (Parent (N));
+            Op2 : constant Node_Id := Right_Opnd (Parent (N));
+            Lhs : constant Node_Id := Name (Parent (Parent (N)));
+
+         begin
+            if Safe_In_Place_Array_Op (Lhs, Op1, Op2) then
+               if N = Op1
+                 and then Nkind (Op2) = N_Op_Not
+               then
+                  --  (not A) op (not B) can be reduced to a single call
+
+                  return;
+
+               elsif N = Op2
+                 and then Nkind (Parent (N)) = N_Op_Xor
+               then
+                  --  A xor (not B) can also be special-cased
+
+                  return;
+               end if;
+            end if;
+         end;
+      end if;
+
       A := Make_Defining_Identifier (Loc, Name_uA);
       B := Make_Defining_Identifier (Loc, Name_uB);
       J := Make_Defining_Identifier (Loc, Name_uJ);
@@ -3931,7 +6335,7 @@ package body Exp_Ch4 is
                 Make_Parameter_Specification (Loc,
                   Defining_Identifier => A,
                   Parameter_Type      => New_Reference_To (Typ, Loc))),
-              Subtype_Mark => New_Reference_To (Typ, Loc)),
+              Result_Definition => New_Reference_To (Typ, Loc)),
 
           Declarations => New_List (
             Make_Object_Declaration (Loc,
@@ -3942,7 +6346,7 @@ package body Exp_Ch4 is
             Make_Handled_Sequence_Of_Statements (Loc,
               Statements => New_List (
                 Loop_Statement,
-                Make_Return_Statement (Loc,
+                Make_Simple_Return_Statement (Loc,
                   Expression =>
                     Make_Identifier (Loc, Chars (B)))))));
 
@@ -3990,6 +6394,7 @@ package body Exp_Ch4 is
 
    procedure Expand_N_Op_Rem (N : Node_Id) is
       Loc : constant Source_Ptr := Sloc (N);
+      Typ : constant Entity_Id  := Etype (N);
 
       Left  : constant Node_Id := Left_Opnd (N);
       Right : constant Node_Id := Right_Opnd (N);
@@ -4001,7 +6406,8 @@ package body Exp_Ch4 is
       Rlo : Uint;
       Rhi : Uint;
       ROK : Boolean;
-      Typ : Entity_Id;
+
+      pragma Warnings (Off, Lhi);
 
    begin
       Binary_Op_Validity_Checks (N);
@@ -4010,6 +6416,19 @@ package body Exp_Ch4 is
          Apply_Divide_Check (N);
       end if;
 
+      --  Apply optimization x rem 1 = 0. We don't really need that with
+      --  gcc, but it is useful with other back ends (e.g. AAMP), and is
+      --  certainly harmless.
+
+      if Is_Integer_Type (Etype (N))
+        and then Compile_Time_Known_Value (Right)
+        and then Expr_Value (Right) = Uint_1
+      then
+         Rewrite (N, Make_Integer_Literal (Loc, 0));
+         Analyze_And_Resolve (N, Typ);
+         return;
+      end if;
+
       --  Deal with annoying case of largest negative number remainder
       --  minus one. Gigi does not handle this case correctly, because
       --  it generates a divide instruction which may trap in this case.
@@ -4020,8 +6439,16 @@ package body Exp_Ch4 is
 
       Determine_Range (Right, ROK, Rlo, Rhi);
       Determine_Range (Left, LOK, Llo, Lhi);
-      LLB := Expr_Value (Type_Low_Bound (Base_Type (Etype (Left))));
-      Typ := Etype (N);
+
+      --  The operand type may be private (e.g. in the expansion of an
+      --  an intrinsic operation) so we must use the underlying type to
+      --  get the bounds, and convert the literals explicitly.
+
+      LLB :=
+        Expr_Value
+          (Type_Low_Bound (Base_Type (Underlying_Type (Etype (Left)))));
+
+      --  Now perform the test, generating code only if needed
 
       if ((not ROK) or else (Rlo <= (-1) and then (-1) <= Rhi))
         and then
@@ -4033,9 +6460,11 @@ package body Exp_Ch4 is
                Make_Op_Eq (Loc,
                  Left_Opnd => Duplicate_Subexpr (Right),
                  Right_Opnd =>
-                   Make_Integer_Literal (Loc, -1)),
+                   Unchecked_Convert_To (Typ,
+                     Make_Integer_Literal (Loc, -1))),
 
-               Make_Integer_Literal (Loc, Uint_0),
+               Unchecked_Convert_To (Typ,
+                 Make_Integer_Literal (Loc, Uint_0)),
 
                Relocate_Node (N))));
 
@@ -4165,10 +6594,11 @@ package body Exp_Ch4 is
          Adjust_Condition (Left);
          Adjust_Condition (Right);
          Set_Etype (N, Standard_Boolean);
+      end if;
 
       --  Check for cases of left argument is True or False
 
-      elsif Nkind (Left) = N_Identifier then
+      if Nkind (Left) = N_Identifier then
 
          --  If left argument is False, change (False or else Right) to Right.
          --  Any actions associated with Right will be executed unconditionally
@@ -4256,6 +6686,16 @@ package body Exp_Ch4 is
       Target_Type : constant Entity_Id := Entity (Subtype_Mark (N));
 
    begin
+      --  Do validity check if validity checking operands
+
+      if Validity_Checks_On
+        and then Validity_Check_Operands
+      then
+         Ensure_Valid (Operand);
+      end if;
+
+      --  Apply possible constraint check
+
       Apply_Constraint_Check (Operand, Target_Type, No_Sliding => True);
    end Expand_N_Qualified_Expression;
 
@@ -4270,26 +6710,49 @@ package body Exp_Ch4 is
       Loc   : constant Source_Ptr := Sloc (N);
       Par   : constant Node_Id    := Parent (N);
       P     : constant Node_Id    := Prefix (N);
+      Ptyp  : Entity_Id           := Underlying_Type (Etype (P));
       Disc  : Entity_Id;
-      Ptyp  : Entity_Id := Underlying_Type (Etype (P));
       New_N : Node_Id;
+      Dcon  : Elmt_Id;
 
       function In_Left_Hand_Side (Comp : Node_Id) return Boolean;
       --  Gigi needs a temporary for prefixes that depend on a discriminant,
       --  unless the context of an assignment can provide size information.
+      --  Don't we have a general routine that does this???
+
+      -----------------------
+      -- In_Left_Hand_Side --
+      -----------------------
 
       function In_Left_Hand_Side (Comp : Node_Id) return Boolean is
       begin
-         return
-             (Nkind (Parent (Comp)) = N_Assignment_Statement
-               and then Comp = Name (Parent (Comp)))
-           or else
-             (Present (Parent (Comp))
-                and then Nkind (Parent (Comp)) in N_Subexpr
-                and then In_Left_Hand_Side (Parent (Comp)));
+         return (Nkind (Parent (Comp)) = N_Assignment_Statement
+                   and then Comp = Name (Parent (Comp)))
+           or else (Present (Parent (Comp))
+                      and then Nkind (Parent (Comp)) in N_Subexpr
+                      and then In_Left_Hand_Side (Parent (Comp)));
       end In_Left_Hand_Side;
 
+   --  Start of processing for Expand_N_Selected_Component
+
    begin
+      --  Insert explicit dereference if required
+
+      if Is_Access_Type (Ptyp) then
+         Insert_Explicit_Dereference (P);
+         Analyze_And_Resolve (P, Designated_Type (Ptyp));
+
+         if Ekind (Etype (P)) = E_Private_Subtype
+           and then Is_For_Access_Subtype (Etype (P))
+         then
+            Set_Etype (P, Base_Type (Etype (P)));
+         end if;
+
+         Ptyp := Etype (P);
+      end if;
+
+      --  Deal with discriminant check required
+
       if Do_Discriminant_Check (N) then
 
          --  Present the discrminant checking function to the backend,
@@ -4298,21 +6761,18 @@ package body Exp_Ch4 is
          Add_Inlined_Body
            (Discriminant_Checking_Func
              (Original_Record_Component (Entity (Selector_Name (N)))));
-      end if;
 
-      --  Insert explicit dereference call for the checked storage pool case
+         --  Now reset the flag and generate the call
 
-      if Is_Access_Type (Ptyp) then
-         Insert_Dereference_Action (P);
-         return;
+         Set_Do_Discriminant_Check (N, False);
+         Generate_Discriminant_Check (N);
       end if;
 
-   --  Gigi cannot handle unchecked conversions that are the prefix of
-   --  a selected component with discriminants. This must be checked
-   --  during expansion, because during analysis the type of the selector
-   --  is not known at the point the prefix is analyzed. If the conversion
-   --  is the target of an assignment, we cannot force the evaluation, of
-   --  course.
+      --  Gigi cannot handle unchecked conversions that are the prefix of a
+      --  selected component with discriminants. This must be checked during
+      --  expansion, because during analysis the type of the selector is not
+      --  known at the point the prefix is analyzed. If the conversion is the
+      --  target of an assignment, then we cannot force the evaluation.
 
       if Nkind (Prefix (N)) = N_Unchecked_Type_Conversion
         and then Has_Discriminants (Etype (N))
@@ -4326,64 +6786,120 @@ package body Exp_Ch4 is
       if Ekind (Entity (Selector_Name (N))) = E_Discriminant then
 
          --  If the selector is a discriminant of a constrained record type,
-         --  rewrite the expression with the actual value of the discriminant.
-         --  Don't do this on the left hand of an assignment statement (this
-         --  happens in generated code, and means we really want to set it!)
-         --  We also only do this optimization for discrete types, and not
-         --  for access types (access discriminants get us into trouble!)
-         --  We also do not expand the prefix of an attribute or the
-         --  operand of an object renaming declaration.
+         --  we may be able to rewrite the expression with the actual value
+         --  of the discriminant, a useful optimization in some cases.
 
          if Is_Record_Type (Ptyp)
            and then Has_Discriminants (Ptyp)
            and then Is_Constrained (Ptyp)
-           and then Is_Discrete_Type (Etype (N))
-           and then (Nkind (Par) /= N_Assignment_Statement
-                       or else Name (Par) /= N)
-           and then (Nkind (Par) /= N_Attribute_Reference
-                       or else Prefix (Par) /= N)
-           and then not Is_Renamed_Object (N)
          then
-            declare
-               D : Entity_Id;
-               E : Elmt_Id;
+            --  Do this optimization for discrete types only, and not for
+            --  access types (access discriminants get us into trouble!)
 
-            begin
-               D := First_Discriminant (Ptyp);
-               E := First_Elmt (Discriminant_Constraint (Ptyp));
+            if not Is_Discrete_Type (Etype (N)) then
+               null;
+
+            --  Don't do this on the left hand of an assignment statement.
+            --  Normally one would think that references like this would
+            --  not occur, but they do in generated code, and mean that
+            --  we really do want to assign the discriminant!
+
+            elsif Nkind (Par) = N_Assignment_Statement
+              and then Name (Par) = N
+            then
+               null;
+
+            --  Don't do this optimization for the prefix of an attribute
+            --  or the operand of an object renaming declaration since these
+            --  are contexts where we do not want the value anyway.
+
+            elsif (Nkind (Par) = N_Attribute_Reference
+                     and then Prefix (Par) = N)
+              or else Is_Renamed_Object (N)
+            then
+               null;
+
+            --  Don't do this optimization if we are within the code for a
+            --  discriminant check, since the whole point of such a check may
+            --  be to verify the condition on which the code below depends!
+
+            elsif Is_In_Discriminant_Check (N) then
+               null;
+
+            --  Green light to see if we can do the optimization. There is
+            --  still one condition that inhibits the optimization below
+            --  but now is the time to check the particular discriminant.
+
+            else
+               --  Loop through discriminants to find the matching
+               --  discriminant constraint to see if we can copy it.
+
+               Disc := First_Discriminant (Ptyp);
+               Dcon := First_Elmt (Discriminant_Constraint (Ptyp));
+               Discr_Loop : while Present (Dcon) loop
+
+                  --  Check if this is the matching discriminant
+
+                  if Disc = Entity (Selector_Name (N)) then
 
-               while Present (E) loop
-                  if D = Entity (Selector_Name (N)) then
+                     --  Here we have the matching discriminant. Check for
+                     --  the case of a discriminant of a component that is
+                     --  constrained by an outer discriminant, which cannot
+                     --  be optimized away.
+
+                     if
+                       Denotes_Discriminant
+                        (Node (Dcon), Check_Concurrent => True)
+                     then
+                        exit Discr_Loop;
 
                      --  In the context of a case statement, the expression
                      --  may have the base type of the discriminant, and we
                      --  need to preserve the constraint to avoid spurious
                      --  errors on missing cases.
 
-                     if Nkind (Parent (N)) = N_Case_Statement
-                       and then Etype (Node (E)) /= Etype (D)
+                     elsif Nkind (Parent (N)) = N_Case_Statement
+                       and then Etype (Node (Dcon)) /= Etype (Disc)
                      then
                         Rewrite (N,
                           Make_Qualified_Expression (Loc,
-                            Subtype_Mark => New_Occurrence_Of (Etype (D), Loc),
-                            Expression   => New_Copy (Node (E))));
-                        Analyze (N);
+                            Subtype_Mark =>
+                              New_Occurrence_Of (Etype (Disc), Loc),
+                            Expression   =>
+                              New_Copy_Tree (Node (Dcon))));
+                        Analyze_And_Resolve (N, Etype (Disc));
+
+                        --  In case that comes out as a static expression,
+                        --  reset it (a selected component is never static).
+
+                        Set_Is_Static_Expression (N, False);
+                        return;
+
+                     --  Otherwise we can just copy the constraint, but the
+                     --  result is certainly not static! In some cases the
+                     --  discriminant constraint has been analyzed in the
+                     --  context of the original subtype indication, but for
+                     --  itypes the constraint might not have been analyzed
+                     --  yet, and this must be done now.
+
                      else
-                        Rewrite (N, New_Copy (Node (E)));
+                        Rewrite (N, New_Copy_Tree (Node (Dcon)));
+                        Analyze_And_Resolve (N);
+                        Set_Is_Static_Expression (N, False);
+                        return;
                      end if;
-
-                     Set_Is_Static_Expression (N, False);
-                     return;
                   end if;
 
-                  Next_Elmt (E);
-                  Next_Discriminant (D);
-               end loop;
+                  Next_Elmt (Dcon);
+                  Next_Discriminant (Disc);
+               end loop Discr_Loop;
 
-               --  Note: the above loop should always terminate, but if
-               --  it does not, we just missed an optimization due to
-               --  some glitch (perhaps a previous error), so ignore!
-            end;
+               --  Note: the above loop should always find a matching
+               --  discriminant, but if it does not, we just missed an
+               --  optimization due to some glitch (perhaps a previous
+               --  error), so ignore.
+
+            end if;
          end if;
 
          --  The only remaining processing is in the case of a discriminant of
@@ -4414,7 +6930,6 @@ package body Exp_Ch4 is
          Rewrite (N, New_N);
          Analyze (N);
       end if;
-
    end Expand_N_Selected_Component;
 
    --------------------
@@ -4426,36 +6941,98 @@ package body Exp_Ch4 is
       Typ  : constant Entity_Id  := Etype (N);
       Pfx  : constant Node_Id    := Prefix (N);
       Ptp  : Entity_Id           := Etype (Pfx);
-      Ent  : Entity_Id;
-      Decl : Node_Id;
 
-   begin
-      --  Special handling for access types
+      function Is_Procedure_Actual (N : Node_Id) return Boolean;
+      --  Check whether the argument is an actual for a procedure call,
+      --  in which case the expansion of a bit-packed slice is deferred
+      --  until the call itself is expanded. The reason this is required
+      --  is that we might have an IN OUT or OUT parameter, and the copy out
+      --  is essential, and that copy out would be missed if we created a
+      --  temporary here in Expand_N_Slice. Note that we don't bother
+      --  to test specifically for an IN OUT or OUT mode parameter, since it
+      --  is a bit tricky to do, and it is harmless to defer expansion
+      --  in the IN case, since the call processing will still generate the
+      --  appropriate copy in operation, which will take care of the slice.
+
+      procedure Make_Temporary;
+      --  Create a named variable for the value of the slice, in
+      --  cases where the back-end cannot handle it properly, e.g.
+      --  when packed types or unaligned slices are involved.
 
-      if Is_Access_Type (Ptp) then
+      -------------------------
+      -- Is_Procedure_Actual --
+      -------------------------
 
-         --  Check for explicit dereference required for checked pool
+      function Is_Procedure_Actual (N : Node_Id) return Boolean is
+         Par : Node_Id := Parent (N);
+
+      begin
+         loop
+            --  If our parent is a procedure call we can return
 
-         Insert_Dereference_Action (Pfx);
+            if Nkind (Par) = N_Procedure_Call_Statement then
+               return True;
 
-         --  If we have an access to a packed array type, then put in an
-         --  explicit dereference. We do this in case the slice must be
-         --  expanded, and we want to make sure we get an access check.
+            --  If our parent is a type conversion, keep climbing the
+            --  tree, since a type conversion can be a procedure actual.
+            --  Also keep climbing if parameter association or a qualified
+            --  expression, since these are additional cases that do can
+            --  appear on procedure actuals.
 
-         Ptp := Designated_Type (Ptp);
+            elsif Nkind_In (Par, N_Type_Conversion,
+                                 N_Parameter_Association,
+                                 N_Qualified_Expression)
+            then
+               Par := Parent (Par);
 
-         if Is_Array_Type (Ptp) and then Is_Packed (Ptp) then
-            Rewrite (Pfx,
-              Make_Explicit_Dereference (Sloc (N),
-                Prefix => Relocate_Node (Pfx)));
+               --  Any other case is not what we are looking for
 
-            Analyze_And_Resolve (Pfx, Ptp);
+            else
+               return False;
+            end if;
+         end loop;
+      end Is_Procedure_Actual;
 
-            --  The prefix will now carry the Access_Check flag for the back
-            --  end, remove it from slice itself.
+      --------------------
+      -- Make_Temporary --
+      --------------------
 
-            Set_Do_Access_Check (N, False);
-         end if;
+      procedure Make_Temporary is
+         Decl : Node_Id;
+         Ent  : constant Entity_Id :=
+                  Make_Defining_Identifier (Loc, New_Internal_Name ('T'));
+      begin
+         Decl :=
+           Make_Object_Declaration (Loc,
+             Defining_Identifier => Ent,
+             Object_Definition   => New_Occurrence_Of (Typ, Loc));
+
+         Set_No_Initialization (Decl);
+
+         Insert_Actions (N, New_List (
+           Decl,
+           Make_Assignment_Statement (Loc,
+             Name => New_Occurrence_Of (Ent, Loc),
+             Expression => Relocate_Node (N))));
+
+         Rewrite (N, New_Occurrence_Of (Ent, Loc));
+         Analyze_And_Resolve (N, Typ);
+      end Make_Temporary;
+
+   --  Start of processing for Expand_N_Slice
+
+   begin
+      --  Special handling for access types
+
+      if Is_Access_Type (Ptp) then
+
+         Ptp := Designated_Type (Ptp);
+
+         Rewrite (Pfx,
+           Make_Explicit_Dereference (Sloc (N),
+            Prefix => Relocate_Node (Pfx)));
+
+         Analyze_And_Resolve (Pfx, Ptp);
       end if;
 
       --  Range checks are potentially also needed for cases involving
@@ -4466,6 +7043,20 @@ package body Exp_Ch4 is
         and then (not Is_Entity_Name (Pfx)
                    or else not Index_Checks_Suppressed (Entity (Pfx)))
         and then Nkind (Discrete_Range (N)) /= N_Subtype_Indication
+
+         --  Do not enable range check to nodes associated with the frontend
+         --  expansion of the dispatch table. We first check if Ada.Tags is
+         --  already loaded to avoid the addition of an undesired dependence
+         --  on such run-time unit.
+
+        and then
+          (VM_Target /= No_VM
+            or else not
+             (RTU_Loaded (Ada_Tags)
+               and then Nkind (Prefix (N)) = N_Selected_Component
+               and then Present (Entity (Selector_Name (Prefix (N))))
+               and then Entity (Selector_Name (Prefix (N))) =
+                                  RTE_Record_Component (RE_Prims_Ptr)))
       then
          Enable_Range_Check (Discrete_Range (N));
       end if;
@@ -4477,7 +7068,7 @@ package body Exp_Ch4 is
       --       situation correctly in the assignment statement expansion).
 
       --    2. Prefix of indexed component (the slide is optimized away
-      --       in this case, see the start of Expand_N_Slice.
+      --       in this case, see the start of Expand_N_Slice.)
 
       --    3. Object renaming declaration, since we want the name of
       --       the slice, not the value.
@@ -4490,33 +7081,36 @@ package body Exp_Ch4 is
       --       is caught elsewhere, and the expansion would intefere
       --       with generating the error message).
 
-      if Is_Packed (Typ)
-        and then Nkind (Parent (N)) /= N_Assignment_Statement
-        and then Nkind (Parent (N)) /= N_Indexed_Component
-        and then not Is_Renamed_Object (N)
-        and then Nkind (Parent (N)) /= N_Procedure_Call_Statement
-        and then (Nkind (Parent (N)) /= N_Attribute_Reference
-                    or else
-                  Attribute_Name (Parent (N)) /= Name_Address)
-      then
-         Ent :=
-           Make_Defining_Identifier (Loc, New_Internal_Name ('T'));
+      if not Is_Packed (Typ) then
 
-         Decl :=
-           Make_Object_Declaration (Loc,
-             Defining_Identifier => Ent,
-             Object_Definition   => New_Occurrence_Of (Typ, Loc));
+         --  Apply transformation for actuals of a function call,
+         --  where Expand_Actuals is not used.
 
-         Set_No_Initialization (Decl);
+         if Nkind (Parent (N)) = N_Function_Call
+           and then Is_Possibly_Unaligned_Slice (N)
+         then
+            Make_Temporary;
+         end if;
 
-         Insert_Actions (N, New_List (
-           Decl,
-           Make_Assignment_Statement (Loc,
-             Name => New_Occurrence_Of (Ent, Loc),
-             Expression => Relocate_Node (N))));
+      elsif Nkind (Parent (N)) = N_Assignment_Statement
+        or else (Nkind (Parent (Parent (N))) = N_Assignment_Statement
+                   and then Parent (N) = Name (Parent (Parent (N))))
+      then
+         return;
 
-         Rewrite (N, New_Occurrence_Of (Ent, Loc));
-         Analyze_And_Resolve (N, Typ);
+      elsif Nkind (Parent (N)) = N_Indexed_Component
+        or else Is_Renamed_Object (N)
+        or else Is_Procedure_Actual (N)
+      then
+         return;
+
+      elsif Nkind (Parent (N)) = N_Attribute_Reference
+        and then Attribute_Name (Parent (N)) = Name_Address
+      then
+         return;
+
+      else
+         Make_Temporary;
       end if;
    end Expand_N_Slice;
 
@@ -4554,7 +7148,7 @@ package body Exp_Ch4 is
          Cons : List_Id;
 
       begin
-         --  Nothing to do if no change of representation
+         --  Nothing else to do if no change of representation
 
          if Same_Representation (Operand_Type, Target_Type) then
             return;
@@ -4580,11 +7174,16 @@ package body Exp_Ch4 is
             if not Is_Constrained (Target_Type) then
                if Has_Discriminants (Operand_Type) then
                   Disc := First_Discriminant (Operand_Type);
+
+                  if Disc /= First_Stored_Discriminant (Operand_Type) then
+                     Disc := First_Stored_Discriminant (Operand_Type);
+                  end if;
+
                   Cons := New_List;
                   while Present (Disc) loop
                      Append_To (Cons,
                        Make_Selected_Component (Loc,
-                         Prefix => Duplicate_Subexpr (Operand),
+                         Prefix => Duplicate_Subexpr_Move_Checks (Operand),
                          Selector_Name =>
                            Make_Identifier (Loc, Chars (Disc))));
                      Next_Discriminant (Disc);
@@ -4605,7 +7204,7 @@ package body Exp_Ch4 is
                            Unchecked_Convert_To (Etype (N_Ix),
                              Make_Attribute_Reference (Loc,
                                Prefix =>
-                                 Duplicate_Subexpr
+                                 Duplicate_Subexpr_No_Checks
                                    (Operand, Name_Req => True),
                                Attribute_Name => Name_First,
                                Expressions    => New_List (
@@ -4615,7 +7214,7 @@ package body Exp_Ch4 is
                            Unchecked_Convert_To (Etype (N_Ix),
                              Make_Attribute_Reference (Loc,
                                Prefix =>
-                                 Duplicate_Subexpr
+                                 Duplicate_Subexpr_No_Checks
                                    (Operand, Name_Req => True),
                                Attribute_Name => Name_Last,
                                Expressions    => New_List (
@@ -4678,10 +7277,16 @@ package body Exp_Ch4 is
       --     [constraint_error when Tnn < typ'First or else Tnn > typ'Last]
       --     Tnn
 
+      --  This is necessary when there is a conversion of integer to float
+      --  or to fixed-point to ensure that the correct checks are made. It
+      --  is not necessary for float to float where it is enough to simply
+      --  set the Do_Range_Check flag.
+
       procedure Real_Range_Check is
          Btyp : constant Entity_Id := Base_Type (Target_Type);
          Lo   : constant Node_Id   := Type_Low_Bound  (Target_Type);
          Hi   : constant Node_Id   := Type_High_Bound (Target_Type);
+         Xtyp : constant Entity_Id := Etype (Operand);
          Conv : Node_Id;
          Tnn  : Entity_Id;
 
@@ -4706,25 +7311,77 @@ package body Exp_Ch4 is
          --  Nothing to do if expression is an entity on which checks
          --  have been suppressed.
 
-         if Is_Entity_Name (Expression (N))
-           and then Range_Checks_Suppressed (Entity (Expression (N)))
+         if Is_Entity_Name (Operand)
+           and then Range_Checks_Suppressed (Entity (Operand))
+         then
+            return;
+         end if;
+
+         --  Nothing to do if bounds are all static and we can tell that
+         --  the expression is within the bounds of the target. Note that
+         --  if the operand is of an unconstrained floating-point type,
+         --  then we do not trust it to be in range (might be infinite)
+
+         declare
+            S_Lo : constant Node_Id := Type_Low_Bound (Xtyp);
+            S_Hi : constant Node_Id := Type_High_Bound (Xtyp);
+
+         begin
+            if (not Is_Floating_Point_Type (Xtyp)
+                 or else Is_Constrained (Xtyp))
+              and then Compile_Time_Known_Value (S_Lo)
+              and then Compile_Time_Known_Value (S_Hi)
+              and then Compile_Time_Known_Value (Hi)
+              and then Compile_Time_Known_Value (Lo)
+            then
+               declare
+                  D_Lov : constant Ureal := Expr_Value_R (Lo);
+                  D_Hiv : constant Ureal := Expr_Value_R (Hi);
+                  S_Lov : Ureal;
+                  S_Hiv : Ureal;
+
+               begin
+                  if Is_Real_Type (Xtyp) then
+                     S_Lov := Expr_Value_R (S_Lo);
+                     S_Hiv := Expr_Value_R (S_Hi);
+                  else
+                     S_Lov := UR_From_Uint (Expr_Value (S_Lo));
+                     S_Hiv := UR_From_Uint (Expr_Value (S_Hi));
+                  end if;
+
+                  if D_Hiv > D_Lov
+                    and then S_Lov >= D_Lov
+                    and then S_Hiv <= D_Hiv
+                  then
+                     Set_Do_Range_Check (Operand, False);
+                     return;
+                  end if;
+               end;
+            end if;
+         end;
+
+         --  For float to float conversions, we are done
+
+         if Is_Floating_Point_Type (Xtyp)
+              and then
+            Is_Floating_Point_Type (Btyp)
          then
             return;
          end if;
 
-         --  Here we rewrite the conversion as described above
+         --  Otherwise rewrite the conversion as described above
 
          Conv := Relocate_Node (N);
          Rewrite
            (Subtype_Mark (Conv), New_Occurrence_Of (Btyp, Loc));
          Set_Etype (Conv, Btyp);
 
-         --  Skip overflow check for integer to float conversions,
-         --  since it is not needed, and in any case gigi generates
-         --  incorrect code for such overflow checks ???
+         --  Enable overflow except for case of integer to float conversions,
+         --  where it is never required, since we can never have overflow in
+         --  this case.
 
-         if not Is_Integer_Type (Etype (Expression (N))) then
-            Set_Do_Overflow_Check (Conv, True);
+         if not Is_Integer_Type (Etype (Operand)) then
+            Enable_Overflow_Check (Conv);
          end if;
 
          Tnn :=
@@ -4738,25 +7395,26 @@ package body Exp_Ch4 is
              Expression => Conv),
 
            Make_Raise_Constraint_Error (Loc,
-            Condition =>
-             Make_Or_Else (Loc,
-               Left_Opnd =>
-                 Make_Op_Lt (Loc,
-                   Left_Opnd  => New_Occurrence_Of (Tnn, Loc),
-                   Right_Opnd =>
-                     Make_Attribute_Reference (Loc,
-                       Attribute_Name => Name_First,
-                       Prefix =>
-                         New_Occurrence_Of (Target_Type, Loc))),
+             Condition =>
+              Make_Or_Else (Loc,
+                Left_Opnd =>
+                  Make_Op_Lt (Loc,
+                    Left_Opnd  => New_Occurrence_Of (Tnn, Loc),
+                    Right_Opnd =>
+                      Make_Attribute_Reference (Loc,
+                        Attribute_Name => Name_First,
+                        Prefix =>
+                          New_Occurrence_Of (Target_Type, Loc))),
 
-               Right_Opnd =>
-                 Make_Op_Gt (Loc,
-                   Left_Opnd  => New_Occurrence_Of (Tnn, Loc),
-                   Right_Opnd =>
-                     Make_Attribute_Reference (Loc,
-                       Attribute_Name => Name_Last,
-                       Prefix =>
-                         New_Occurrence_Of (Target_Type, Loc)))))));
+                Right_Opnd =>
+                  Make_Op_Gt (Loc,
+                    Left_Opnd  => New_Occurrence_Of (Tnn, Loc),
+                    Right_Opnd =>
+                      Make_Attribute_Reference (Loc,
+                        Attribute_Name => Name_Last,
+                        Prefix =>
+                          New_Occurrence_Of (Target_Type, Loc)))),
+             Reason => CE_Range_Check_Failed)));
 
          Rewrite (N, New_Occurrence_Of (Tnn, Loc));
          Analyze_And_Resolve (N, Btyp);
@@ -4769,14 +7427,7 @@ package body Exp_Ch4 is
       --  so remove the conversion completely, it is useless.
 
       if Operand_Type = Target_Type then
-         Rewrite (N, Relocate_Node (Expression (N)));
-         return;
-      end if;
-
-      --  Deal with Vax floating-point cases
-
-      if Vax_Float (Operand_Type) or else Vax_Float (Target_Type) then
-         Expand_Vax_Conversion (N);
+         Rewrite (N, Relocate_Node (Operand));
          return;
       end if;
 
@@ -4793,6 +7444,14 @@ package body Exp_Ch4 is
 
       --  Here if we may need to expand conversion
 
+      --  Do validity check if validity checking operands
+
+      if Validity_Checks_On
+        and then Validity_Check_Operands
+      then
+         Ensure_Valid (Operand);
+      end if;
+
       --  Special case of converting from non-standard boolean type
 
       if Is_Boolean_Type (Operand_Type)
@@ -4807,13 +7466,23 @@ package body Exp_Ch4 is
 
       if Is_Access_Type (Target_Type) then
 
-         --  Apply an accessibility check if the operand is an
-         --  access parameter. Note that other checks may still
-         --  need to be applied below (such as tagged type checks).
+         --  Apply an accessibility check when the conversion operand is an
+         --  access parameter (or a renaming thereof), unless conversion was
+         --  expanded from an unchecked or unrestricted access attribute. Note
+         --  that other checks may still need to be applied below (such as
+         --  tagged type checks).
 
          if Is_Entity_Name (Operand)
-           and then Ekind (Entity (Operand)) in Formal_Kind
+           and then
+             (Is_Formal (Entity (Operand))
+               or else
+                 (Present (Renamed_Object (Entity (Operand)))
+                   and then Is_Entity_Name (Renamed_Object (Entity (Operand)))
+                   and then Is_Formal
+                              (Entity (Renamed_Object (Entity (Operand))))))
            and then Ekind (Etype (Operand)) = E_Anonymous_Access_Type
+           and then (Nkind (Original_Node (N)) /= N_Attribute_Reference
+                      or else Attribute_Name (Original_Node (N)) = Name_Access)
          then
             Apply_Accessibility_Check (Operand, Target_Type);
 
@@ -4826,10 +7495,12 @@ package body Exp_Ch4 is
          --  cases.
 
          elsif In_Instance_Body
-           and then Type_Access_Level (Operand_Type)
-                  > Type_Access_Level (Target_Type)
+           and then Type_Access_Level (Operand_Type) >
+                    Type_Access_Level (Target_Type)
          then
-            Rewrite (N, Make_Raise_Program_Error (Sloc (N)));
+            Rewrite (N,
+              Make_Raise_Program_Error (Sloc (N),
+                Reason => PE_Accessibility_Check_Failed));
             Set_Etype (N, Target_Type);
 
          --  When the operand is a selected access discriminant
@@ -4845,7 +7516,9 @@ package body Exp_Ch4 is
            and then Object_Access_Level (Operand) >
                       Type_Access_Level (Target_Type)
          then
-            Rewrite (N, Make_Raise_Program_Error (Sloc (N)));
+            Rewrite (N,
+              Make_Raise_Program_Error (Sloc (N),
+                Reason => PE_Accessibility_Check_Failed));
             Set_Etype (N, Target_Type);
          end if;
       end if;
@@ -4880,7 +7553,7 @@ package body Exp_Ch4 is
             return;
          end if;
 
-         --  Oherwise, proceed with processing tagged conversion
+         --  Otherwise, proceed with processing tagged conversion
 
          declare
             Actual_Operand_Type : Entity_Id;
@@ -4898,6 +7571,13 @@ package body Exp_Ch4 is
                Actual_Target_Type  := Target_Type;
             end if;
 
+            --  Ada 2005 (AI-251): Handle interface type conversion
+
+            if Is_Interface (Actual_Operand_Type) then
+               Expand_Interface_Conversion (N, Is_Static => False);
+               return;
+            end if;
+
             if Is_Class_Wide_Type (Actual_Operand_Type)
               and then Root_Type (Actual_Operand_Type) /=  Actual_Target_Type
               and then Is_Ancestor
@@ -4915,31 +7595,41 @@ package body Exp_Ch4 is
                      Make_And_Then (Loc,
                        Left_Opnd =>
                          Make_Op_Ne (Loc,
-                           Left_Opnd  => Duplicate_Subexpr (Operand),
+                           Left_Opnd  => Duplicate_Subexpr_No_Checks (Operand),
                            Right_Opnd => Make_Null (Loc)),
 
                        Right_Opnd =>
                          Make_Not_In (Loc,
                            Left_Opnd  =>
                              Make_Explicit_Dereference (Loc,
-                               Prefix => Duplicate_Subexpr (Operand)),
+                               Prefix =>
+                                 Duplicate_Subexpr_No_Checks (Operand)),
                            Right_Opnd =>
                              New_Reference_To (Actual_Target_Type, Loc)));
 
                else
                   Cond :=
                     Make_Not_In (Loc,
-                      Left_Opnd  => Duplicate_Subexpr (Operand),
+                      Left_Opnd  => Duplicate_Subexpr_No_Checks (Operand),
                       Right_Opnd =>
                         New_Reference_To (Actual_Target_Type, Loc));
                end if;
 
                Insert_Action (N,
                  Make_Raise_Constraint_Error (Loc,
-                   Condition => Cond));
+                   Condition => Cond,
+                   Reason    => CE_Tag_Check_Failed));
 
-               Change_Conversion_To_Unchecked (N);
-               Analyze_And_Resolve (N, Target_Type);
+               declare
+                  Conv : Node_Id;
+               begin
+                  Conv :=
+                    Make_Unchecked_Type_Conversion (Loc,
+                      Subtype_Mark => New_Occurrence_Of (Target_Type, Loc),
+                      Expression => Relocate_Node (Expression (N)));
+                  Rewrite (N, Conv);
+                  Analyze_And_Resolve (N, Target_Type);
+               end;
             end if;
          end;
 
@@ -5026,47 +7716,31 @@ package body Exp_Ch4 is
       elsif Is_Floating_Point_Type (Operand_Type)
         and then
           (Is_Integer_Type (Target_Type)
-            or else
-          (Is_Fixed_Point_Type (Target_Type) and then Conversion_OK (N)))
-      then
-         --  Special processing required if the conversion is the expression
-         --  of a Truncation attribute reference. In this case we replace:
-
-         --     ityp (ftyp'Truncation (x))
-
-         --  by
-
-         --     ityp (x)
-
-         --  with the Float_Truncate flag set. This is clearly more efficient.
-
-         if Nkind (Operand) = N_Attribute_Reference
-           and then Attribute_Name (Operand) = Name_Truncation
-         then
-            Rewrite (Operand,
-              Relocate_Node (First (Expressions (Operand))));
-            Set_Float_Truncate (N, True);
-         end if;
-
+            or else
+          (Is_Fixed_Point_Type (Target_Type) and then Conversion_OK (N)))
+      then
          --  One more check here, gcc is still not able to do conversions of
          --  this type with proper overflow checking, and so gigi is doing an
          --  approximation of what is required by doing floating-point compares
          --  with the end-point. But that can lose precision in some cases, and
-         --  give a wrong result. Converting the operand to Long_Long_Float is
+         --  give a wrong result. Converting the operand to Universal_Real is
          --  helpful, but still does not catch all cases with 64-bit integers
-         --  on targets with only 64-bit floats ???
+         --  on targets with only 64-bit floats
+
+         --  The above comment seems obsoleted by Apply_Float_Conversion_Check
+         --  Can this code be removed ???
 
-         if Do_Range_Check (Expression (N)) then
-            Rewrite (Expression (N),
+         if Do_Range_Check (Operand) then
+            Rewrite (Operand,
               Make_Type_Conversion (Loc,
                 Subtype_Mark =>
-                  New_Occurrence_Of (Standard_Long_Long_Float, Loc),
+                  New_Occurrence_Of (Universal_Real, Loc),
                 Expression =>
-                  Relocate_Node (Expression (N))));
+                  Relocate_Node (Operand)));
 
-            Set_Etype (Expression (N), Standard_Long_Long_Float);
-            Enable_Range_Check (Expression (N));
-            Set_Do_Range_Check (Expression (Expression (N)), False);
+            Set_Etype (Operand, Universal_Real);
+            Enable_Range_Check (Operand);
+            Set_Do_Range_Check (Expression (Operand), False);
          end if;
 
       --  Case of array conversions
@@ -5101,7 +7775,33 @@ package body Exp_Ch4 is
       --  assignment processing.
 
       elsif Is_Record_Type (Target_Type) then
-         Handle_Changed_Representation;
+
+         --  Ada 2005 (AI-216): Program_Error is raised when converting from
+         --  a derived Unchecked_Union type to an unconstrained non-Unchecked_
+         --  Union type if the operand lacks inferable discriminants.
+
+         if Is_Derived_Type (Operand_Type)
+           and then Is_Unchecked_Union (Base_Type (Operand_Type))
+           and then not Is_Constrained (Target_Type)
+           and then not Is_Unchecked_Union (Base_Type (Target_Type))
+           and then not Has_Inferable_Discriminants (Operand)
+         then
+            --  To prevent Gigi from generating illegal code, we make a
+            --  Program_Error node, but we give it the target type of the
+            --  conversion.
+
+            declare
+               PE : constant Node_Id := Make_Raise_Program_Error (Loc,
+                      Reason => PE_Unchecked_Union_Restriction);
+
+            begin
+               Set_Etype (PE, Target_Type);
+               Rewrite (N, PE);
+
+            end;
+         else
+            Handle_Changed_Representation;
+         end if;
 
       --  Case of conversions of enumeration types
 
@@ -5131,11 +7831,6 @@ package body Exp_Ch4 is
 
       elsif Is_Floating_Point_Type (Target_Type) then
          Real_Range_Check;
-
-      --  The remaining cases require no front end processing
-
-      else
-         null;
       end if;
 
       --  At this stage, either the conversion node has been transformed
@@ -5150,8 +7845,76 @@ package body Exp_Ch4 is
       --    only if Conversion_OK is set, i.e. if the fixed-point values
       --    are to be treated as integers.
 
-      --  No other conversions should be passed to Gigi.
+      --  No other conversions should be passed to Gigi
+
+      --  Check: are these rules stated in sinfo??? if so, why restate here???
+
+      --  The only remaining step is to generate a range check if we still
+      --  have a type conversion at this stage and Do_Range_Check is set.
+      --  For now we do this only for conversions of discrete types.
+
+      if Nkind (N) = N_Type_Conversion
+        and then Is_Discrete_Type (Etype (N))
+      then
+         declare
+            Expr : constant Node_Id := Expression (N);
+            Ftyp : Entity_Id;
+            Ityp : Entity_Id;
+
+         begin
+            if Do_Range_Check (Expr)
+              and then Is_Discrete_Type (Etype (Expr))
+            then
+               Set_Do_Range_Check (Expr, False);
+
+               --  Before we do a range check, we have to deal with treating
+               --  a fixed-point operand as an integer. The way we do this
+               --  is simply to do an unchecked conversion to an appropriate
+               --  integer type large enough to hold the result.
+
+               --  This code is not active yet, because we are only dealing
+               --  with discrete types so far ???
+
+               if Nkind (Expr) in N_Has_Treat_Fixed_As_Integer
+                 and then Treat_Fixed_As_Integer (Expr)
+               then
+                  Ftyp := Base_Type (Etype (Expr));
+
+                  if Esize (Ftyp) >= Esize (Standard_Integer) then
+                     Ityp := Standard_Long_Long_Integer;
+                  else
+                     Ityp := Standard_Integer;
+                  end if;
+
+                  Rewrite (Expr, Unchecked_Convert_To (Ityp, Expr));
+               end if;
+
+               --  Reset overflow flag, since the range check will include
+               --  dealing with possible overflow, and generate the check
+               --  If Address is either source or target type, suppress
+               --  range check to avoid typing anomalies when it is a visible
+               --  integer type.
+
+               Set_Do_Overflow_Check (N, False);
+               if not Is_Descendent_Of_Address (Etype (Expr))
+                 and then not Is_Descendent_Of_Address (Target_Type)
+               then
+                  Generate_Range_Check
+                    (Expr, Target_Type, CE_Range_Check_Failed);
+               end if;
+            end if;
+         end;
+      end if;
+
+      --  Final step, if the result is a type conversion involving Vax_Float
+      --  types, then it is subject for further special processing.
 
+      if Nkind (N) = N_Type_Conversion
+        and then (Vax_Float (Operand_Type) or else Vax_Float (Target_Type))
+      then
+         Expand_Vax_Conversion (N);
+         return;
+      end if;
    end Expand_N_Type_Conversion;
 
    -----------------------------------
@@ -5194,8 +7957,14 @@ package body Exp_Ch4 is
       --  flag is set, since then the value may be outside the expected range.
       --  This happens in the Normalize_Scalars case.
 
+      --  We also skip this if either the target or operand type is biased
+      --  because in this case, the unchecked conversion is supposed to
+      --  preserve the bit pattern, not the integer value.
+
       if Is_Integer_Type (Target_Type)
+        and then not Has_Biased_Representation (Target_Type)
         and then Is_Integer_Type (Operand_Type)
+        and then not Has_Biased_Representation (Operand_Type)
         and then Compile_Time_Known_Value (Operand)
         and then not Kill_Range_Check (N)
       then
@@ -5212,7 +7981,17 @@ package body Exp_Ch4 is
                Val <= Expr_Value (Type_High_Bound (Target_Type))
             then
                Rewrite (N, Make_Integer_Literal (Sloc (N), Val));
-               Analyze_And_Resolve (N, Target_Type);
+
+               --  If Address is the target type, just set the type
+               --  to avoid a spurious type error on the literal when
+               --  Address is a visible integer type.
+
+               if Is_Descendent_Of_Address (Target_Type) then
+                  Set_Etype (N, Target_Type);
+               else
+                  Analyze_And_Resolve (N, Target_Type);
+               end if;
+
                return;
             end if;
          end;
@@ -5257,14 +8036,22 @@ package body Exp_Ch4 is
       Typ    : Entity_Id;
       Lhs    : Node_Id;
       Rhs    : Node_Id;
-      Bodies : List_Id)
-      return   Node_Id
+      Bodies : List_Id) return Node_Id
    is
       Loc : constant Source_Ptr := Sloc (Nod);
 
+      Result : Node_Id;
+      C      : Entity_Id;
+
+      First_Time : Boolean := True;
+
       function Suitable_Element (C : Entity_Id) return Entity_Id;
       --  Return the first field to compare beginning with C, skipping the
-      --  inherited components
+      --  inherited components.
+
+      ----------------------
+      -- Suitable_Element --
+      ----------------------
 
       function Suitable_Element (C : Entity_Id) return Entity_Id is
       begin
@@ -5286,41 +8073,17 @@ package body Exp_Ch4 is
          then
             return Suitable_Element (Next_Entity (C));
 
+         elsif Is_Interface (Etype (C)) then
+            return Suitable_Element (Next_Entity (C));
+
          else
             return C;
          end if;
       end Suitable_Element;
 
-      Result : Node_Id;
-      C      : Entity_Id;
-
-      First_Time : Boolean := True;
-
    --  Start of processing for Expand_Record_Equality
 
    begin
-      --  Special processing for the unchecked union case, which will occur
-      --  only in the context of tagged types and dynamic dispatching, since
-      --  other cases are handled statically. We return True, but insert a
-      --  raise Program_Error statement.
-
-      if Is_Unchecked_Union (Typ) then
-
-         --  If this is a component of an enclosing record, return the Raise
-         --  statement directly.
-
-         if No (Parent (Lhs)) then
-            Result := Make_Raise_Program_Error (Loc);
-            Set_Etype (Result, Standard_Boolean);
-            return Result;
-
-         else
-            Insert_Action (Lhs,
-              Make_Raise_Program_Error (Loc));
-            return New_Occurrence_Of (Standard_True, Loc);
-         end if;
-      end if;
-
       --  Generates the following code: (assuming that Typ has one Discr and
       --  component C2 is also a record)
 
@@ -5335,36 +8098,46 @@ package body Exp_Ch4 is
       C := Suitable_Element (First_Entity (Typ));
 
       while Present (C) loop
-
          declare
             New_Lhs : Node_Id;
             New_Rhs : Node_Id;
+            Check   : Node_Id;
 
          begin
             if First_Time then
                First_Time := False;
                New_Lhs := Lhs;
                New_Rhs := Rhs;
-
             else
                New_Lhs := New_Copy_Tree (Lhs);
                New_Rhs := New_Copy_Tree (Rhs);
             end if;
 
-            Result :=
-              Make_And_Then (Loc,
-                Left_Opnd  => Result,
-                Right_Opnd =>
-                  Expand_Composite_Equality (Nod, Etype (C),
-                    Lhs =>
-                      Make_Selected_Component (Loc,
-                        Prefix => New_Lhs,
-                        Selector_Name => New_Reference_To (C, Loc)),
-                    Rhs =>
-                      Make_Selected_Component (Loc,
-                        Prefix => New_Rhs,
-                        Selector_Name => New_Reference_To (C, Loc)),
-                    Bodies => Bodies));
+            Check :=
+              Expand_Composite_Equality (Nod, Etype (C),
+               Lhs =>
+                 Make_Selected_Component (Loc,
+                   Prefix => New_Lhs,
+                   Selector_Name => New_Reference_To (C, Loc)),
+               Rhs =>
+                 Make_Selected_Component (Loc,
+                   Prefix => New_Rhs,
+                   Selector_Name => New_Reference_To (C, Loc)),
+               Bodies => Bodies);
+
+            --  If some (sub)component is an unchecked_union, the whole
+            --  operation will raise program error.
+
+            if Nkind (Check) = N_Raise_Program_Error then
+               Result := Check;
+               Set_Etype (Result, Standard_Boolean);
+               exit;
+            else
+               Result :=
+                 Make_And_Then (Loc,
+                   Left_Opnd  => Result,
+                   Right_Opnd => Check);
+            end if;
          end;
 
          C := Suitable_Element (Next_Entity (C));
@@ -5403,6 +8176,159 @@ package body Exp_Ch4 is
       end if;
    end Fixup_Universal_Fixed_Operation;
 
+   ------------------------------
+   -- Get_Allocator_Final_List --
+   ------------------------------
+
+   function Get_Allocator_Final_List
+     (N    : Node_Id;
+      T    : Entity_Id;
+      PtrT : Entity_Id) return Entity_Id
+   is
+      Loc : constant Source_Ptr := Sloc (N);
+
+      Owner : Entity_Id := PtrT;
+      --  The entity whose finalization list must be used to attach the
+      --  allocated object.
+
+   begin
+      if Ekind (PtrT) = E_Anonymous_Access_Type then
+
+         --  If the context is an access parameter, we need to create a
+         --  non-anonymous access type in order to have a usable final list,
+         --  because there is otherwise no pool to which the allocated object
+         --  can belong. We create both the type and the finalization chain
+         --  here, because freezing an internal type does not create such a
+         --  chain. The Final_Chain that is thus created is shared by the
+         --  access parameter. The access type is tested against the result
+         --  type of the function to exclude allocators whose type is an
+         --  anonymous access result type.
+
+         if Nkind (Associated_Node_For_Itype (PtrT))
+              in N_Subprogram_Specification
+           and then
+             PtrT /=
+               Etype (Defining_Unit_Name (Associated_Node_For_Itype (PtrT)))
+         then
+            Owner := Make_Defining_Identifier (Loc, New_Internal_Name ('J'));
+            Insert_Action (N,
+              Make_Full_Type_Declaration (Loc,
+                Defining_Identifier => Owner,
+                Type_Definition =>
+                   Make_Access_To_Object_Definition (Loc,
+                     Subtype_Indication =>
+                       New_Occurrence_Of (T, Loc))));
+
+            Build_Final_List (N, Owner);
+            Set_Associated_Final_Chain (PtrT, Associated_Final_Chain (Owner));
+
+         --  Ada 2005 (AI-318-02): If the context is a return object
+         --  declaration, then the anonymous return subtype is defined to have
+         --  the same accessibility level as that of the function's result
+         --  subtype, which means that we want the scope where the function is
+         --  declared.
+
+         elsif Nkind (Associated_Node_For_Itype (PtrT)) = N_Object_Declaration
+           and then Ekind (Scope (PtrT)) = E_Return_Statement
+         then
+            Owner := Scope (Return_Applies_To (Scope (PtrT)));
+
+         --  Case of an access discriminant, or (Ada 2005), of an anonymous
+         --  access component or anonymous access function result: find the
+         --  final list associated with the scope of the type. (In the
+         --  anonymous access component kind, a list controller will have
+         --  been allocated when freezing the record type, and PtrT has an
+         --  Associated_Final_Chain attribute designating it.)
+
+         elsif No (Associated_Final_Chain (PtrT)) then
+            Owner := Scope (PtrT);
+         end if;
+      end if;
+
+      return Find_Final_List (Owner);
+   end Get_Allocator_Final_List;
+
+   ---------------------------------
+   -- Has_Inferable_Discriminants --
+   ---------------------------------
+
+   function Has_Inferable_Discriminants (N : Node_Id) return Boolean is
+
+      function Prefix_Is_Formal_Parameter (N : Node_Id) return Boolean;
+      --  Determines whether the left-most prefix of a selected component is a
+      --  formal parameter in a subprogram. Assumes N is a selected component.
+
+      --------------------------------
+      -- Prefix_Is_Formal_Parameter --
+      --------------------------------
+
+      function Prefix_Is_Formal_Parameter (N : Node_Id) return Boolean is
+         Sel_Comp : Node_Id := N;
+
+      begin
+         --  Move to the left-most prefix by climbing up the tree
+
+         while Present (Parent (Sel_Comp))
+           and then Nkind (Parent (Sel_Comp)) = N_Selected_Component
+         loop
+            Sel_Comp := Parent (Sel_Comp);
+         end loop;
+
+         return Ekind (Entity (Prefix (Sel_Comp))) in Formal_Kind;
+      end Prefix_Is_Formal_Parameter;
+
+   --  Start of processing for Has_Inferable_Discriminants
+
+   begin
+      --  For identifiers and indexed components, it is sufficent to have a
+      --  constrained Unchecked_Union nominal subtype.
+
+      if Nkind_In (N, N_Identifier, N_Indexed_Component) then
+         return Is_Unchecked_Union (Base_Type (Etype (N)))
+                  and then
+                Is_Constrained (Etype (N));
+
+      --  For selected components, the subtype of the selector must be a
+      --  constrained Unchecked_Union. If the component is subject to a
+      --  per-object constraint, then the enclosing object must have inferable
+      --  discriminants.
+
+      elsif Nkind (N) = N_Selected_Component then
+         if Has_Per_Object_Constraint (Entity (Selector_Name (N))) then
+
+            --  A small hack. If we have a per-object constrained selected
+            --  component of a formal parameter, return True since we do not
+            --  know the actual parameter association yet.
+
+            if Prefix_Is_Formal_Parameter (N) then
+               return True;
+            end if;
+
+            --  Otherwise, check the enclosing object and the selector
+
+            return Has_Inferable_Discriminants (Prefix (N))
+                     and then
+                   Has_Inferable_Discriminants (Selector_Name (N));
+         end if;
+
+         --  The call to Has_Inferable_Discriminants will determine whether
+         --  the selector has a constrained Unchecked_Union nominal type.
+
+         return Has_Inferable_Discriminants (Selector_Name (N));
+
+      --  A qualified expression has inferable discriminants if its subtype
+      --  mark is a constrained Unchecked_Union subtype.
+
+      elsif Nkind (N) = N_Qualified_Expression then
+         return Is_Unchecked_Union (Subtype_Mark (N))
+                  and then
+                Is_Constrained (Subtype_Mark (N));
+
+      end if;
+
+      return False;
+   end Has_Inferable_Discriminants;
+
    -------------------------------
    -- Insert_Dereference_Action --
    -------------------------------
@@ -5411,9 +8337,14 @@ package body Exp_Ch4 is
       Loc  : constant Source_Ptr := Sloc (N);
       Typ  : constant Entity_Id  := Etype (N);
       Pool : constant Entity_Id  := Associated_Storage_Pool (Typ);
+      Pnod : constant Node_Id    := Parent (N);
 
       function Is_Checked_Storage_Pool (P : Entity_Id) return Boolean;
-      --  return true if type of P is derived from Checked_Pool;
+      --  Return true if type of P is derived from Checked_Pool;
+
+      -----------------------------
+      -- Is_Checked_Storage_Pool --
+      -----------------------------
 
       function Is_Checked_Storage_Pool (P : Entity_Id) return Boolean is
          T : Entity_Id;
@@ -5438,10 +8369,11 @@ package body Exp_Ch4 is
    --  Start of processing for Insert_Dereference_Action
 
    begin
-      if not Comes_From_Source (Parent (N)) then
-         return;
+      pragma Assert (Nkind (Pnod) = N_Explicit_Dereference);
 
-      elsif not Is_Checked_Storage_Pool (Pool) then
+      if not (Is_Checked_Storage_Pool (Pool)
+              and then Comes_From_Source (Original_Node (Pnod)))
+      then
          return;
       end if;
 
@@ -5456,12 +8388,15 @@ package body Exp_Ch4 is
 
              New_Reference_To (Pool, Loc),
 
-            --  Storage_Address
+            --  Storage_Address. We use the attribute Pool_Address,
+            --  which uses the pointer itself to find the address of
+            --  the object, and which handles unconstrained arrays
+            --  properly by computing the address of the template.
+            --  i.e. the correct address of the corresponding allocation.
 
              Make_Attribute_Reference (Loc,
-               Prefix         =>
-                 Make_Explicit_Dereference (Loc, Duplicate_Subexpr (N)),
-               Attribute_Name => Name_Address),
+               Prefix         => Duplicate_Subexpr_Move_Checks (N),
+               Attribute_Name => Name_Pool_Address),
 
             --  Size_In_Storage_Elements
 
@@ -5469,7 +8404,8 @@ package body Exp_Ch4 is
                Left_Opnd  =>
                 Make_Attribute_Reference (Loc,
                   Prefix         =>
-                    Make_Explicit_Dereference (Loc, Duplicate_Subexpr (N)),
+                    Make_Explicit_Dereference (Loc,
+                      Duplicate_Subexpr_Move_Checks (N)),
                   Attribute_Name => Name_Size),
                Right_Opnd =>
                  Make_Integer_Literal (Loc, System_Storage_Unit)),
@@ -5478,9 +8414,13 @@ package body Exp_Ch4 is
 
              Make_Attribute_Reference (Loc,
                Prefix         =>
-                 Make_Explicit_Dereference (Loc, Duplicate_Subexpr (N)),
+                 Make_Explicit_Dereference (Loc,
+                   Duplicate_Subexpr_Move_Checks (N)),
                Attribute_Name => Name_Alignment))));
 
+   exception
+      when RE_Not_Available =>
+         return;
    end Insert_Dereference_Action;
 
    ------------------------------
@@ -5493,17 +8433,17 @@ package body Exp_Ch4 is
    --    type elem is  (<>);
    --    type index is (<>);
    --    type a is array (index range <>) of elem;
-   --
+
    --  function Gnnn (X : a; Y: a) return boolean is
    --    J : index := Y'first;
-   --
+
    --  begin
    --    if X'length = 0 then
    --       return false;
-   --
+
    --    elsif Y'length = 0 then
    --       return true;
-   --
+
    --    else
    --      for I in X'range loop
    --        if X (I) = Y (J) then
@@ -5512,12 +8452,12 @@ package body Exp_Ch4 is
    --          else
    --            J := index'succ (J);
    --          end if;
-   --
+
    --        else
    --           return X (I) > Y (J);
    --        end if;
    --      end loop;
-   --
+
    --      return X'length > Y'length;
    --    end if;
    --  end Gnnn;
@@ -5527,9 +8467,8 @@ package body Exp_Ch4 is
    --  instantiated function itself.
 
    function Make_Array_Comparison_Op
-     (Typ   : Entity_Id;
-      Nod   : Node_Id)
-      return  Node_Id
+     (Typ : Entity_Id;
+      Nod : Node_Id) return Node_Id
    is
       Loc : constant Source_Ptr := Sloc (Nod);
 
@@ -5604,7 +8543,7 @@ package body Exp_Ch4 is
           Then_Statements => New_List (Inner_If),
 
           Else_Statements => New_List (
-            Make_Return_Statement (Loc,
+            Make_Simple_Return_Statement (Loc,
               Expression =>
                 Make_Op_Gt (Loc,
                   Left_Opnd =>
@@ -5675,7 +8614,7 @@ package body Exp_Ch4 is
 
           Then_Statements =>
             New_List (
-              Make_Return_Statement (Loc,
+              Make_Simple_Return_Statement (Loc,
                 Expression => New_Reference_To (Standard_False, Loc))),
 
           Elsif_Parts => New_List (
@@ -5691,12 +8630,12 @@ package body Exp_Ch4 is
 
               Then_Statements =>
                 New_List (
-                  Make_Return_Statement (Loc,
+                  Make_Simple_Return_Statement (Loc,
                      Expression => New_Reference_To (Standard_True, Loc))))),
 
           Else_Statements => New_List (
             Loop_Statement,
-            Make_Return_Statement (Loc,
+            Make_Simple_Return_Statement (Loc,
               Expression => Final_Expr)));
 
       --  (X : a; Y: a)
@@ -5724,7 +8663,7 @@ package body Exp_Ch4 is
             Make_Function_Specification (Loc,
               Defining_Unit_Name       => Func_Name,
               Parameter_Specifications => Formals,
-              Subtype_Mark => New_Reference_To (Standard_Boolean, Loc)),
+              Result_Definition => New_Reference_To (Standard_Boolean, Loc)),
 
           Declarations => New_List (
             Make_Object_Declaration (Loc,
@@ -5740,7 +8679,6 @@ package body Exp_Ch4 is
               Statements => New_List (If_Stat)));
 
       return Func_Body;
-
    end Make_Array_Comparison_Op;
 
    ---------------------------
@@ -5762,9 +8700,8 @@ package body Exp_Ch4 is
    --  Here typ is the boolean array type
 
    function Make_Boolean_Array_Op
-     (Typ  : Entity_Id;
-      N    : Node_Id)
-      return Node_Id
+     (Typ : Entity_Id;
+      N   : Node_Id) return Node_Id
    is
       Loc : constant Source_Ptr := Sloc (N);
 
@@ -5856,7 +8793,7 @@ package body Exp_Ch4 is
             Make_Function_Specification (Loc,
               Defining_Unit_Name       => Func_Name,
               Parameter_Specifications => Formals,
-              Subtype_Mark             => New_Reference_To (Typ, Loc)),
+              Result_Definition        => New_Reference_To (Typ, Loc)),
 
           Declarations => New_List (
             Make_Object_Declaration (Loc,
@@ -5867,7 +8804,7 @@ package body Exp_Ch4 is
             Make_Handled_Sequence_Of_Statements (Loc,
               Statements => New_List (
                 Loop_Statement,
-                Make_Return_Statement (Loc,
+                Make_Simple_Return_Statement (Loc,
                   Expression => New_Reference_To (C, Loc)))));
 
       return Func_Body;
@@ -5878,54 +8815,183 @@ package body Exp_Ch4 is
    ------------------------
 
    procedure Rewrite_Comparison (N : Node_Id) is
-      Typ : constant Entity_Id := Etype (N);
-      Op1 : constant Node_Id   := Left_Opnd (N);
-      Op2 : constant Node_Id   := Right_Opnd (N);
+   begin
+      if Nkind (N) = N_Type_Conversion then
+         Rewrite_Comparison (Expression (N));
+         return;
+
+      elsif Nkind (N) not in N_Op_Compare then
+         return;
+      end if;
+
+      declare
+         Typ : constant Entity_Id := Etype (N);
+         Op1 : constant Node_Id   := Left_Opnd (N);
+         Op2 : constant Node_Id   := Right_Opnd (N);
+
+         Res : constant Compare_Result := Compile_Time_Compare (Op1, Op2);
+         --  Res indicates if compare outcome can be compile time determined
+
+         True_Result  : Boolean;
+         False_Result : Boolean;
+
+      begin
+         case N_Op_Compare (Nkind (N)) is
+            when N_Op_Eq =>
+               True_Result  := Res = EQ;
+               False_Result := Res = LT or else Res = GT or else Res = NE;
+
+            when N_Op_Ge =>
+               True_Result  := Res in Compare_GE;
+               False_Result := Res = LT;
+
+               if Res = LE
+                 and then Constant_Condition_Warnings
+                 and then Comes_From_Source (Original_Node (N))
+                 and then Nkind (Original_Node (N)) = N_Op_Ge
+                 and then not In_Instance
+                 and then not Warnings_Off (Etype (Left_Opnd (N)))
+                 and then Is_Integer_Type (Etype (Left_Opnd (N)))
+               then
+                  Error_Msg_N
+                    ("can never be greater than, could replace by ""'=""?", N);
+               end if;
+
+            when N_Op_Gt =>
+               True_Result  := Res = GT;
+               False_Result := Res in Compare_LE;
+
+            when N_Op_Lt =>
+               True_Result  := Res = LT;
+               False_Result := Res in Compare_GE;
+
+            when N_Op_Le =>
+               True_Result  := Res in Compare_LE;
+               False_Result := Res = GT;
+
+               if Res = GE
+                 and then Constant_Condition_Warnings
+                 and then Comes_From_Source (Original_Node (N))
+                 and then Nkind (Original_Node (N)) = N_Op_Le
+                 and then not In_Instance
+                 and then not Warnings_Off (Etype (Left_Opnd (N)))
+                 and then Is_Integer_Type (Etype (Left_Opnd (N)))
+               then
+                  Error_Msg_N
+                    ("can never be less than, could replace by ""'=""?", N);
+               end if;
+
+            when N_Op_Ne =>
+               True_Result  := Res = NE or else Res = GT or else Res = LT;
+               False_Result := Res = EQ;
+         end case;
+
+         if True_Result then
+            Rewrite (N,
+              Convert_To (Typ,
+                New_Occurrence_Of (Standard_True, Sloc (N))));
+            Analyze_And_Resolve (N, Typ);
+            Warn_On_Known_Condition (N);
+
+         elsif False_Result then
+            Rewrite (N,
+              Convert_To (Typ,
+                New_Occurrence_Of (Standard_False, Sloc (N))));
+            Analyze_And_Resolve (N, Typ);
+            Warn_On_Known_Condition (N);
+         end if;
+      end;
+   end Rewrite_Comparison;
+
+   ----------------------------
+   -- Safe_In_Place_Array_Op --
+   ----------------------------
+
+   function Safe_In_Place_Array_Op
+     (Lhs : Node_Id;
+      Op1 : Node_Id;
+      Op2 : Node_Id) return Boolean
+   is
+      Target : Entity_Id;
+
+      function Is_Safe_Operand (Op : Node_Id) return Boolean;
+      --  Operand is safe if it cannot overlap part of the target of the
+      --  operation. If the operand and the target are identical, the operand
+      --  is safe. The operand can be empty in the case of negation.
+
+      function Is_Unaliased (N : Node_Id) return Boolean;
+      --  Check that N is a stand-alone entity
+
+      ------------------
+      -- Is_Unaliased --
+      ------------------
+
+      function Is_Unaliased (N : Node_Id) return Boolean is
+      begin
+         return
+           Is_Entity_Name (N)
+             and then No (Address_Clause (Entity (N)))
+             and then No (Renamed_Object (Entity (N)));
+      end Is_Unaliased;
+
+      ---------------------
+      -- Is_Safe_Operand --
+      ---------------------
+
+      function Is_Safe_Operand (Op : Node_Id) return Boolean is
+      begin
+         if No (Op) then
+            return True;
+
+         elsif Is_Entity_Name (Op) then
+            return Is_Unaliased (Op);
+
+         elsif Nkind_In (Op, N_Indexed_Component, N_Selected_Component) then
+            return Is_Unaliased (Prefix (Op));
+
+         elsif Nkind (Op) = N_Slice then
+            return
+              Is_Unaliased (Prefix (Op))
+                and then Entity (Prefix (Op)) /= Target;
+
+         elsif Nkind (Op) = N_Op_Not then
+            return Is_Safe_Operand (Right_Opnd (Op));
 
-      Res : constant Compare_Result := Compile_Time_Compare (Op1, Op2);
-      --  Res indicates if compare outcome can be determined at compile time
+         else
+            return False;
+         end if;
+      end Is_Safe_Operand;
 
-      True_Result  : Boolean;
-      False_Result : Boolean;
+      --  Start of processing for Is_Safe_In_Place_Array_Op
 
    begin
-      case N_Op_Compare (Nkind (N)) is
-         when N_Op_Eq =>
-            True_Result  := Res = EQ;
-            False_Result := Res = LT or else Res = GT or else Res = NE;
-
-         when N_Op_Ge =>
-            True_Result  := Res in Compare_GE;
-            False_Result := Res = LT;
-
-         when N_Op_Gt =>
-            True_Result  := Res = GT;
-            False_Result := Res in Compare_LE;
-
-         when N_Op_Lt =>
-            True_Result  := Res = LT;
-            False_Result := Res in Compare_GE;
-
-         when N_Op_Le =>
-            True_Result  := Res in Compare_LE;
-            False_Result := Res = GT;
-
-         when N_Op_Ne =>
-            True_Result  := Res = NE;
-            False_Result := Res = LT or else Res = GT or else Res = EQ;
-      end case;
+      --  We skip this processing if the component size is not the
+      --  same as a system storage unit (since at least for NOT
+      --  this would cause problems).
 
-      if True_Result then
-         Rewrite (N,
-           Convert_To (Typ, New_Occurrence_Of (Standard_True, Sloc (N))));
-         Analyze_And_Resolve (N, Typ);
+      if Component_Size (Etype (Lhs)) /= System_Storage_Unit then
+         return False;
 
-      elsif False_Result then
-         Rewrite (N,
-           Convert_To (Typ, New_Occurrence_Of (Standard_False, Sloc (N))));
-         Analyze_And_Resolve (N, Typ);
+      --  Cannot do in place stuff on VM_Target since cannot pass addresses
+
+      elsif VM_Target /= No_VM then
+         return False;
+
+      --  Cannot do in place stuff if non-standard Boolean representation
+
+      elsif Has_Non_Standard_Rep (Component_Type (Etype (Lhs))) then
+         return False;
+
+      elsif not Is_Unaliased (Lhs) then
+         return False;
+      else
+         Target := Entity (Lhs);
+
+         return
+           Is_Safe_Operand (Op1)
+             and then Is_Safe_Operand (Op2);
       end if;
-   end Rewrite_Comparison;
+   end Safe_In_Place_Array_Op;
 
    -----------------------
    -- Tagged_Membership --
@@ -5941,6 +9007,11 @@ package body Exp_Ch4 is
    --  is usually implemented by looking in the ancestor tables contained in
    --  the dispatch table pointed by Left_Expr.Tag for Typ'Tag
 
+   --  Ada 2005 (AI-251): If it is a class-wide interface type we use the RT
+   --  function IW_Membership which is usually implemented by looking in the
+   --  table of abstract interface types plus the ancestor table contained in
+   --  the dispatch table pointed by Left_Expr.Tag for Typ'Tag
+
    function Tagged_Membership (N : Node_Id) return Node_Id is
       Left  : constant Node_Id    := Left_Opnd  (N);
       Right : constant Node_Id    := Right_Opnd (N);
@@ -5961,24 +9032,102 @@ package body Exp_Ch4 is
       Obj_Tag :=
         Make_Selected_Component (Loc,
           Prefix        => Relocate_Node (Left),
-          Selector_Name => New_Reference_To (Tag_Component (Left_Type), Loc));
+          Selector_Name =>
+            New_Reference_To (First_Tag_Component (Left_Type), Loc));
 
       if Is_Class_Wide_Type (Right_Type) then
-         return
-           Make_DT_Access_Action (Left_Type,
-             Action => CW_Membership,
-             Args   => New_List (
-               Obj_Tag,
-               New_Reference_To (
-                 Access_Disp_Table (Root_Type (Right_Type)), Loc)));
+
+         --  No need to issue a run-time check if we statically know that the
+         --  result of this membership test is always true. For example,
+         --  considering the following declarations:
+
+         --    type Iface is interface;
+         --    type T     is tagged null record;
+         --    type DT    is new T and Iface with null record;
+
+         --    Obj1 : T;
+         --    Obj2 : DT;
+
+         --  These membership tests are always true:
+
+         --    Obj1 in T'Class
+         --    Obj2 in T'Class;
+         --    Obj2 in Iface'Class;
+
+         --  We do not need to handle cases where the membership is illegal.
+         --  For example:
+
+         --    Obj1 in DT'Class;     --  Compile time error
+         --    Obj1 in Iface'Class;  --  Compile time error
+
+         if not Is_Class_Wide_Type (Left_Type)
+           and then (Is_Parent (Etype (Right_Type), Left_Type)
+                       or else (Is_Interface (Etype (Right_Type))
+                                 and then Interface_Present_In_Ancestor
+                                           (Typ   => Left_Type,
+                                            Iface => Etype (Right_Type))))
+         then
+            return New_Reference_To (Standard_True, Loc);
+         end if;
+
+         --  Ada 2005 (AI-251): Class-wide applied to interfaces
+
+         if Is_Interface (Etype (Class_Wide_Type (Right_Type)))
+
+            --   Support to: "Iface_CW_Typ in Typ'Class"
+
+           or else Is_Interface (Left_Type)
+         then
+            --  Issue error if IW_Membership operation not available in a
+            --  configurable run time setting.
+
+            if not RTE_Available (RE_IW_Membership) then
+               Error_Msg_CRT ("abstract interface types", N);
+               return Empty;
+            end if;
+
+            return
+              Make_Function_Call (Loc,
+                 Name => New_Occurrence_Of (RTE (RE_IW_Membership), Loc),
+                 Parameter_Associations => New_List (
+                   Make_Attribute_Reference (Loc,
+                     Prefix => Obj_Tag,
+                     Attribute_Name => Name_Address),
+                   New_Reference_To (
+                     Node (First_Elmt
+                            (Access_Disp_Table (Root_Type (Right_Type)))),
+                     Loc)));
+
+         --  Ada 95: Normal case
+
+         else
+            return
+              Build_CW_Membership (Loc,
+                Obj_Tag_Node => Obj_Tag,
+                Typ_Tag_Node =>
+                   New_Reference_To (
+                     Node (First_Elmt
+                            (Access_Disp_Table (Root_Type (Right_Type)))),
+                     Loc));
+         end if;
+
+      --  Right_Type is not a class-wide type
+
       else
-         return
-           Make_Op_Eq (Loc,
-           Left_Opnd  => Obj_Tag,
-           Right_Opnd =>
-             New_Reference_To (Access_Disp_Table (Right_Type), Loc));
-      end if;
+         --  No need to check the tag of the object if Right_Typ is abstract
+
+         if Is_Abstract_Type (Right_Type) then
+            return New_Reference_To (Standard_False, Loc);
 
+         else
+            return
+              Make_Op_Eq (Loc,
+                Left_Opnd  => Obj_Tag,
+                Right_Opnd =>
+                  New_Reference_To
+                    (Node (First_Elmt (Access_Disp_Table (Right_Type))), Loc));
+         end if;
+      end if;
    end Tagged_Membership;
 
    ------------------------------