OSDN Git Service

* gigi.h (standard_datatypes): Add ADT_fdesc_type and ADT_null_fdesc.
[pf3gnuchains/gcc-fork.git] / gcc / ada / decl.c
index 145ece8..aca69ff 100644 (file)
@@ -6,18 +6,17 @@
  *                                                                          *
  *                          C Implementation File                           *
  *                                                                          *
- *          Copyright (C) 1992-2005, Free Software Foundation, Inc.         *
+ *          Copyright (C) 1992-2008, 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,  51  Franklin  Street,  Fifth  Floor, *
- * Boston, MA 02110-1301, USA.                                              *
+ * for  more details.  You should have received a copy of the GNU General   *
+ * Public License along with GCC; see the file COPYING3.  If not see        *
+ * <http://www.gnu.org/licenses/>.                                          *
  *                                                                          *
  * GNAT was originally developed  by the GNAT team at  New York University. *
  * Extensive contributions were provided by Ada Core Technologies Inc.      *
@@ -35,6 +34,7 @@
 #include "ggc.h"
 #include "obstack.h"
 #include "target.h"
+#include "expr.h"
 
 #include "ada.h"
 #include "types.h"
 #include "fe.h"
 #include "sinfo.h"
 #include "einfo.h"
+#include "hashtab.h"
 #include "ada-tree.h"
 #include "gigi.h"
 
-/* Provide default values for the macros controlling stack checking.
-   This is copied from GCC's expr.h.  */
-
-#ifndef STACK_CHECK_BUILTIN
-#define STACK_CHECK_BUILTIN 0
-#endif
-#ifndef STACK_CHECK_PROBE_INTERVAL
-#define STACK_CHECK_PROBE_INTERVAL 4096
-#endif
-#ifndef STACK_CHECK_MAX_FRAME_SIZE
-#define STACK_CHECK_MAX_FRAME_SIZE \
-  (STACK_CHECK_PROBE_INTERVAL - UNITS_PER_WORD)
-#endif
-#ifndef STACK_CHECK_MAX_VAR_SIZE
-#define STACK_CHECK_MAX_VAR_SIZE (STACK_CHECK_MAX_FRAME_SIZE / 100)
+#ifndef MAX_FIXED_MODE_SIZE
+#define MAX_FIXED_MODE_SIZE GET_MODE_BITSIZE (DImode)
 #endif
 
-/* These two variables are used to defer recursively expanding incomplete
-   types while we are processing a record or subprogram type.  */
+/* Convention_Stdcall should be processed in a specific way on Windows targets
+   only.  The macro below is a helper to avoid having to check for a Windows
+   specific attribute throughout this unit.  */
 
-static int defer_incomplete_level = 0;
-static struct incomplete
+#if TARGET_DLLIMPORT_DECL_ATTRIBUTES
+#define Has_Stdcall_Convention(E) (Convention (E) == Convention_Stdcall)
+#else
+#define Has_Stdcall_Convention(E) (0)
+#endif
+
+struct incomplete
 {
   struct incomplete *next;
   tree old_type;
   Entity_Id full_type;
-} *defer_incomplete_list = 0;
+};
+
+/* These variables are used to defer recursively expanding incomplete types
+   while we are processing an array, a record or a subprogram type.  */
+static int defer_incomplete_level = 0;
+static struct incomplete *defer_incomplete_list;
+
+/* This variable is used to delay expanding From_With_Type types until the
+   end of the spec.  */
+static struct incomplete *defer_limited_with;
 
-/* These two variables are used to defer emission of debug information for
-   nested incomplete record types  */
+/* These variables are used to defer finalizing types.  The element of the
+   list is the TYPE_DECL associated with the type.  */
+static int defer_finalize_level = 0;
+static VEC (tree,heap) *defer_finalize_list;
 
-static int defer_debug_level = 0;
-static tree defer_debug_incomplete_list;
+/* A hash table used to cache the result of annotate_value.  */
+static GTY ((if_marked ("tree_int_map_marked_p"),
+            param_is (struct tree_int_map))) htab_t annotate_value_cache;
 
 static void copy_alias_set (tree, tree);
 static tree substitution_list (Entity_Id, Entity_Id, tree, bool);
 static bool allocatable_size_p (tree, bool);
+static void prepend_one_attribute_to (struct attrib **,
+                                     enum attr_type, tree, tree, Node_Id);
 static void prepend_attributes (Entity_Id, struct attrib **);
 static tree elaborate_expression (Node_Id, Entity_Id, tree, bool, bool, bool);
 static bool is_variable_size (tree);
 static tree elaborate_expression_1 (Node_Id, Entity_Id, tree, tree,
                                    bool, bool);
-static tree make_packable_type (tree);
+static tree make_packable_type (tree, bool);
 static tree gnat_to_gnu_field (Entity_Id, tree, int, bool);
+static tree gnat_to_gnu_param (Entity_Id, Mechanism_Type, Entity_Id, bool,
+                              bool *);
+static bool same_discriminant_p (Entity_Id, Entity_Id);
+static bool array_type_has_nonaliased_component (Entity_Id, tree);
 static void components_to_record (tree, Node_Id, tree, int, bool, tree *,
-                                  bool, bool, bool);
-static int compare_field_bitpos (const PTR, const PTR);
+                                 bool, bool, bool, bool);
 static Uint annotate_value (tree);
 static void annotate_rep (Entity_Id, tree);
 static tree compute_field_positions (tree, tree, tree, tree, unsigned int);
@@ -106,8 +117,10 @@ static tree validate_size (Uint, tree, Entity_Id, enum tree_code, bool, bool);
 static void set_rm_size (Uint, tree, Entity_Id);
 static tree make_type_from_size (tree, tree, bool);
 static unsigned int validate_alignment (Uint, Entity_Id, unsigned int);
+static unsigned int ceil_alignment (unsigned HOST_WIDE_INT);
 static void check_ok_for_atomic (tree, Entity_Id, bool);
-static int  compatible_signatures_p (tree ftype1, tree ftype2);
+static int compatible_signatures_p (tree ftype1, tree ftype2);
+static void rest_of_type_decl_compilation_no_defer (tree);
 
 /* Given GNAT_ENTITY, an entity in the incoming GNAT tree, return a
    GCC type corresponding to that entity.  GNAT_ENTITY is assumed to
@@ -140,13 +153,14 @@ gnat_to_gnu_type (Entity_Id gnat_entity)
    DEFINITION is nonzero if this call is intended for a definition.  This is
    used for separate compilation where it necessary to know whether an
    external declaration or a definition should be created if the GCC equivalent
-   was not created previously.  The value of 1 is normally used for a non-zero
+   was not created previously.  The value of 1 is normally used for a nonzero
    DEFINITION, but a value of 2 is used in special circumstances, defined in
    the code.  */
 
 tree
 gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 {
+  Entity_Id gnat_equiv_type = Gigi_Equivalent_Type (gnat_entity);
   tree gnu_entity_id;
   tree gnu_type = NULL_TREE;
   /* Contains the gnu XXXX_DECL tree node which is equivalent to the input
@@ -157,8 +171,6 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
   bool saved = false;
   /* Nonzero if we incremented defer_incomplete_level.  */
   bool this_deferred = false;
-  /* Nonzero if we incremented defer_debug_level.  */
-  bool debug_deferred = false;
   /* Nonzero if we incremented force_global.  */
   bool this_global = false;
   /* Nonzero if we should check to see if elaborated during processing.  */
@@ -181,8 +193,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        : LONG_LONG_TYPE_SIZE);
   tree gnu_size = 0;
   bool imported_p
-    = ((Is_Imported (gnat_entity) && No (Address_Clause (gnat_entity)))
-       || From_With_Type (gnat_entity));
+    = (Is_Imported (gnat_entity) && No (Address_Clause (gnat_entity)));
   unsigned int align = 0;
 
   /* Since a use of an Itype is a definition, process it as such if it
@@ -246,8 +257,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
      when a Full_View exists.  */
 
   if (present_gnu_tree (gnat_entity)
-      && (! definition
-         || (Is_Type (gnat_entity) && imported_p)))
+      && (!definition || (Is_Type (gnat_entity) && imported_p)))
     {
       gnu_decl = get_gnu_tree (gnat_entity);
 
@@ -273,6 +283,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
              || (!IN (kind, Numeric_Kind) && !IN (kind, Enumeration_Kind)
                  && (!IN (kind, Access_Kind)
                      || kind == E_Access_Protected_Subprogram_Type
+                     || kind == E_Anonymous_Access_Protected_Subprogram_Type
                      || kind == E_Access_Subtype)));
 
   /* Likewise, RM_Size must be specified for all discrete and fixed-point
@@ -327,39 +338,37 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       if (!definition && Present (Full_View (gnat_entity)))
        {
          gnu_decl = gnat_to_gnu_entity (Full_View (gnat_entity),
-                                        gnu_expr, definition);
+                                        gnu_expr, 0);
          saved = true;
          break;
        }
 
-      /* If we have an external constant that we are not defining,
-        get the expression that is was defined to represent.  We
-        may throw that expression away later if it is not a
-        constant.
-         Do not retrieve the expression if it is an aggregate, because
-         in complex instantiation contexts it may not be expanded  */
-
+      /* If we have an external constant that we are not defining, get the
+        expression that is was defined to represent.  We may throw that
+        expression away later if it is not a constant.  Do not retrieve the
+        expression if it is an aggregate or allocator, because in complex
+        instantiation contexts it may not be expanded  */
       if (!definition
          && Present (Expression (Declaration_Node (gnat_entity)))
          && !No_Initialization (Declaration_Node (gnat_entity))
-          && (Nkind (Expression   (Declaration_Node (gnat_entity)))
-             != N_Aggregate))
+         && (Nkind (Expression (Declaration_Node (gnat_entity)))
+             != N_Aggregate)
+         && (Nkind (Expression (Declaration_Node (gnat_entity)))
+             != N_Allocator))
        gnu_expr = gnat_to_gnu (Expression (Declaration_Node (gnat_entity)));
 
       /* Ignore deferred constant definitions; they are processed fully in the
-        front-end.  For deferred constant references, get the full
-         definition.  On the other hand, constants that are renamings are
-        handled like variable renamings.  If No_Initialization is set, this is
-        not a deferred constant but a constant whose value is built
-        manually.  */
-
+        front-end.  For deferred constant references get the full definition.
+        On the other hand, constants that are renamings are handled like
+        variable renamings.  If No_Initialization is set, this is not a
+        deferred constant but a constant whose value is built manually.  */
       if (definition && !gnu_expr
          && !No_Initialization (Declaration_Node (gnat_entity))
          && No (Renamed_Object (gnat_entity)))
        {
          gnu_decl = error_mark_node;
          saved = true;
-          break;
+         break;
        }
       else if (!definition && IN (kind, Incomplete_Or_Private_Kind)
               && Present (Full_View (gnat_entity)))
@@ -429,6 +438,15 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            break;
          }
 
+       else if (Present (CR_Discriminant (gnat_entity))
+                && type_annotate_only)
+         {
+           gnu_decl = gnat_to_gnu_entity (CR_Discriminant (gnat_entity),
+                                          gnu_expr, definition);
+           saved = true;
+           break;
+         }
+
        /* If the enclosing record has explicit stored discriminants,
           then it is an untagged record. If the Corresponding_Discriminant
           is not empty then this must be a renamed discriminant and its
@@ -450,18 +468,17 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
               type and we have an Original_Record_Component, use it.
               This is a workaround for major problems in protected type
               handling.  */
-
-            Entity_Id Scop = Scope (Scope (gnat_entity));
-            if ((Is_Protected_Type (Scop)
-                || (Is_Private_Type (Scop)
-                     && Present (Full_View (Scop))
-                     && Is_Protected_Type (Full_View (Scop))))
+           Entity_Id Scop = Scope (Scope (gnat_entity));
+           if ((Is_Protected_Type (Scop)
+                || (Is_Private_Type (Scop)
+                    && Present (Full_View (Scop))
+                    && Is_Protected_Type (Full_View (Scop))))
                && Present (Original_Record_Component (gnat_entity)))
              {
                gnu_decl
                  = gnat_to_gnu_entity (Original_Record_Component
                                        (gnat_entity),
-                                       gnu_expr, definition);
+                                       gnu_expr, 0);
                saved = true;
                break;
              }
@@ -483,13 +500,12 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
     case E_Out_Parameter:
     case E_Variable:
 
-      /* Simple variables, loop variables, OUT parameters, and exceptions.  */
+      /* Simple variables, loop variables, Out parameters, and exceptions.  */
     object:
       {
        bool used_by_ref = false;
        bool const_flag
          = ((kind == E_Constant || kind == E_Variable)
-            && !Is_Statically_Allocated (gnat_entity)
             && Is_True_Constant (gnat_entity)
             && (((Nkind (Declaration_Node (gnat_entity))
                   == N_Object_Declaration)
@@ -497,6 +513,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                 || Present (Renamed_Object (gnat_entity))));
        bool inner_const_flag = const_flag;
        bool static_p = Is_Statically_Allocated (gnat_entity);
+       bool mutable_p = false;
        tree gnu_ext_name = NULL_TREE;
        tree renamed_obj = NULL_TREE;
 
@@ -512,6 +529,21 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        /* Get the type after elaborating the renamed object.  */
        gnu_type = gnat_to_gnu_type (Etype (gnat_entity));
 
+       /* For a debug renaming declaration, build a pure debug entity.  */
+       if (Present (Debug_Renaming_Link (gnat_entity)))
+         {
+           rtx addr;
+           gnu_decl = build_decl (VAR_DECL, gnu_entity_id, gnu_type);
+           /* The (MEM (CONST (0))) pattern is prescribed by STABS.  */
+           if (global_bindings_p ())
+             addr = gen_rtx_CONST (VOIDmode, const0_rtx);
+           else
+             addr = stack_pointer_rtx;
+           SET_DECL_RTL (gnu_decl, gen_rtx_MEM (Pmode, addr));
+           gnat_pushdecl (gnu_decl, gnat_entity);
+           break;
+         }
+
        /* If this is a loop variable, its type should be the base type.
           This is because the code for processing a loop determines whether
           a normal loop end test can be done by comparing the bounds of the
@@ -530,6 +562,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            || TREE_CODE (gnu_type) == VOID_TYPE)
          {
            gcc_assert (type_annotate_only);
+           if (this_global)
+             force_global--;
            return error_mark_node;
          }
 
@@ -542,8 +576,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            gcc_assert (Present (Alignment (gnat_entity)));
            align = validate_alignment (Alignment (gnat_entity), gnat_entity,
                                        TYPE_ALIGN (gnu_type));
-           gnu_type = maybe_pad_type (gnu_type, NULL_TREE, align,
-                                      gnat_entity, "PAD", 0, definition, 1);
+           gnu_type = maybe_pad_type (gnu_type, NULL_TREE, align, gnat_entity,
+                                      "PAD", false, definition, true);
          }
 
        /* If we are defining the object, see if it has a Size value and
@@ -574,15 +608,34 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           initializing expression, in which case we can get the size from
           that.  Note that the resulting size may still be a variable, so
           this may end up with an indirect allocation.  */
-
        if (No (Renamed_Object (gnat_entity))
            && CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_type)))
          {
            if (gnu_expr && kind == E_Constant)
-             gnu_size
-               = SUBSTITUTE_PLACEHOLDER_IN_EXPR
-                 (TYPE_SIZE (TREE_TYPE (gnu_expr)), gnu_expr);
-
+             {
+               tree size = TYPE_SIZE (TREE_TYPE (gnu_expr));
+               if (CONTAINS_PLACEHOLDER_P (size))
+                 {
+                   /* If the initializing expression is itself a constant,
+                      despite having a nominal type with self-referential
+                      size, we can get the size directly from it.  */
+                   if (TREE_CODE (gnu_expr) == COMPONENT_REF
+                       && TREE_CODE (TREE_TYPE (TREE_OPERAND (gnu_expr, 0)))
+                          == RECORD_TYPE
+                       && TYPE_IS_PADDING_P
+                          (TREE_TYPE (TREE_OPERAND (gnu_expr, 0)))
+                       && TREE_CODE (TREE_OPERAND (gnu_expr, 0)) == VAR_DECL
+                       && (TREE_READONLY (TREE_OPERAND (gnu_expr, 0))
+                           || DECL_READONLY_ONCE_ELAB
+                              (TREE_OPERAND (gnu_expr, 0))))
+                     gnu_size = DECL_SIZE (TREE_OPERAND (gnu_expr, 0));
+                   else
+                     gnu_size
+                       = SUBSTITUTE_PLACEHOLDER_IN_EXPR (size, gnu_expr);
+                 }
+               else
+                 gnu_size = size;
+             }
            /* We may have no GNU_EXPR because No_Initialization is
               set even though there's an Expression.  */
            else if (kind == E_Constant
@@ -594,7 +647,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                             (Etype
                              (Expression (Declaration_Node (gnat_entity)))));
            else
-             gnu_size = max_size (TYPE_SIZE (gnu_type), true);
+             {
+               gnu_size = max_size (TYPE_SIZE (gnu_type), true);
+               mutable_p = true;
+             }
          }
 
        /* If the size is zero bytes, make it one byte since some linkers have
@@ -604,31 +660,54 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           clause, as we would lose useful information on the view size
           (e.g. for null array slices) and we are not allocating the object
           here anyway.  */
-       if (((gnu_size && integer_zerop (gnu_size))
-            || (TYPE_SIZE (gnu_type) && integer_zerop (TYPE_SIZE (gnu_type))))
+       if (((gnu_size
+             && integer_zerop (gnu_size)
+             && !TREE_OVERFLOW (gnu_size))
+            || (TYPE_SIZE (gnu_type)
+                && integer_zerop (TYPE_SIZE (gnu_type))
+                && !TREE_OVERFLOW (TYPE_SIZE (gnu_type))))
            && (!Is_Constr_Subt_For_UN_Aliased (Etype (gnat_entity))
                || !Is_Array_Type (Etype (gnat_entity)))
            && !Present (Renamed_Object (gnat_entity))
            && !Present (Address_Clause (gnat_entity)))
          gnu_size = bitsize_unit_node;
 
-       /* If this is an atomic object with no specified size and alignment,
-          but where the size of the type is a constant, set the alignment to
-          the lowest power of two greater than the size, or to the
-          biggest meaningful alignment, whichever is smaller.  */
-
-       if (Is_Atomic (gnat_entity) && !gnu_size && align == 0
+       /* If this is an object with no specified size and alignment, and if
+          either it is atomic or we are not optimizing alignment for space
+          and it is a non-scalar variable, and the size of its type is a
+          constant, set the alignment to the smallest not less than the
+          size, or to the biggest meaningful one, whichever is smaller.  */
+       if (!gnu_size && align == 0
+           && (Is_Atomic (gnat_entity)
+               || (Debug_Flag_Dot_A
+                   && !Optimize_Alignment_Space (gnat_entity)
+                   && kind == E_Variable
+                   && AGGREGATE_TYPE_P (gnu_type)
+                   && !const_flag && No (Renamed_Object (gnat_entity))
+                   && !imported_p && No (Address_Clause (gnat_entity))))
            && TREE_CODE (TYPE_SIZE (gnu_type)) == INTEGER_CST)
          {
+           /* No point in jumping through all the hoops needed in order
+              to support BIGGEST_ALIGNMENT if we don't really have to.  */
+           unsigned int align_cap = Is_Atomic (gnat_entity)
+                                    ? BIGGEST_ALIGNMENT
+                                    : MAX_FIXED_MODE_SIZE;
+
            if (!host_integerp (TYPE_SIZE (gnu_type), 1)
-               || 0 <= compare_tree_int (TYPE_SIZE (gnu_type),
-                                         BIGGEST_ALIGNMENT))
-             align = BIGGEST_ALIGNMENT;
+               || compare_tree_int (TYPE_SIZE (gnu_type), align_cap) >= 0)
+             align = align_cap;
            else
-             align = ((unsigned int) 1
-                      << (floor_log2 (tree_low_cst
-                                      (TYPE_SIZE (gnu_type), 1) - 1)
-                          + 1));
+             align = ceil_alignment (tree_low_cst (TYPE_SIZE (gnu_type), 1));
+
+           /* But make sure not to under-align the object.  */
+           if (align < TYPE_ALIGN (gnu_type))
+             align = TYPE_ALIGN (gnu_type);
+
+           /* And honor the minimum valid atomic alignment, if any.  */
+#ifdef MINIMUM_ATOMIC_ALIGNMENT
+           if (align < MINIMUM_ATOMIC_ALIGNMENT)
+             align = MINIMUM_ATOMIC_ALIGNMENT;
+#endif
          }
 
        /* If the object is set to have atomic components, find the component
@@ -666,11 +745,9 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        {
          tree gnu_fat
            = TREE_TYPE (gnat_to_gnu_type (Base_Type (Etype (gnat_entity))));
-         tree gnu_temp_type
-           = TREE_TYPE (TREE_TYPE (TREE_CHAIN (TYPE_FIELDS (gnu_fat))));
 
          gnu_type
-           = build_unc_object_type (gnu_temp_type, gnu_type,
+           = build_unc_object_type_from_ptr (gnu_fat, gnu_type,
                                     concat_id_with_name (gnu_entity_id,
                                                          "UNC"));
        }
@@ -698,11 +775,15 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        gnu_type = maybe_pad_type (gnu_type, gnu_size, align, gnat_entity,
                                   "PAD", false, definition, true);
 
-       /* Make a volatile version of this object's type if we are to
-          make the object volatile.  Note that 13.3(19) says that we
-          should treat other types of objects as volatile as well.  */
+       /* Make a volatile version of this object's type if we are to make
+          the object volatile.  We also interpret 13.3(19) conservatively
+          and disallow any optimizations for an object covered by it.  */
        if ((Treat_As_Volatile (gnat_entity)
-            || Is_Exported (gnat_entity)
+            || (Is_Exported (gnat_entity)
+                /* Exclude exported constants created by the compiler,
+                   which should boil down to static dispatch tables and
+                   make it possible to put them in read-only memory.  */
+                && (Comes_From_Source (gnat_entity) || !const_flag))
             || Is_Imported (gnat_entity)
             || Present (Address_Clause (gnat_entity)))
            && !TYPE_VOLATILE (gnu_type))
@@ -710,99 +791,150 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                           (TYPE_QUALS (gnu_type)
                                            | TYPE_QUAL_VOLATILE));
 
-       /* Convert the expression to the type of the object except in the
-          case where the object's type is unconstrained or the object's type
-          is a padded record whose field is of self-referential size.  In
-          the former case, converting will generate unnecessary evaluations
-          of the CONSTRUCTOR to compute the size and in the latter case, we
-          want to only copy the actual data.  */
-       if (gnu_expr
-           && TREE_CODE (gnu_type) != UNCONSTRAINED_ARRAY_TYPE
-           && !CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_type))
-           && !(TREE_CODE (gnu_type) == RECORD_TYPE
-                && TYPE_IS_PADDING_P (gnu_type)
-                && (CONTAINS_PLACEHOLDER_P
-                    (TYPE_SIZE (TREE_TYPE (TYPE_FIELDS (gnu_type)))))))
-         gnu_expr = convert (gnu_type, gnu_expr);
-
-       /* See if this is a renaming.  If this is a constant renaming, treat
-          it as a normal variable whose initial value is what is being
-          renamed.  We cannot do this if the type is unconstrained or
-          class-wide.
-
-          Otherwise, if what we are renaming is a reference, we can simply
-          return a stabilized version of that reference, after forcing any
-          SAVE_EXPRs to be evaluated.  But, if this is at global level, we
-          can only do this if we know no SAVE_EXPRs will be made.
-
-          Otherwise, make this into a constant pointer to the object we are
-          to rename.  */
-
+       /* If this is a renaming, avoid as much as possible to create a new
+          object.  However, in several cases, creating it is required.
+          This processing needs to be applied to the raw expression so
+          as to make it more likely to rename the underlying object.  */
        if (Present (Renamed_Object (gnat_entity)))
          {
+           bool create_normal_object = false;
+
            /* If the renamed object had padding, strip off the reference
               to the inner object and reset our type.  */
-           if (TREE_CODE (gnu_expr) == COMPONENT_REF
-               && (TREE_CODE (TREE_TYPE (TREE_OPERAND (gnu_expr, 0)))
-                   == RECORD_TYPE)
-               && (TYPE_IS_PADDING_P
-                   (TREE_TYPE (TREE_OPERAND (gnu_expr, 0)))))
+           if ((TREE_CODE (gnu_expr) == COMPONENT_REF
+                && TREE_CODE (TREE_TYPE (TREE_OPERAND (gnu_expr, 0)))
+                   == RECORD_TYPE
+                && TYPE_IS_PADDING_P (TREE_TYPE (TREE_OPERAND (gnu_expr, 0))))
+               /* Strip useless conversions around the object.  */
+               || TREE_CODE (gnu_expr) == NOP_EXPR)
              {
                gnu_expr = TREE_OPERAND (gnu_expr, 0);
                gnu_type = TREE_TYPE (gnu_expr);
              }
 
-           if (const_flag
-               && !TREE_SIDE_EFFECTS (gnu_expr)
-               && TREE_CODE (gnu_type) != UNCONSTRAINED_ARRAY_TYPE
-               && TYPE_MODE (gnu_type) != BLKmode
-               && Ekind (Etype (gnat_entity)) != E_Class_Wide_Type
-                && !Is_Array_Type (Etype (gnat_entity)))
-             ;
-
-           /* If this is a declaration or reference that we can stabilize,
-              just use that declaration or reference as this entity unless
-              the latter has to be materialized.  */
-           else if ((DECL_P (gnu_expr) || REFERENCE_CLASS_P (gnu_expr))
-                    && !Materialize_Entity (gnat_entity)
-                    && (!global_bindings_p ()
-                        || (staticp (gnu_expr)
-                            && !TREE_SIDE_EFFECTS (gnu_expr))))
+           /* Case 1: If this is a constant renaming stemming from a function
+              call, treat it as a normal object whose initial value is what
+              is being renamed.  RM 3.3 says that the result of evaluating a
+              function call is a constant object.  As a consequence, it can
+              be the inner object of a constant renaming.  In this case, the
+              renaming must be fully instantiated, i.e. it cannot be a mere
+              reference to (part of) an existing object.  */
+           if (const_flag)
              {
-               gnu_decl = gnat_stabilize_reference (gnu_expr, true);
-               save_gnu_tree (gnat_entity, gnu_decl, true);
-               saved = true;
-               break;
+               tree inner_object = gnu_expr;
+               while (handled_component_p (inner_object))
+                 inner_object = TREE_OPERAND (inner_object, 0);
+               if (TREE_CODE (inner_object) == CALL_EXPR)
+                 create_normal_object = true;
              }
 
-           /* Otherwise, make this into a constant pointer to the object we
-              are to rename and attach the object to the pointer.  We need
-              to stabilize too since the renaming evaluation may directly
-              reference the renamed object instead of the pointer we will
-              attach it to.  We don't want variables in the expression to
-              be evaluated every time the renaming is used, since their
-              value may change in between.  */
-           else
+           /* Otherwise, see if we can proceed with a stabilized version of
+              the renamed entity or if we need to make a new object.  */
+           if (!create_normal_object)
              {
-               bool has_side_effects = TREE_SIDE_EFFECTS (gnu_expr);
-               inner_const_flag = TREE_READONLY (gnu_expr);
-               const_flag = true;
-               gnu_type = build_reference_type (gnu_type);
-               renamed_obj = gnat_stabilize_reference (gnu_expr, true);
-               gnu_expr = build_unary_op (ADDR_EXPR, gnu_type, renamed_obj);
-
-               if (!global_bindings_p ())
+               tree maybe_stable_expr = NULL_TREE;
+               bool stable = false;
+
+               /* Case 2: If the renaming entity need not be materialized and
+                  the renamed expression is something we can stabilize, use
+                  that for the renaming.  At the global level, we can only do
+                  this if we know no SAVE_EXPRs need be made, because the
+                  expression we return might be used in arbitrary conditional
+                  branches so we must force the SAVE_EXPRs evaluation
+                  immediately and this requires a function context.  */
+               if (!Materialize_Entity (gnat_entity)
+                   && (!global_bindings_p ()
+                       || (staticp (gnu_expr)
+                           && !TREE_SIDE_EFFECTS (gnu_expr))))
                  {
-                   /* If the original expression had side effects, put a
-                      SAVE_EXPR around this whole thing.  */
-                   if (has_side_effects)
-                     gnu_expr = save_expr (gnu_expr);
+                   maybe_stable_expr
+                     = maybe_stabilize_reference (gnu_expr, true, &stable);
+
+                   if (stable)
+                     {
+                       gnu_decl = maybe_stable_expr;
+                       /* ??? No DECL_EXPR is created so we need to mark
+                          the expression manually lest it is shared.  */
+                       if (global_bindings_p ())
+                         TREE_VISITED (gnu_decl) = 1;
+                       save_gnu_tree (gnat_entity, gnu_decl, true);
+                       saved = true;
+                       break;
+                     }
 
-                   add_stmt (gnu_expr);
+                   /* The stabilization failed.  Keep maybe_stable_expr
+                      untouched here to let the pointer case below know
+                      about that failure.  */
                  }
 
-               gnu_size = NULL_TREE;
-               used_by_ref = true;
+               /* Case 3: If this is a constant renaming and creating a
+                  new object is allowed and cheap, treat it as a normal
+                  object whose initial value is what is being renamed.  */
+               if (const_flag && Is_Elementary_Type (Etype (gnat_entity)))
+                 ;
+
+               /* Case 4: Make this into a constant pointer to the object we
+                  are to rename and attach the object to the pointer if it is
+                  something we can stabilize.
+
+                  From the proper scope, attached objects will be referenced
+                  directly instead of indirectly via the pointer to avoid
+                  subtle aliasing problems with non-addressable entities.
+                  They have to be stable because we must not evaluate the
+                  variables in the expression every time the renaming is used.
+                  The pointer is called a "renaming" pointer in this case.
+
+                  In the rare cases where we cannot stabilize the renamed
+                  object, we just make a "bare" pointer, and the renamed
+                  entity is always accessed indirectly through it.  */
+               else
+                 {
+                   gnu_type = build_reference_type (gnu_type);
+                   inner_const_flag = TREE_READONLY (gnu_expr);
+                   const_flag = true;
+
+                   /* If the previous attempt at stabilizing failed, there
+                      is no point in trying again and we reuse the result
+                      without attaching it to the pointer.  In this case it
+                      will only be used as the initializing expression of
+                      the pointer and thus needs no special treatment with
+                      regard to multiple evaluations.  */
+                   if (maybe_stable_expr)
+                     ;
+
+                   /* Otherwise, try to stabilize and attach the expression
+                      to the pointer if the stabilization succeeds.
+
+                      Note that this might introduce SAVE_EXPRs and we don't
+                      check whether we're at the global level or not.  This
+                      is fine since we are building a pointer initializer and
+                      neither the pointer nor the initializing expression can
+                      be accessed before the pointer elaboration has taken
+                      place in a correct program.
+
+                      These SAVE_EXPRs will be evaluated at the right place
+                      by either the evaluation of the initializer for the
+                      non-global case or the elaboration code for the global
+                      case, and will be attached to the elaboration procedure
+                      in the latter case.  */
+                   else
+                    {
+                       maybe_stable_expr
+                         = maybe_stabilize_reference (gnu_expr, true, &stable);
+
+                       if (stable)
+                         renamed_obj = maybe_stable_expr;
+
+                       /* Attaching is actually performed downstream, as soon
+                          as we have a VAR_DECL for the pointer we make.  */
+                     }
+
+                   gnu_expr
+                     = build_unary_op (ADDR_EXPR, gnu_type, maybe_stable_expr);
+
+                   gnu_size = NULL_TREE;
+                   used_by_ref = true;
+                 }
              }
          }
 
@@ -811,21 +943,22 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           the object.  If there is an initializer, it will have already
           been converted to the right type, but we need to create the
           template if there is no initializer.  */
-       else if (definition && TREE_CODE (gnu_type) == RECORD_TYPE
+       else if (definition
+                && TREE_CODE (gnu_type) == RECORD_TYPE
                 && (TYPE_CONTAINS_TEMPLATE_P (gnu_type)
                     /* Beware that padding might have been introduced
                        via maybe_pad_type above.  */
                     || (TYPE_IS_PADDING_P (gnu_type)
                         && TREE_CODE (TREE_TYPE (TYPE_FIELDS (gnu_type)))
-                           == RECORD_TYPE
+                           == RECORD_TYPE
                         && TYPE_CONTAINS_TEMPLATE_P
-                           (TREE_TYPE (TYPE_FIELDS (gnu_type)))))
+                           (TREE_TYPE (TYPE_FIELDS (gnu_type)))))
                 && !gnu_expr)
          {
            tree template_field
              = TYPE_IS_PADDING_P (gnu_type)
-               ? TYPE_FIELDS (TREE_TYPE (TYPE_FIELDS (gnu_type)))
-               : TYPE_FIELDS (gnu_type);
+               ? TYPE_FIELDS (TREE_TYPE (TYPE_FIELDS (gnu_type)))
+               : TYPE_FIELDS (gnu_type);
 
            gnu_expr
              = gnat_build_constructor
@@ -838,12 +971,27 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                NULL_TREE));
          }
 
+       /* Convert the expression to the type of the object except in the
+          case where the object's type is unconstrained or the object's type
+          is a padded record whose field is of self-referential size.  In
+          the former case, converting will generate unnecessary evaluations
+          of the CONSTRUCTOR to compute the size and in the latter case, we
+          want to only copy the actual data.  */
+       if (gnu_expr
+           && TREE_CODE (gnu_type) != UNCONSTRAINED_ARRAY_TYPE
+           && !CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_type))
+           && !(TREE_CODE (gnu_type) == RECORD_TYPE
+                && TYPE_IS_PADDING_P (gnu_type)
+                && (CONTAINS_PLACEHOLDER_P
+                    (TYPE_SIZE (TREE_TYPE (TYPE_FIELDS (gnu_type)))))))
+         gnu_expr = convert (gnu_type, gnu_expr);
+
        /* If this is a pointer and it does not have an initializing
           expression, initialize it to NULL, unless the object is
-           imported.  */
+          imported.  */
        if (definition
            && (POINTER_TYPE_P (gnu_type) || TYPE_FAT_POINTER_P (gnu_type))
-            && !Is_Imported (gnat_entity) && !gnu_expr)
+           && !Is_Imported (gnat_entity) && !gnu_expr)
          gnu_expr = integer_zero_node;
 
        /* If we are defining the object and it has an Address clause we must
@@ -863,7 +1011,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            /* Ignore the size.  It's either meaningless or was handled
               above.  */
            gnu_size = NULL_TREE;
-           gnu_type = build_reference_type (gnu_type);
+           /* Convert the type of the object to a reference type that can
+              alias everything as per 13.3(19).  */
+           gnu_type
+             = build_reference_type_for_mode (gnu_type, ptr_mode, true);
            gnu_address = convert (gnu_type, gnu_address);
            used_by_ref = true;
            const_flag = !Is_Public (gnat_entity);
@@ -890,10 +1041,18 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           imported.  */
        if ((!definition && Present (Address_Clause (gnat_entity)))
            || (Is_Imported (gnat_entity)
-               && Convention (gnat_entity) == Convention_Stdcall))
+               && Has_Stdcall_Convention (gnat_entity)))
          {
-           gnu_type = build_reference_type (gnu_type);
+           /* Convert the type of the object to a reference type that can
+              alias everything as per 13.3(19).  */
+           gnu_type
+             = build_reference_type_for_mode (gnu_type, ptr_mode, true);
            gnu_size = NULL_TREE;
+
+           gnu_expr = NULL_TREE;
+           /* No point in taking the address of an initializing expression
+              that isn't going to be used.  */
+
            used_by_ref = true;
          }
 
@@ -928,7 +1087,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
               If we have a template initializer only (that we made above),
               pretend there is none and rely on what build_allocator creates
               again anyway.  Otherwise (if we have a full initializer), get
-              the data part and feed that to build_allocator.  */
+              the data part and feed that to build_allocator.
+
+              If we are elaborating a mutable object, tell build_allocator to
+              ignore a possibly simpler size from the initializer, if any, as
+              we must allocate the maximum possible size in this case.  */
 
            if (definition)
              {
@@ -941,25 +1104,25 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                      = TREE_TYPE (TREE_CHAIN (TYPE_FIELDS (gnu_alloc_type)));
 
                    if (TREE_CODE (gnu_expr) == CONSTRUCTOR
-                       && VEC_length (constructor_elt,
-                                      CONSTRUCTOR_ELTS (gnu_expr)) == 1)
+                       && 1 == VEC_length (constructor_elt,
+                                           CONSTRUCTOR_ELTS (gnu_expr)))
                      gnu_expr = 0;
                    else
                      gnu_expr
                        = build_component_ref
-                         (gnu_expr, NULL_TREE,
-                         TREE_CHAIN (TYPE_FIELDS (TREE_TYPE (gnu_expr))),
-                             false);
+                           (gnu_expr, NULL_TREE,
+                            TREE_CHAIN (TYPE_FIELDS (TREE_TYPE (gnu_expr))),
+                            false);
                  }
 
                if (TREE_CODE (TYPE_SIZE_UNIT (gnu_alloc_type)) == INTEGER_CST
-                   && TREE_CONSTANT_OVERFLOW (TYPE_SIZE_UNIT (gnu_alloc_type))
+                   && TREE_OVERFLOW (TYPE_SIZE_UNIT (gnu_alloc_type))
                    && !Is_Imported (gnat_entity))
-                 post_error ("Storage_Error will be raised at run-time?",
+                 post_error ("?Storage_Error will be raised at run-time!",
                              gnat_entity);
 
                gnu_expr = build_allocator (gnu_alloc_type, gnu_expr, gnu_type,
-                                           0, 0, gnat_entity, false);
+                                           0, 0, gnat_entity, mutable_p);
              }
            else
              {
@@ -968,24 +1131,24 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
              }
          }
 
-       /* If this object would go into the stack and has an alignment
-          larger than the default largest alignment, make a variable
-          to hold the "aligning type" with a modified initial value,
-          if any, then point to it and make that the value of this
-          variable, which is now indirect.  */
+       /* If this object would go into the stack and has an alignment larger
+          than the largest stack alignment the back-end can honor, resort to
+          a variable of "aligning type".  */
        if (!global_bindings_p () && !static_p && definition
            && !imported_p && TYPE_ALIGN (gnu_type) > BIGGEST_ALIGNMENT)
          {
+           /* Create the new variable.  No need for extra room before the
+              aligned field as this is in automatic storage.  */
            tree gnu_new_type
              = make_aligning_type (gnu_type, TYPE_ALIGN (gnu_type),
-                                   TYPE_SIZE_UNIT (gnu_type));
-           tree gnu_new_var;
-
-           gnu_new_var
+                                   TYPE_SIZE_UNIT (gnu_type),
+                                   BIGGEST_ALIGNMENT, 0);
+           tree gnu_new_var
              = create_var_decl (create_concat_name (gnat_entity, "ALIGN"),
-                                NULL_TREE, gnu_new_type, gnu_expr, false,
+                                NULL_TREE, gnu_new_type, NULL_TREE, false,
                                 false, false, false, NULL, gnat_entity);
 
+           /* Initialize the aligned field if we have an initializer.  */
            if (gnu_expr)
              add_stmt_with_node
                (build_binary_op (MODIFY_EXPR, NULL_TREE,
@@ -995,6 +1158,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                  gnu_expr),
                 gnat_entity);
 
+           /* And setup this entity as a reference to the aligned field.  */
            gnu_type = build_reference_type (gnu_type);
            gnu_expr
              = build_unary_op
@@ -1058,17 +1222,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        if (TREE_CODE (gnu_decl) == VAR_DECL && renamed_obj)
          {
            SET_DECL_RENAMED_OBJECT (gnu_decl, renamed_obj);
-           DECL_RENAMING_GLOBAL_P (gnu_decl) = global_bindings_p ();
+           if (global_bindings_p ())
+             {
+               DECL_RENAMING_GLOBAL_P (gnu_decl) = 1;
+               record_global_renaming_pointer (gnu_decl);
+             }
          }
 
-       /* If we have an address clause and we've made this indirect, it's
-          not enough to merely mark the type as volatile since volatile
-          references only conflict with other volatile references while this
-          reference must conflict with all other references.  So ensure that
-          the dereferenced value has alias set 0.  */
-       if (Present (Address_Clause (gnat_entity)) && used_by_ref)
-         DECL_POINTER_ALIAS_SET (gnu_decl) = 0;
-
        if (definition && DECL_SIZE (gnu_decl)
            && get_block_jmpbuf_decl ()
            && (TREE_CODE (DECL_SIZE (gnu_decl)) != INTEGER_CST
@@ -1082,10 +1242,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                              gnat_entity);
 
        /* If this is a public constant or we're not optimizing and we're not
-          making a VAR_DECL for it, make one just for export or debugger
-          use.  Likewise if the address is taken or if the object or type is
-          aliased.  */
-       if (definition && TREE_CODE (gnu_decl) == CONST_DECL
+          making a VAR_DECL for it, make one just for export or debugger use.
+          Likewise if the address is taken or if either the object or type is
+          aliased.  Make an external declaration for a reference, unless this
+          is a Standard entity since there no real symbol at the object level
+          for these.  */
+       if (TREE_CODE (gnu_decl) == CONST_DECL
+           && (definition || Sloc (gnat_entity) > Standard_Location)
            && (Is_Public (gnat_entity)
                || optimize == 0
                || Address_Taken (gnat_entity)
@@ -1093,39 +1256,48 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                || Is_Aliased (Etype (gnat_entity))))
          {
            tree gnu_corr_var
-             = create_var_decl (gnu_entity_id, gnu_ext_name, gnu_type,
-                                gnu_expr, false, Is_Public (gnat_entity),
-                                false, static_p, NULL, gnat_entity);
+             = create_true_var_decl (gnu_entity_id, gnu_ext_name, gnu_type,
+                                     gnu_expr, true, Is_Public (gnat_entity),
+                                     !definition, static_p, NULL,
+                                     gnat_entity);
 
            SET_DECL_CONST_CORRESPONDING_VAR (gnu_decl, gnu_corr_var);
          }
 
-       /* If this is declared in a block that contains an block with an
+       /* If this is declared in a block that contains a block with an
           exception handler, we must force this variable in memory to
           suppress an invalid optimization.  */
        if (Has_Nested_Block_With_Handler (Scope (gnat_entity))
-           && Exception_Mechanism != GCC_ZCX)
+           && Exception_Mechanism != Back_End_Exceptions)
          TREE_ADDRESSABLE (gnu_decl) = 1;
 
-       /* Back-annotate the Alignment of the object if not already in the
-          tree.  Likewise for Esize if the object is of a constant size.
-          But if the "object" is actually a pointer to an object, the
-          alignment and size are the same as the type, so don't back-annotate
-          the values for the pointer.  */
+       gnu_type = TREE_TYPE (gnu_decl);
+
+       /* Back-annotate Alignment and Esize of the object if not already
+          known, except for when the object is actually a pointer to the
+          real object, since alignment and size of a pointer don't have
+          anything to do with those of the designated object.  Note that
+          we pick the values of the type, not those of the object, to
+          shield ourselves from low-level platform-dependent adjustments
+          like alignment promotion.  This is both consistent with all the
+          treatment above, where alignment and size are set on the type of
+          the object and not on the object directly, and makes it possible
+          to support confirming representation clauses in all cases.  */
+
        if (!used_by_ref && Unknown_Alignment (gnat_entity))
          Set_Alignment (gnat_entity,
-                        UI_From_Int (DECL_ALIGN (gnu_decl) / BITS_PER_UNIT));
+                        UI_From_Int (TYPE_ALIGN (gnu_type) / BITS_PER_UNIT));
 
-       if (!used_by_ref && Unknown_Esize (gnat_entity)
-           && DECL_SIZE (gnu_decl))
+       if (!used_by_ref && Unknown_Esize (gnat_entity))
          {
-           tree gnu_back_size = DECL_SIZE (gnu_decl);
+           tree gnu_back_size;
 
-           if (TREE_CODE (TREE_TYPE (gnu_decl)) == RECORD_TYPE
-               && TYPE_CONTAINS_TEMPLATE_P (TREE_TYPE (gnu_decl)))
+           if (TREE_CODE (gnu_type) == RECORD_TYPE
+               && TYPE_CONTAINS_TEMPLATE_P (gnu_type))
              gnu_back_size
-               = TYPE_SIZE (TREE_TYPE (TREE_CHAIN
-                                       (TYPE_FIELDS (TREE_TYPE (gnu_decl)))));
+               = TYPE_SIZE (TREE_TYPE (TREE_CHAIN (TYPE_FIELDS (gnu_type))));
+            else
+             gnu_back_size = TYPE_SIZE (gnu_type);
 
            Set_Esize (gnat_entity, annotate_value (gnu_back_size));
          }
@@ -1139,11 +1311,17 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
     case E_Enumeration_Type:
       /* A special case, for the types Character and Wide_Character in
-         Standard, we do not list all the literals. So if the literals
-         are not specified, make this an unsigned type.  */
+        Standard, we do not list all the literals. So if the literals
+        are not specified, make this an unsigned type.  */
       if (No (First_Literal (gnat_entity)))
        {
          gnu_type = make_unsigned_type (esize);
+         TYPE_NAME (gnu_type) = gnu_entity_id;
+
+         /* Set the TYPE_STRING_FLAG for Ada Character and
+            Wide_Character types. This is needed by the dwarf-2 debug writer to
+            distinguish between unsigned integer types and character types.  */
+         TYPE_STRING_FLAG (gnu_type) = 1;
          break;
        }
 
@@ -1222,7 +1400,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
        gnu_type = make_unsigned_type (GET_MODE_BITSIZE (mode));
        TYPE_PACKED_ARRAY_TYPE_P (gnu_type)
-         = Is_Packed_Array_Type (gnat_entity);
+         = (Is_Packed_Array_Type (gnat_entity)
+            && Is_Bit_Packed_Array (Original_Array_Type (gnat_entity)));
 
        /* Get the modulus in this type.  If it overflows, assume it is because
           it is equal to 2**Esize.  Note that there is no overflow checking
@@ -1234,8 +1413,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          {
            TYPE_MODULAR_P (gnu_type) = 1;
            SET_TYPE_MODULUS (gnu_type, gnu_modulus);
-           gnu_high = fold (build2 (MINUS_EXPR, gnu_type, gnu_modulus,
-                                    convert (gnu_type, integer_one_node)));
+           gnu_high = fold_build2 (MINUS_EXPR, gnu_type, gnu_modulus,
+                                   convert (gnu_type, integer_one_node));
          }
 
        /* If we have to set TYPE_PRECISION different from its natural value,
@@ -1257,7 +1436,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            TYPE_UNSIGNED (gnu_subtype) = 1;
            TYPE_EXTRA_SUBTYPE_P (gnu_subtype) = 1;
            TYPE_PACKED_ARRAY_TYPE_P (gnu_subtype)
-             = Is_Packed_Array_Type (gnat_entity);
+             = (Is_Packed_Array_Type (gnat_entity)
+                && Is_Bit_Packed_Array (Original_Array_Type (gnat_entity)));
            layout_type (gnu_subtype);
 
            gnu_type = gnu_subtype;
@@ -1286,16 +1466,17 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
         bounds by always elaborating the first such subtype first, thus
         using its name. */
 
-      if (definition == 0
+      if (!definition
          && Present (Ancestor_Subtype (gnat_entity))
          && !In_Extended_Main_Code_Unit (Ancestor_Subtype (gnat_entity))
          && (!Compile_Time_Known_Value (Type_Low_Bound (gnat_entity))
              || !Compile_Time_Known_Value (Type_High_Bound (gnat_entity))))
        gnat_to_gnu_entity (Ancestor_Subtype (gnat_entity),
-                           gnu_expr, definition);
+                           gnu_expr, 0);
 
       gnu_type = make_node (INTEGER_TYPE);
-      if (Is_Packed_Array_Type (gnat_entity))
+      if (Is_Packed_Array_Type (gnat_entity)
+         && Is_Bit_Packed_Array (Original_Array_Type (gnat_entity)))
        {
          esize = UI_To_Int (RM_Size (gnat_entity));
          TYPE_PACKED_ARRAY_TYPE_P (gnu_type) = 1;
@@ -1341,6 +1522,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
       layout_type (gnu_type);
 
+      /* Inherit our alias set from what we're a subtype of.  Subtypes
+        are not different types and a pointer can designate any instance
+        within a subtype hierarchy.  */
+      copy_alias_set (gnu_type, TREE_TYPE (gnu_type));
+
       /* If the type we are dealing with is to represent a packed array,
         we need to have the bits left justified on big-endian targets
         and right justified on little-endian targets.  We also need to
@@ -1348,7 +1534,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
         such values), we only get the good bits, since the unused bits
         are uninitialized.  Both goals are accomplished by wrapping the
         modular value in an enclosing struct.  */
-       if (Is_Packed_Array_Type (gnat_entity))
+      if (Is_Packed_Array_Type (gnat_entity)
+           && Is_Bit_Packed_Array (Original_Array_Type (gnat_entity)))
        {
          tree gnu_field_type = gnu_type;
          tree gnu_field;
@@ -1357,6 +1544,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            = UI_To_gnu (RM_Size (gnat_entity), bitsizetype);
          gnu_type = make_node (RECORD_TYPE);
          TYPE_NAME (gnu_type) = create_concat_name (gnat_entity, "JM");
+
+         /* Propagate the alignment of the modular type to the record.
+            This means that bitpacked arrays have "ceil" alignment for
+            their size, which may seem counter-intuitive but makes it
+            possible to easily overlay them on modular types.  */
          TYPE_ALIGN (gnu_type) = TYPE_ALIGN (gnu_field_type);
          TYPE_PACKED (gnu_type) = 1;
 
@@ -1371,11 +1563,52 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          gnu_field = create_field_decl (get_identifier ("OBJECT"),
                                         gnu_field_type, gnu_type, 1, 0, 0, 0);
 
-         finish_record_type (gnu_type, gnu_field, false, false);
+         finish_record_type (gnu_type, gnu_field, 0, false);
          TYPE_JUSTIFIED_MODULAR_P (gnu_type) = 1;
          SET_TYPE_ADA_SIZE (gnu_type, bitsize_int (esize));
+
+         copy_alias_set (gnu_type, gnu_field_type);
+       }
+
+      /* If the type we are dealing with has got a smaller alignment than the
+        natural one, we need to wrap it up in a record type and under-align
+        the latter.  We reuse the padding machinery for this purpose.  */
+      else if (Known_Alignment (gnat_entity)
+              && UI_Is_In_Int_Range (Alignment (gnat_entity))
+              && (align = UI_To_Int (Alignment (gnat_entity)) * BITS_PER_UNIT)
+              && align < TYPE_ALIGN (gnu_type))
+       {
+         tree gnu_field_type = gnu_type;
+         tree gnu_field;
+
+         gnu_type = make_node (RECORD_TYPE);
+         TYPE_NAME (gnu_type) = create_concat_name (gnat_entity, "PAD");
+
+         TYPE_ALIGN (gnu_type) = align;
+         TYPE_PACKED (gnu_type) = 1;
+
+         /* Create a stripped-down declaration of the original type, mainly
+            for debugging.  */
+         create_type_decl (get_entity_name (gnat_entity), gnu_field_type,
+                           NULL, true, debug_info_p, gnat_entity);
+
+         /* Don't notify the field as "addressable", since we won't be taking
+            it's address and it would prevent create_field_decl from making a
+            bitfield.  */
+         gnu_field = create_field_decl (get_identifier ("OBJECT"),
+                                        gnu_field_type, gnu_type, 1, 0, 0, 0);
+
+         finish_record_type (gnu_type, gnu_field, 0, false);
+         TYPE_IS_PADDING_P (gnu_type) = 1;
+         SET_TYPE_ADA_SIZE (gnu_type, bitsize_int (esize));
+
+         copy_alias_set (gnu_type, gnu_field_type);
        }
 
+      /* Otherwise reset the alignment lest we computed it above.  */
+      else
+       align = 0;
+
       break;
 
     case E_Floating_Point_Type:
@@ -1406,13 +1639,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        }
 
       {
-       if (definition == 0
+       if (!definition
            && Present (Ancestor_Subtype (gnat_entity))
            && !In_Extended_Main_Code_Unit (Ancestor_Subtype (gnat_entity))
            && (!Compile_Time_Known_Value (Type_Low_Bound (gnat_entity))
                || !Compile_Time_Known_Value (Type_High_Bound (gnat_entity))))
          gnat_to_gnu_entity (Ancestor_Subtype (gnat_entity),
-                             gnu_expr, definition);
+                             gnu_expr, 0);
 
        gnu_type = make_node (REAL_TYPE);
        TREE_TYPE (gnu_type) = get_unpadded_type (Etype (gnat_entity));
@@ -1441,6 +1674,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          }
 
        layout_type (gnu_type);
+
+       /* Inherit our alias set from what we're a subtype of, as for
+          integer subtypes.  */
+       copy_alias_set (gnu_type, TREE_TYPE (gnu_type));
       }
     break;
 
@@ -1469,12 +1706,12 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          = (Convention (gnat_entity) == Convention_Fortran) ? ndim - 1 : 0;
        int nextdim
          = (Convention (gnat_entity) == Convention_Fortran) ? - 1 : 1;
+       int index;
        tree *gnu_index_types = (tree *) alloca (ndim * sizeof (tree *));
        tree *gnu_temp_fields = (tree *) alloca (ndim * sizeof (tree *));
        tree gnu_comp_size = 0;
        tree gnu_max_size = size_one_node;
        tree gnu_max_size_unit;
-       int index;
        Entity_Id gnat_ind_subtype;
        Entity_Id gnat_ind_base_subtype;
        tree gnu_template_reference;
@@ -1482,24 +1719,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
        TYPE_NAME (gnu_template_type)
          = create_concat_name (gnat_entity, "XUB");
-       TYPE_NAME (gnu_fat_type) = create_concat_name (gnat_entity, "XUP");
-       TYPE_IS_FAT_POINTER_P (gnu_fat_type) = 1;
-       TYPE_READONLY (gnu_template_type) = 1;
 
        /* Make a node for the array.  If we are not defining the array
-          suppress expanding incomplete types and save the node as the type
-          for GNAT_ENTITY.  */
+          suppress expanding incomplete types.  */
        gnu_type = make_node (UNCONSTRAINED_ARRAY_TYPE);
+
        if (!definition)
-         {
-           defer_incomplete_level++;
-           this_deferred = this_made_decl = true;
-           gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
-                                        !Comes_From_Source (gnat_entity),
-                                        debug_info_p, gnat_entity);
-           save_gnu_tree (gnat_entity, gnu_decl, false);
-           saved = true;
-         }
+         defer_incomplete_level++, this_deferred = true;
 
        /* Build the fat pointer type.  Use a "void *" object instead of
           a pointer to the array type since we don't have the array type
@@ -1514,7 +1740,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
        /* Make sure we can put this into a register.  */
        TYPE_ALIGN (gnu_fat_type) = MIN (BIGGEST_ALIGNMENT, 2 * POINTER_SIZE);
-       finish_record_type (gnu_fat_type, tem, false, true);
+
+       /* Do not finalize this record type since the types of its fields
+          are still incomplete at this point.  */
+       finish_record_type (gnu_fat_type, tem, 0, true);
+       TYPE_IS_FAT_POINTER_P (gnu_fat_type) = 1;
 
        /* Build a reference to the template from a PLACEHOLDER_EXPR that
           is the fat pointer.  This will be used to access the individual
@@ -1581,7 +1811,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
              = create_index_type (convert (sizetype, gnu_min),
                                   convert (sizetype, gnu_max),
                                   build_range_type (gnu_ind_subtype,
-                                                    gnu_min, gnu_max));
+                                                    gnu_min, gnu_max),
+                                  gnat_entity);
            /* Update the maximum size of the array, in elements. */
            gnu_max_size
              = size_binop (MULT_EXPR, gnu_max_size,
@@ -1598,15 +1829,26 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            = chainon (gnu_template_fields, gnu_temp_fields[index]);
 
        /* Install all the fields into the template.  */
-       finish_record_type (gnu_template_type, gnu_template_fields,
-                           false, false);
+       finish_record_type (gnu_template_type, gnu_template_fields, 0, false);
        TYPE_READONLY (gnu_template_type) = 1;
 
        /* Now make the array of arrays and update the pointer to the array
           in the fat pointer.  Note that it is the first field.  */
-
        tem = gnat_to_gnu_type (Component_Type (gnat_entity));
 
+       /* Try to get a smaller form of the component if needed.  */
+       if ((Is_Packed (gnat_entity)
+            || Has_Component_Size_Clause (gnat_entity))
+           && !Is_Bit_Packed_Array (gnat_entity)
+           && !Has_Aliased_Components (gnat_entity)
+           && !Strict_Alignment (Component_Type (gnat_entity))
+           && TREE_CODE (tem) == RECORD_TYPE
+           && host_integerp (TYPE_SIZE (tem), 1))
+         tem = make_packable_type (tem, false);
+
+       if (Has_Atomic_Components (gnat_entity))
+         check_ok_for_atomic (tem, gnat_entity, true);
+
        /* Get and validate any specified Component_Size, but if Packed,
           ignore it since the front end will have taken care of it. */
        gnu_comp_size
@@ -1616,20 +1858,25 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                            ? TYPE_DECL : VAR_DECL),
                           true, Has_Component_Size_Clause (gnat_entity));
 
-       if (Has_Atomic_Components (gnat_entity))
-         check_ok_for_atomic (tem, gnat_entity, true);
-
        /* If the component type is a RECORD_TYPE that has a self-referential
           size, use the maxium size.  */
        if (!gnu_comp_size && TREE_CODE (tem) == RECORD_TYPE
            && CONTAINS_PLACEHOLDER_P (TYPE_SIZE (tem)))
          gnu_comp_size = max_size (TYPE_SIZE (tem), true);
 
-       if (!Is_Bit_Packed_Array (gnat_entity) && gnu_comp_size)
+       if (gnu_comp_size && !Is_Bit_Packed_Array (gnat_entity))
          {
+           tree orig_tem;
            tem = make_type_from_size (tem, gnu_comp_size, false);
+           orig_tem = tem;
            tem = maybe_pad_type (tem, gnu_comp_size, 0, gnat_entity,
                                  "C_PAD", false, definition, true);
+           /* If a padding record was made, declare it now since it will
+              never be declared otherwise.  This is necessary to ensure
+              that its subtrees are properly marked.  */
+           if (tem != orig_tem)
+             create_type_decl (TYPE_NAME (tem), tem, NULL, true, false,
+                               gnat_entity);
          }
 
        if (Has_Volatile_Components (gnat_entity))
@@ -1654,30 +1901,23 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          {
            tem = build_array_type (tem, gnu_index_types[index]);
            TYPE_MULTI_ARRAY_P (tem) = (index > 0);
-
-           /* If the type below this an multi-array type, then this
-              does not not have aliased components.
-
-              ??? Otherwise, for now, we say that any component of aggregate
-              type is addressable because the front end may take 'Reference
-              of it. But we have to make it addressable if it must be passed
-              by reference or it that is the default.  */
-           TYPE_NONALIASED_COMPONENT (tem)
-             = ((TREE_CODE (TREE_TYPE (tem)) == ARRAY_TYPE
-                 && TYPE_MULTI_ARRAY_P (TREE_TYPE (tem))) ? 1
-                : (!Has_Aliased_Components (gnat_entity)
-                   && !AGGREGATE_TYPE_P (TREE_TYPE (tem))));
+           if (array_type_has_nonaliased_component (gnat_entity, tem))
+             TYPE_NONALIASED_COMPONENT (tem) = 1;
          }
 
        /* If an alignment is specified, use it if valid.  But ignore it for
-          types that represent the unpacked base type for packed arrays.  */
+          types that represent the unpacked base type for packed arrays.  If
+          the alignment was requested with an explicit user alignment clause,
+          state so.  */
        if (No (Packed_Array_Type (gnat_entity))
-            && Known_Alignment (gnat_entity))
+           && Known_Alignment (gnat_entity))
          {
            gcc_assert (Present (Alignment (gnat_entity)));
            TYPE_ALIGN (tem)
              = validate_alignment (Alignment (gnat_entity), gnat_entity,
                                    TYPE_ALIGN (tem));
+           if (Present (Alignment_Clause (gnat_entity)))
+             TYPE_USER_ALIGN (tem) = 1;
          }
 
        TYPE_CONVENTION_FORTRAN_P (tem)
@@ -1707,16 +1947,18 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                          tem, NULL, !Comes_From_Source (gnat_entity),
                          debug_info_p, gnat_entity);
 
-       /* Create a record type for the object and its template and
-          set the template at a negative offset.  */
+       /* Give the fat pointer type a name.  */
+       create_type_decl (create_concat_name (gnat_entity, "XUP"),
+                         gnu_fat_type, NULL, !Comes_From_Source (gnat_entity),
+                         debug_info_p, gnat_entity);
+
+       /* Create the type to be used as what a thin pointer designates: an
+          record type for the object and its template with the field offsets
+          shifted to have the template at a negative offset.  */
        tem = build_unc_object_type (gnu_template_type, tem,
                                     create_concat_name (gnat_entity, "XUT"));
-       DECL_FIELD_OFFSET (TYPE_FIELDS (tem))
-         = size_binop (MINUS_EXPR, size_zero_node,
-                       byte_position (TREE_CHAIN (TYPE_FIELDS (tem))));
-       DECL_FIELD_OFFSET (TREE_CHAIN (TYPE_FIELDS (tem))) = size_zero_node;
-       DECL_FIELD_BIT_OFFSET (TREE_CHAIN (TYPE_FIELDS (tem)))
-         = bitsize_zero_node;
+       shift_unc_components_for_thin_pointers (tem);
+
        SET_TYPE_UNCONSTRAINED_ARRAY (tem, gnu_type);
        TYPE_OBJECT_RECORD_TYPE (gnu_type) = tem;
 
@@ -1772,7 +2014,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
          for (index = first_dim, gnat_ind_subtype = First_Index (gnat_entity),
               gnat_ind_base_subtype
-                = First_Index (Implementation_Base_Type (gnat_entity));
+                = First_Index (Implementation_Base_Type (gnat_entity));
               index < array_dim && index >= 0;
               index += next_dim,
               gnat_ind_subtype = Next_Index (gnat_ind_subtype),
@@ -1810,19 +2052,20 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                  && TREE_CODE (gnu_max) == INTEGER_CST
                  && TREE_OVERFLOW (gnu_min) && TREE_OVERFLOW (gnu_max)
                  && (!TREE_OVERFLOW
-                     (fold (build2 (MINUS_EXPR, gnu_index_subtype,
-                                    TYPE_MAX_VALUE (gnu_index_subtype),
-                                    TYPE_MIN_VALUE (gnu_index_subtype))))))
-               TREE_OVERFLOW (gnu_min) = TREE_OVERFLOW (gnu_max)
-                 = TREE_CONSTANT_OVERFLOW (gnu_min)
-                 = TREE_CONSTANT_OVERFLOW (gnu_max) = 0;
+                     (fold_build2 (MINUS_EXPR, gnu_index_subtype,
+                                   TYPE_MAX_VALUE (gnu_index_subtype),
+                                   TYPE_MIN_VALUE (gnu_index_subtype)))))
+               {
+                 TREE_OVERFLOW (gnu_min) = 0;
+                 TREE_OVERFLOW (gnu_max) = 0;
+               }
 
              /* Similarly, if the range is null, use bounds of 1..0 for
                 the sizetype bounds.  */
              else if ((TYPE_PRECISION (gnu_index_subtype)
                        > TYPE_PRECISION (sizetype)
                       || TYPE_UNSIGNED (gnu_index_subtype)
-                         != TYPE_UNSIGNED (sizetype))
+                         != TYPE_UNSIGNED (sizetype))
                       && TREE_CODE (gnu_min) == INTEGER_CST
                       && TREE_CODE (gnu_max) == INTEGER_CST
                       && (TREE_OVERFLOW (gnu_min) || TREE_OVERFLOW (gnu_max))
@@ -1845,13 +2088,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                 code below to malfunction if we don't handle it specially.  */
              if (TREE_CODE (gnu_base_min) == INTEGER_CST
                  && TREE_CODE (gnu_base_max) == INTEGER_CST
-                 && !TREE_CONSTANT_OVERFLOW (gnu_base_min)
-                 && !TREE_CONSTANT_OVERFLOW (gnu_base_max)
+                 && !TREE_OVERFLOW (gnu_base_min)
+                 && !TREE_OVERFLOW (gnu_base_max)
                  && tree_int_cst_lt (gnu_base_max, gnu_base_min))
                gnu_high = size_zero_node, gnu_min = size_one_node;
 
              /* If gnu_high is now an integer which overflowed, the array
-                 cannot be superflat.  */
+                cannot be superflat.  */
              else if (TREE_CODE (gnu_high) == INTEGER_CST
                       && TREE_OVERFLOW (gnu_high))
                gnu_high = gnu_max;
@@ -1866,7 +2109,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                     gnu_max, gnu_high);
 
              gnu_index_type[index]
-               = create_index_type (gnu_min, gnu_high, gnu_index_subtype);
+               = create_index_type (gnu_min, gnu_high, gnu_index_subtype,
+                                    gnat_entity);
 
              /* Also compute the maximum size of the array.  Here we
                 see if any constraint on the index type of the base type
@@ -1878,20 +2122,24 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
              if ((TREE_CODE (gnu_min) == INTEGER_CST
                   && !TREE_OVERFLOW (gnu_min)
                   && !operand_equal_p (gnu_min, gnu_base_base_min, 0))
-                 || !CONTAINS_PLACEHOLDER_P (gnu_min))
+                 || !CONTAINS_PLACEHOLDER_P (gnu_min)
+                 || !(TREE_CODE (gnu_base_min) == INTEGER_CST
+                      && !TREE_OVERFLOW (gnu_base_min)))
                gnu_base_min = gnu_min;
 
              if ((TREE_CODE (gnu_max) == INTEGER_CST
                   && !TREE_OVERFLOW (gnu_max)
                   && !operand_equal_p (gnu_max, gnu_base_base_max, 0))
-                 || !CONTAINS_PLACEHOLDER_P (gnu_max))
+                 || !CONTAINS_PLACEHOLDER_P (gnu_max)
+                 || !(TREE_CODE (gnu_base_max) == INTEGER_CST
+                      && !TREE_OVERFLOW (gnu_base_max)))
                gnu_base_max = gnu_max;
 
              if ((TREE_CODE (gnu_base_min) == INTEGER_CST
-                  && TREE_CONSTANT_OVERFLOW (gnu_base_min))
+                  && TREE_OVERFLOW (gnu_base_min))
                  || operand_equal_p (gnu_base_min, gnu_base_base_min, 0)
                  || (TREE_CODE (gnu_base_max) == INTEGER_CST
-                     && TREE_CONSTANT_OVERFLOW (gnu_base_max))
+                     && TREE_OVERFLOW (gnu_base_max))
                  || operand_equal_p (gnu_base_max, gnu_base_base_max, 0))
                max_overflow = true;
 
@@ -1906,7 +2154,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                              size_zero_node);
 
              if (TREE_CODE (gnu_this_max) == INTEGER_CST
-                 && TREE_CONSTANT_OVERFLOW (gnu_this_max))
+                 && TREE_OVERFLOW (gnu_this_max))
                max_overflow = true;
 
              gnu_max_size
@@ -1925,45 +2173,85 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                need_index_type_struct = true;
            }
 
-         /* Then flatten: create the array of arrays.  */
-
-         gnu_type = gnat_to_gnu_type (Component_Type (gnat_entity));
-
-         /* One of the above calls might have caused us to be elaborated,
-            so don't blow up if so.  */
-         if (present_gnu_tree (gnat_entity))
+         /* Then flatten: create the array of arrays.  For an array type
+            used to implement a packed array, get the component type from
+            the original array type since the representation clauses that
+            can affect it are on the latter.  */
+         if (Is_Packed_Array_Type (gnat_entity)
+             && !Is_Bit_Packed_Array (Original_Array_Type (gnat_entity)))
            {
-             maybe_present = true;
-             break;
-           }
+             gnu_type = gnat_to_gnu_type (Original_Array_Type (gnat_entity));
+             for (index = array_dim - 1; index >= 0; index--)
+               gnu_type = TREE_TYPE (gnu_type);
 
-         /* Get and validate any specified Component_Size, but if Packed,
-            ignore it since the front end will have taken care of it. */
-         gnu_comp_size
-           = validate_size (Component_Size (gnat_entity), gnu_type,
-                            gnat_entity,
-                            (Is_Bit_Packed_Array (gnat_entity)
-                             ? TYPE_DECL : VAR_DECL),
-                            true, Has_Component_Size_Clause (gnat_entity));
-
-         /* If the component type is a RECORD_TYPE that has a self-referential
-            size, use the maxium size.  */
-         if (!gnu_comp_size && TREE_CODE (gnu_type) == RECORD_TYPE
-             && CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_type)))
-           gnu_comp_size = max_size (TYPE_SIZE (gnu_type), true);
-
-         if (!Is_Bit_Packed_Array (gnat_entity) && gnu_comp_size)
-           {
-             gnu_type = make_type_from_size (gnu_type, gnu_comp_size, false);
-             gnu_type = maybe_pad_type (gnu_type, gnu_comp_size, 0,
-                                        gnat_entity, "C_PAD", false,
-                                        definition, true);
+             /* One of the above calls might have caused us to be elaborated,
+                so don't blow up if so.  */
+             if (present_gnu_tree (gnat_entity))
+               {
+                 maybe_present = true;
+                 break;
+               }
            }
+         else
+           {
+             gnu_type = gnat_to_gnu_type (Component_Type (gnat_entity));
+
+             /* One of the above calls might have caused us to be elaborated,
+                so don't blow up if so.  */
+             if (present_gnu_tree (gnat_entity))
+               {
+                 maybe_present = true;
+                 break;
+               }
+
+             /* Try to get a smaller form of the component if needed.  */
+             if ((Is_Packed (gnat_entity)
+                  || Has_Component_Size_Clause (gnat_entity))
+                 && !Is_Bit_Packed_Array (gnat_entity)
+                 && !Has_Aliased_Components (gnat_entity)
+                 && !Strict_Alignment (Component_Type (gnat_entity))
+                 && TREE_CODE (gnu_type) == RECORD_TYPE
+                 && host_integerp (TYPE_SIZE (gnu_type), 1))
+               gnu_type = make_packable_type (gnu_type, false);
+
+             /* Get and validate any specified Component_Size, but if Packed,
+                ignore it since the front end will have taken care of it. */
+             gnu_comp_size
+               = validate_size (Component_Size (gnat_entity), gnu_type,
+                                gnat_entity,
+                                (Is_Bit_Packed_Array (gnat_entity)
+                                 ? TYPE_DECL : VAR_DECL), true,
+                                Has_Component_Size_Clause (gnat_entity));
+
+             /* If the component type is a RECORD_TYPE that has a
+                self-referential size, use the maxium size.  */
+             if (!gnu_comp_size
+                 && TREE_CODE (gnu_type) == RECORD_TYPE
+                 && CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_type)))
+               gnu_comp_size = max_size (TYPE_SIZE (gnu_type), true);
+
+             if (gnu_comp_size && !Is_Bit_Packed_Array (gnat_entity))
+               {
+                 tree orig_gnu_type;
+                 gnu_type
+                   = make_type_from_size (gnu_type, gnu_comp_size, false);
+                 orig_gnu_type = gnu_type;
+                 gnu_type = maybe_pad_type (gnu_type, gnu_comp_size, 0,
+                                            gnat_entity, "C_PAD", false,
+                                            definition, true);
+                 /* If a padding record was made, declare it now since it
+                    will never be declared otherwise.  This is necessary
+                    to ensure that its subtrees are properly marked.  */
+                 if (gnu_type != orig_gnu_type)
+                   create_type_decl (TYPE_NAME (gnu_type), gnu_type, NULL,
+                                     true, false, gnat_entity);
+               }
 
-         if (Has_Volatile_Components (Base_Type (gnat_entity)))
-           gnu_type = build_qualified_type (gnu_type,
-                                            (TYPE_QUALS (gnu_type)
-                                             | TYPE_QUAL_VOLATILE));
+             if (Has_Volatile_Components (Base_Type (gnat_entity)))
+               gnu_type = build_qualified_type (gnu_type,
+                                                (TYPE_QUALS (gnu_type)
+                                                 | TYPE_QUAL_VOLATILE));
+           }
 
          gnu_max_size_unit = size_binop (MULT_EXPR, gnu_max_size,
                                          TYPE_SIZE_UNIT (gnu_type));
@@ -1975,18 +2263,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            {
              gnu_type = build_array_type (gnu_type, gnu_index_type[index]);
              TYPE_MULTI_ARRAY_P (gnu_type) = (index > 0);
-           /* If the type below this an multi-array type, then this
-              does not not have aliased components.
-
-              ??? Otherwise, for now, we say that any component of aggregate
-              type is addressable because the front end may take 'Reference
-              of it. But we have to make it addressable if it must be passed
-              by reference or it that is the default.  */
-             TYPE_NONALIASED_COMPONENT (gnu_type)
-             = ((TREE_CODE (TREE_TYPE (gnu_type)) == ARRAY_TYPE
-                 && TYPE_MULTI_ARRAY_P (TREE_TYPE (gnu_type))) ? 1
-                : (!Has_Aliased_Components (gnat_entity)
-                   && !AGGREGATE_TYPE_P (TREE_TYPE (gnu_type))));
+             if (array_type_has_nonaliased_component (gnat_entity, gnu_type))
+               TYPE_NONALIASED_COMPONENT (gnu_type) = 1;
            }
 
          /* If we are at file level and this is a multi-dimensional array, we
@@ -2024,6 +2302,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                        concat_id_with_name (gnu_str_name, "A_U"),
                        definition, 0),
                       size_int (TYPE_ALIGN (eltype) / BITS_PER_UNIT));
+
+                 /* ??? create_type_decl is not invoked on the inner types so
+                    the MULT_EXPR node built above will never be marked.  */
+                 TREE_VISITED (TYPE_SIZE_UNIT (gnu_arr_type)) = 1;
                }
            }
 
@@ -2055,13 +2337,14 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                }
 
              finish_record_type (gnu_bound_rec_type, gnu_field_list,
-                                 false, false);
+                                 0, false);
            }
 
          TYPE_CONVENTION_FORTRAN_P (gnu_type)
            = (Convention (gnat_entity) == Convention_Fortran);
          TYPE_PACKED_ARRAY_TYPE_P (gnu_type)
-           = Is_Packed_Array_Type (gnat_entity);
+           = (Is_Packed_Array_Type (gnat_entity)
+              && Is_Bit_Packed_Array (Original_Array_Type (gnat_entity)));
 
          /* If our size depends on a placeholder and the maximum size doesn't
             overflow, use it.  */
@@ -2112,9 +2395,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          gnu_decl = gnat_to_gnu_entity (Packed_Array_Type (gnat_entity),
                                         NULL_TREE, 0);
          this_made_decl = true;
-         gnu_inner_type = gnu_type = TREE_TYPE (gnu_decl);
+         gnu_type = TREE_TYPE (gnu_decl);
          save_gnu_tree (gnat_entity, NULL_TREE, false);
 
+         gnu_inner_type = gnu_type;
          while (TREE_CODE (gnu_inner_type) == RECORD_TYPE
                 && (TYPE_JUSTIFIED_MODULAR_P (gnu_inner_type)
                     || TYPE_IS_PADDING_P (gnu_inner_type)))
@@ -2186,7 +2470,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
     case E_String_Literal_Subtype:
       /* Create the type for a string literal. */
       {
-        Entity_Id gnat_full_type
+       Entity_Id gnat_full_type
          = (IN (Ekind (Etype (gnat_entity)), Private_Kind)
             && Present (Full_View (Etype (gnat_entity)))
             ? Full_View (Etype (gnat_entity)) : Etype (gnat_entity));
@@ -2213,7 +2497,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                        TYPE_MIN_VALUE (gnu_range_type)),
                               convert (sizetype,
                                        TYPE_MAX_VALUE (gnu_range_type)),
-                              gnu_range_type);
+                              gnu_range_type, gnat_entity);
 
        gnu_type
          = build_array_type (gnat_to_gnu_type (Component_Type (gnat_entity)),
@@ -2227,7 +2511,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        The following fields are defined on record types:
 
                Has_Discriminants       True if the record has discriminants
-                First_Discriminant      Points to head of list of discriminants
+               First_Discriminant      Points to head of list of discriminants
                First_Entity            Points to head of list of fields
                Is_Tagged_Type          True if the record is tagged
 
@@ -2263,16 +2547,23 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        }
 
       {
-        Node_Id full_definition = Declaration_Node (gnat_entity);
-        Node_Id record_definition = Type_Definition (full_definition);
+       Node_Id full_definition = Declaration_Node (gnat_entity);
+       Node_Id record_definition = Type_Definition (full_definition);
        Entity_Id gnat_field;
-        tree gnu_field;
+       tree gnu_field;
        tree gnu_field_list = NULL_TREE;
        tree gnu_get_parent;
-       int packed = (Is_Packed (gnat_entity) ? 1
-                     : (Component_Alignment (gnat_entity)
-                        == Calign_Storage_Unit) ? -1
-                     : 0);
+       /* Set PACKED in keeping with gnat_to_gnu_field.  */
+       int packed
+         = Is_Packed (gnat_entity)
+           ? 1
+           : Component_Alignment (gnat_entity) == Calign_Storage_Unit
+             ? -1
+             : (Known_Alignment (gnat_entity)
+                || (Strict_Alignment (gnat_entity)
+                    && Known_Static_Esize (gnat_entity)))
+               ? -2
+               : 0;
        bool has_rep = Has_Specified_Layout (gnat_entity);
        bool all_rep = has_rep;
        bool is_extension
@@ -2302,40 +2593,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          }
 
        /* Make a node for the record.  If we are not defining the record,
-          suppress expanding incomplete types and save the node as the type
-          for GNAT_ENTITY.  We use the same RECORD_TYPE as for a dummy type
-          and reset TYPE_DUMMY_P to show it's no longer a dummy.
-
-          It is very tempting to delay resetting this bit until we are done
-          with completing the type, e.g. to let possible intermediate
-          elaboration of access types designating the record know it is not
-          complete and arrange for update_pointer_to to fix things up later.
-
-          It would be wrong, however, because dummy types are expected only
-          to be created for Ada incomplete or private types, which is not
-          what we have here.  Doing so would make other parts of gigi think
-          we are dealing with a really incomplete or private type, and have
-          nasty side effects, typically on the generation of the associated
-          debugging information.  */
-       gnu_type = make_dummy_type (gnat_entity);
-       TYPE_DUMMY_P (gnu_type) = 0;
-
-       if (TREE_CODE (TYPE_NAME (gnu_type)) == TYPE_DECL && debug_info_p)
-         DECL_IGNORED_P (TYPE_NAME (gnu_type)) = 0;
-
-       TYPE_ALIGN (gnu_type) = 0;
-       TYPE_PACKED (gnu_type) = packed || has_rep;
+          suppress expanding incomplete types.  */
+       gnu_type = make_node (tree_code_for_record_type (gnat_entity));
+       TYPE_NAME (gnu_type) = gnu_entity_id;
+       TYPE_PACKED (gnu_type) = (packed != 0) || has_rep;
 
        if (!definition)
-         {
-           defer_incomplete_level++;
-           this_deferred = true;
-           gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
-                                        !Comes_From_Source (gnat_entity),
-                                        debug_info_p, gnat_entity);
-           save_gnu_tree (gnat_entity, gnu_decl, false);
-           this_made_decl = saved = true;
-         }
+         defer_incomplete_level++, this_deferred = true;
 
        /* If both a size and rep clause was specified, put the size in
           the record type now so that it can get the proper mode.  */
@@ -2352,27 +2616,41 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            = validate_alignment (Alignment (gnat_entity), gnat_entity, 0);
        else if (Is_Atomic (gnat_entity))
          TYPE_ALIGN (gnu_type)
-           = (esize >= BITS_PER_WORD ? BITS_PER_WORD
-              : 1 << (floor_log2 (esize - 1) + 1));
+           = esize >= BITS_PER_WORD ? BITS_PER_WORD : ceil_alignment (esize);
+       /* If a type needs strict alignment, the minimum size will be the
+          type size instead of the RM size (see validate_size).  Cap the
+          alignment, lest it causes this type size to become too large.  */
+       else if (Strict_Alignment (gnat_entity)
+                && Known_Static_Esize (gnat_entity))
+         {
+           unsigned int raw_size = UI_To_Int (Esize (gnat_entity));
+           unsigned int raw_align = raw_size & -raw_size;
+           if (raw_align < BIGGEST_ALIGNMENT)
+             TYPE_ALIGN (gnu_type) = raw_align;
+         }
+       else
+         TYPE_ALIGN (gnu_type) = 0;
 
        /* If we have a Parent_Subtype, make a field for the parent.  If
           this record has rep clauses, force the position to zero.  */
        if (Present (Parent_Subtype (gnat_entity)))
          {
+           Entity_Id gnat_parent = Parent_Subtype (gnat_entity);
            tree gnu_parent;
 
            /* A major complexity here is that the parent subtype will
-              reference our discriminants.  But those must reference
-              the parent component of this record.  So here we will
-              initialize each of those components to a COMPONENT_REF.
-              The first operand of that COMPONENT_REF is another
-              COMPONENT_REF which will be filled in below, once
-              the parent type can be safely built.  */
-
+              reference our discriminants in its Discriminant_Constraint
+              list.  But those must reference the parent component of this
+              record which is of the parent subtype we have not built yet!
+              To break the circle we first build a dummy COMPONENT_REF which
+              represents the "get to the parent" operation and initialize
+              each of those discriminants to a COMPONENT_REF of the above
+              dummy parent referencing the corresponding discriminant of the
+              base type of the parent subtype.  */
            gnu_get_parent = build3 (COMPONENT_REF, void_type_node,
                                     build0 (PLACEHOLDER_EXPR, gnu_type),
                                     build_decl (FIELD_DECL, NULL_TREE,
-                                                NULL_TREE),
+                                                void_type_node),
                                     NULL_TREE);
 
            if (Has_Discriminants (gnat_entity))
@@ -2390,8 +2668,35 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                             NULL_TREE),
                     true);
 
-           gnu_parent = gnat_to_gnu_type (Parent_Subtype (gnat_entity));
+           /* Then we build the parent subtype.  */
+           gnu_parent = gnat_to_gnu_type (gnat_parent);
+
+           /* Finally we fix up both kinds of twisted COMPONENT_REF we have
+              initially built.  The discriminants must reference the fields
+              of the parent subtype and not those of its base type for the
+              placeholder machinery to properly work.  */
+           if (Has_Discriminants (gnat_entity))
+             for (gnat_field = First_Stored_Discriminant (gnat_entity);
+                  Present (gnat_field);
+                  gnat_field = Next_Stored_Discriminant (gnat_field))
+               if (Present (Corresponding_Discriminant (gnat_field)))
+                 {
+                   Entity_Id field = Empty;
+                   for (field = First_Stored_Discriminant (gnat_parent);
+                        Present (field);
+                        field = Next_Stored_Discriminant (field))
+                     if (same_discriminant_p (gnat_field, field))
+                       break;
+                   gcc_assert (Present (field));
+                   TREE_OPERAND (get_gnu_tree (gnat_field), 1)
+                     = gnat_to_gnu_field_decl (field);
+                 }
+
+           /* The "get to the parent" COMPONENT_REF must be given its
+              proper type...  */
+           TREE_TYPE (gnu_get_parent) = gnu_parent;
 
+           /* ...and reference the _parent field of this record.  */
            gnu_field_list
              = create_field_decl (get_identifier
                                   (Get_Name_String (Name_uParent)),
@@ -2399,14 +2704,12 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                   has_rep ? TYPE_SIZE (gnu_parent) : 0,
                                   has_rep ? bitsize_zero_node : 0, 1);
            DECL_INTERNAL_P (gnu_field_list) = 1;
-
-           TREE_TYPE (gnu_get_parent) = gnu_parent;
            TREE_OPERAND (gnu_get_parent, 1) = gnu_field_list;
          }
 
-       /* Add the fields for the discriminants into the record.  */
-        if (!Is_Unchecked_Union (gnat_entity)
-           && Has_Discriminants (gnat_entity))
+       /* Make the fields for the discriminants and put them into the record
+          unless it's an Unchecked_Union.  */
+       if (Has_Discriminants (gnat_entity))
          for (gnat_field = First_Stored_Discriminant (gnat_entity);
               Present (gnat_field);
               gnat_field = Next_Stored_Discriminant (gnat_field))
@@ -2432,8 +2735,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                     gnu_field, NULL_TREE),
                             true);
 
-             TREE_CHAIN (gnu_field) = gnu_field_list;
-             gnu_field_list = gnu_field;
+             if (!Is_Unchecked_Union (gnat_entity))
+               {
+                 TREE_CHAIN (gnu_field) = gnu_field_list;
+                 gnu_field_list = gnu_field;
+               }
            }
 
        /* Put the discriminants into the record (backwards), so we can
@@ -2441,25 +2747,15 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           variants.  */
        TYPE_FIELDS (gnu_type) = gnu_field_list;
 
-       /* Add the listed fields into the record and finish up.  */
+       /* Add the listed fields into the record and finish it up.  */
        components_to_record (gnu_type, Component_List (record_definition),
                              gnu_field_list, packed, definition, NULL,
-                             false, all_rep, this_deferred);
-
-        if (this_deferred)
-         {
-           debug_deferred = true;
-           defer_debug_level++;
-
-           defer_debug_incomplete_list
-             = tree_cons (NULL_TREE, gnu_type,
-                          defer_debug_incomplete_list);
-         }
+                             false, all_rep, false,
+                             Is_Unchecked_Union (gnat_entity));
 
        /* We used to remove the associations of the discriminants and
           _Parent for validity checking, but we may need them if there's
           Freeze_Node for a subtype used in this record.  */
-
        TYPE_VOLATILE (gnu_type) = Treat_As_Volatile (gnat_entity);
        TYPE_BY_REFERENCE_P (gnu_type) = Is_By_Reference_Type (gnat_entity);
 
@@ -2497,11 +2793,9 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       /* If an equivalent type is present, that is what we should use.
         Otherwise, fall through to handle this like a record subtype
         since it may have constraints.  */
-
-      if (Present (Equivalent_Type (gnat_entity)))
+      if (gnat_equiv_type != gnat_entity)
        {
-         gnu_decl = gnat_to_gnu_entity (Equivalent_Type (gnat_entity),
-                                        NULL_TREE, 0);
+         gnu_decl = gnat_to_gnu_entity (gnat_equiv_type, NULL_TREE, 0);
          maybe_present = true;
          break;
        }
@@ -2540,12 +2834,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          /* Get the base type initially for its alignment and sizes.  But
             if it is a padded type, we do all the other work with the
             unpadded type.  */
-         gnu_type = gnu_orig_type = gnu_base_type
-           = gnat_to_gnu_type (gnat_base_type);
+         gnu_base_type = gnat_to_gnu_type (gnat_base_type);
 
-         if (TREE_CODE (gnu_type) == RECORD_TYPE
-             && TYPE_IS_PADDING_P (gnu_type))
-           gnu_type = gnu_orig_type = TREE_TYPE (TYPE_FIELDS (gnu_type));
+         if (TREE_CODE (gnu_base_type) == RECORD_TYPE
+             && TYPE_IS_PADDING_P (gnu_base_type))
+           gnu_type = gnu_orig_type = TREE_TYPE (TYPE_FIELDS (gnu_base_type));
+         else
+           gnu_type = gnu_orig_type = gnu_base_type;
 
          if (present_gnu_tree (gnat_entity))
            {
@@ -2583,10 +2878,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
              gnu_type = make_node (RECORD_TYPE);
              TYPE_NAME (gnu_type) = gnu_entity_id;
-             TYPE_STUB_DECL (gnu_type)
-               = create_type_decl (NULL_TREE, gnu_type, NULL, false, false,
-                                   gnat_entity);
              TYPE_ALIGN (gnu_type) = TYPE_ALIGN (gnu_base_type);
+             TYPE_VOLATILE (gnu_type) = Treat_As_Volatile (gnat_entity);
 
              for (gnat_field = First_Entity (gnat_entity);
                   Present (gnat_field); gnat_field = Next_Entity (gnat_field))
@@ -2631,15 +2924,21 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                        gnu_field_type = TREE_TYPE (gnu_old_field);
                      }
 
-                   /* If this was a bitfield, get the size from the old field.
-                      Also ensure the type can be placed into a bitfield.  */
-                   else if (DECL_BIT_FIELD (gnu_old_field))
+                   /* If the old field was packed and of constant size, we
+                      have to get the old size here, as it might differ from
+                      what the Etype conveys and the latter might overlap
+                      onto the following field.  Try to arrange the type for
+                      possible better packing along the way.  */
+                   else if (DECL_PACKED (gnu_old_field)
+                            && TREE_CODE (DECL_SIZE (gnu_old_field))
+                               == INTEGER_CST)
                      {
                        gnu_size = DECL_SIZE (gnu_old_field);
                        if (TYPE_MODE (gnu_field_type) == BLKmode
                            && TREE_CODE (gnu_field_type) == RECORD_TYPE
                            && host_integerp (TYPE_SIZE (gnu_field_type), 1))
-                         gnu_field_type = make_packable_type (gnu_field_type);
+                         gnu_field_type
+                           = make_packable_type (gnu_field_type, true);
                      }
 
                    if (CONTAINS_PLACEHOLDER_P (gnu_pos))
@@ -2658,7 +2957,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                    gnu_field
                      = create_field_decl
                        (DECL_NAME (gnu_old_field), gnu_field_type, gnu_type,
-                        0, gnu_size, gnu_new_pos,
+                        DECL_PACKED (gnu_old_field), gnu_size, gnu_new_pos,
                         !DECL_NONADDRESSABLE_P (gnu_old_field));
 
                    if (!TREE_CONSTANT (gnu_pos))
@@ -2700,8 +2999,9 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                    && !present_gnu_tree (Etype (gnat_field)))
                  gnat_to_gnu_entity (Etype (gnat_field), NULL_TREE, 0);
 
+             /* Do not finalize it since we're going to modify it below.  */
              finish_record_type (gnu_type, nreverse (gnu_field_list),
-                                 true, false);
+                                 2, true);
 
              /* Now set the size, alignment and alias set of the new type to
                 match that of the old one, doing any substitutions, as
@@ -2736,19 +3036,21 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                                   TREE_PURPOSE (gnu_temp),
                                                   TREE_VALUE (gnu_temp)));
 
+             /* Reapply variable_size since we have changed the sizes.  */
+             TYPE_SIZE (gnu_type) = variable_size (TYPE_SIZE (gnu_type));
+             TYPE_SIZE_UNIT (gnu_type)
+               = variable_size (TYPE_SIZE_UNIT (gnu_type));
+
              /* Recompute the mode of this record type now that we know its
                 actual size.  */
              compute_record_mode (gnu_type);
 
              /* Fill in locations of fields.  */
              annotate_rep (gnat_entity, gnu_type);
-           }
 
-         /* If we've made a new type, record it and make an XVS type to show
-            what this is a subtype of.  Some debuggers require the  XVS
-            type to be output first, so do it in that order.  */
-         if (gnu_type != gnu_orig_type)
-           {
+             /* We've built a new type, make an XVS type to show what this
+                is a subtype of.  Some debuggers require the XVS type to be
+                output first, so do it in that order.  */
              if (debug_info_p)
                {
                  tree gnu_subtype_marker = make_node (RECORD_TYPE);
@@ -2765,14 +3067,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                                         gnu_subtype_marker,
                                                         0, NULL_TREE,
                                                         NULL_TREE, 0),
-                                     false, false);
+                                     0, false);
                }
 
-             TYPE_VOLATILE (gnu_type) = Treat_As_Volatile (gnat_entity);
-             TYPE_NAME (gnu_type) = gnu_entity_id;
-             TYPE_STUB_DECL (gnu_type)
-               = create_type_decl (TYPE_NAME (gnu_type), gnu_type,
-                                   NULL, true, debug_info_p, gnat_entity);
+             /* Now we can finalize it.  */
+             rest_of_record_type_compilation (gnu_type);
            }
 
          /* Otherwise, go down all the components in the new type and
@@ -2790,6 +3089,22 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       break;
 
     case E_Access_Subprogram_Type:
+      /* Use the special descriptor type for dispatch tables if needed,
+        that is to say for the Prim_Ptr of a-tags.ads and its clones.
+        Note that we are only required to do so for static tables in
+        order to be compatible with the C++ ABI, but Ada 2005 allows
+        to extend library level tagged types at the local level so
+        we do it in the non-static case as well.  */
+      if (TARGET_VTABLE_USES_DESCRIPTORS
+         && Is_Dispatch_Table_Entity (gnat_entity))
+       {
+           gnu_type = fdesc_type_node;
+           gnu_size = TYPE_SIZE (gnu_type);
+           break;
+       }
+
+      /* ... fall through ... */
+
     case E_Anonymous_Access_Subprogram_Type:
       /* If we are not defining this entity, and we have incomplete
         entities being processed above us, make a dummy type and
@@ -2805,8 +3120,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
                                       !Comes_From_Source (gnat_entity),
                                       debug_info_p, gnat_entity);
+         this_made_decl = true;
+         gnu_type = TREE_TYPE (gnu_decl);
          save_gnu_tree (gnat_entity, gnu_decl, false);
-         this_made_decl = saved = true;
+         saved = true;
 
          p->old_type = TREE_TYPE (gnu_type);
          p->full_type = Directly_Designated_Type (gnat_entity);
@@ -2824,17 +3141,48 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
     case E_General_Access_Type:
       {
        Entity_Id gnat_desig_type = Directly_Designated_Type (gnat_entity);
+       Entity_Id gnat_desig_equiv = Gigi_Equivalent_Type (gnat_desig_type);
+       bool is_from_limited_with
+         = (IN (Ekind (gnat_desig_equiv), Incomplete_Kind)
+            && From_With_Type (gnat_desig_equiv));
+
+       /* Get the "full view" of this entity.  If this is an incomplete
+          entity from a limited with, treat its non-limited view as the full
+          view.  Otherwise, if this is an incomplete or private type, use the
+          full view.  In the former case, we might point to a private type,
+          in which case, we need its full view.  Also, we want to look at the
+          actual type used for the representation, so this takes a total of
+          three steps.  */
+       Entity_Id gnat_desig_full_direct_first
+         = (is_from_limited_with ? Non_Limited_View (gnat_desig_equiv)
+            : (IN (Ekind (gnat_desig_equiv), Incomplete_Or_Private_Kind)
+               ? Full_View (gnat_desig_equiv) : Empty));
+       Entity_Id gnat_desig_full_direct
+         = ((is_from_limited_with
+             && Present (gnat_desig_full_direct_first)
+             && IN (Ekind (gnat_desig_full_direct_first), Private_Kind))
+            ? Full_View (gnat_desig_full_direct_first)
+            : gnat_desig_full_direct_first);
        Entity_Id gnat_desig_full
-         = ((IN (Ekind (Etype (gnat_desig_type)),
-                 Incomplete_Or_Private_Kind))
-            ? Full_View (gnat_desig_type) : 0);
+         = Gigi_Equivalent_Type (gnat_desig_full_direct);
+
+       /* This the type actually used to represent the designated type,
+          either gnat_desig_full or gnat_desig_equiv.  */
+       Entity_Id gnat_desig_rep;
+
+       /* Nonzero if this is a pointer to an unconstrained array.  */
+       bool is_unconstrained_array;
+
        /* We want to know if we'll be seeing the freeze node for any
           incomplete type we may be pointing to.  */
        bool in_main_unit
          = (Present (gnat_desig_full)
             ? In_Extended_Main_Code_Unit (gnat_desig_full)
             : In_Extended_Main_Code_Unit (gnat_desig_type));
+
+       /* Nonzero if we make a dummy type here.  */
        bool got_fat_p = false;
+       /* Nonzero if the dummy is a fat pointer. */
        bool made_dummy = false;
        tree gnu_desig_type = NULL_TREE;
        enum machine_mode p_mode = mode_for_size (esize, MODE_INT, 0);
@@ -2842,76 +3190,52 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        if (!targetm.valid_pointer_mode (p_mode))
          p_mode = ptr_mode;
 
-       if (No (gnat_desig_full)
-           && (Ekind (gnat_desig_type) == E_Class_Wide_Type
-               || (Ekind (gnat_desig_type) == E_Class_Wide_Subtype
-                   && Present (Equivalent_Type (gnat_desig_type)))))
-         {
-           if (Present (Equivalent_Type (gnat_desig_type)))
-             {
-               gnat_desig_full = Equivalent_Type (gnat_desig_type);
-               if (IN (Ekind (gnat_desig_full), Incomplete_Or_Private_Kind))
-                 gnat_desig_full = Full_View (gnat_desig_full);
-             }
-           else if (IN (Ekind (Root_Type (gnat_desig_type)),
-                        Incomplete_Or_Private_Kind))
-             gnat_desig_full = Full_View (Root_Type (gnat_desig_type));
-         }
-
-       if (Present (gnat_desig_full) && Is_Concurrent_Type (gnat_desig_full))
-         gnat_desig_full = Corresponding_Record_Type (gnat_desig_full);
-
-       /* If either the designated type or its full view is an
-          unconstrained array subtype, replace it with the type it's a
-          subtype of.  This avoids problems with multiple copies of
-          unconstrained array types.  */
-       if (Ekind (gnat_desig_type) == E_Array_Subtype
-           && !Is_Constrained (gnat_desig_type))
-         gnat_desig_type = Etype (gnat_desig_type);
+       /* If either the designated type or its full view is an unconstrained
+          array subtype, replace it with the type it's a subtype of.  This
+          avoids problems with multiple copies of unconstrained array types.
+          Likewise, if the designated type is a subtype of an incomplete
+          record type, use the parent type to avoid order of elaboration
+          issues.  This can lose some code efficiency, but there is no
+          alternative.  */
+       if (Ekind (gnat_desig_equiv) == E_Array_Subtype
+           && ! Is_Constrained (gnat_desig_equiv))
+         gnat_desig_equiv = Etype (gnat_desig_equiv);
        if (Present (gnat_desig_full)
-           && Ekind (gnat_desig_full) == E_Array_Subtype
-           && !Is_Constrained (gnat_desig_full))
+           && ((Ekind (gnat_desig_full) == E_Array_Subtype
+                && ! Is_Constrained (gnat_desig_full))
+               || (Ekind (gnat_desig_full) == E_Record_Subtype
+                   && Ekind (Etype (gnat_desig_full)) == E_Record_Type)))
          gnat_desig_full = Etype (gnat_desig_full);
 
-        /* If the designated type is a subtype of an incomplete record type,
-           use the parent type to avoid order of elaboration issues.  This
-          can lose some code efficiency, but there is no alternative.  */
-        if (Present (gnat_desig_full)
-             && Ekind (gnat_desig_full) == E_Record_Subtype
-             && Ekind (Etype (gnat_desig_full)) == E_Record_Type)
-         gnat_desig_full = Etype (gnat_desig_full);
+       /* Now set the type that actually marks the representation of
+          the designated type and also flag whether we have a unconstrained
+          array.  */
+       gnat_desig_rep = gnat_desig_full ? gnat_desig_full : gnat_desig_equiv;
+       is_unconstrained_array
+         = (Is_Array_Type (gnat_desig_rep)
+            && ! Is_Constrained (gnat_desig_rep));
 
        /* If we are pointing to an incomplete type whose completion is an
-          unconstrained array, make a fat pointer type instead of a pointer
-          to VOID.  The two types in our fields will be pointers to VOID and
-          will be replaced in update_pointer_to.  Similarly, if the type
-          itself is a dummy type or an unconstrained array.  Also make
-          a dummy TYPE_OBJECT_RECORD_TYPE in case we have any thin
-          pointers to it.  */
-
-       if ((Present (gnat_desig_full)
-            && Is_Array_Type (gnat_desig_full)
-            && !Is_Constrained (gnat_desig_full))
-           || (present_gnu_tree (gnat_desig_type)
-               && TYPE_IS_DUMMY_P (TREE_TYPE
-                                    (get_gnu_tree (gnat_desig_type)))
-               && Is_Array_Type (gnat_desig_type)
-               && !Is_Constrained (gnat_desig_type))
-           || (present_gnu_tree (gnat_desig_type)
-               && (TREE_CODE (TREE_TYPE (get_gnu_tree (gnat_desig_type)))
-                   == UNCONSTRAINED_ARRAY_TYPE)
-               && !(TYPE_POINTER_TO (TREE_TYPE
-                                    (get_gnu_tree (gnat_desig_type)))))
-           || (No (gnat_desig_full) && !in_main_unit
-               && defer_incomplete_level
-               && !present_gnu_tree (gnat_desig_type)
-               && Is_Array_Type (gnat_desig_type)
-               && !Is_Constrained (gnat_desig_type)))
-         {
+          unconstrained array, make a fat pointer type.  The two types in our
+          fields will be pointers to dummy nodes and will be replaced in
+          update_pointer_to.  Similarly, if the type itself is a dummy type or
+          an unconstrained array.  Also make a dummy TYPE_OBJECT_RECORD_TYPE
+          in case we have any thin pointers to it.  */
+       if (is_unconstrained_array
+           && (Present (gnat_desig_full)
+               || (present_gnu_tree (gnat_desig_equiv)
+                   && TYPE_IS_DUMMY_P (TREE_TYPE
+                                       (get_gnu_tree (gnat_desig_equiv))))
+               || (No (gnat_desig_full) && ! in_main_unit
+                   && defer_incomplete_level != 0
+                   && ! present_gnu_tree (gnat_desig_equiv))
+               || (in_main_unit && is_from_limited_with
+                   && Present (Freeze_Node (gnat_desig_rep)))))
+         {
            tree gnu_old
-             = (present_gnu_tree (gnat_desig_type)
-                ? gnat_to_gnu_type (gnat_desig_type)
-                : make_dummy_type (gnat_desig_type));
+             = (present_gnu_tree (gnat_desig_rep)
+                ? TREE_TYPE (get_gnu_tree (gnat_desig_rep))
+                : make_dummy_type (gnat_desig_rep));
            tree fields;
 
            /* Show the dummy we get will be a fat pointer.  */
@@ -2924,6 +3248,21 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            gnu_type = TYPE_POINTER_TO (gnu_old);
            if (!gnu_type)
              {
+               tree gnu_template_type = make_node (ENUMERAL_TYPE);
+               tree gnu_ptr_template = build_pointer_type (gnu_template_type);
+               tree gnu_array_type = make_node (ENUMERAL_TYPE);
+               tree gnu_ptr_array = build_pointer_type (gnu_array_type);
+
+               TYPE_NAME (gnu_template_type)
+                 = concat_id_with_name (get_entity_name (gnat_desig_equiv),
+                                        "XUB");
+               TYPE_DUMMY_P (gnu_template_type) = 1;
+
+               TYPE_NAME (gnu_array_type)
+                 = concat_id_with_name (get_entity_name (gnat_desig_equiv),
+                                        "XUA");
+               TYPE_DUMMY_P (gnu_array_type) = 1;
+
                gnu_type = make_node (RECORD_TYPE);
                SET_TYPE_UNCONSTRAINED_ARRAY (gnu_type, gnu_old);
                TYPE_POINTER_TO (gnu_old) = gnu_type;
@@ -2933,21 +3272,24 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                  = chainon (chainon (NULL_TREE,
                                      create_field_decl
                                      (get_identifier ("P_ARRAY"),
-                                      ptr_void_type_node, gnu_type,
-                                      0, 0, 0, 0)),
+                                      gnu_ptr_array,
+                                      gnu_type, 0, 0, 0, 0)),
                             create_field_decl (get_identifier ("P_BOUNDS"),
-                                               ptr_void_type_node,
+                                               gnu_ptr_template,
                                                gnu_type, 0, 0, 0, 0));
 
                /* Make sure we can place this into a register.  */
                TYPE_ALIGN (gnu_type)
                  = MIN (BIGGEST_ALIGNMENT, 2 * POINTER_SIZE);
                TYPE_IS_FAT_POINTER_P (gnu_type) = 1;
-               finish_record_type (gnu_type, fields, false, true);
+
+               /* Do not finalize this record type since the types of
+                  its fields are incomplete.  */
+               finish_record_type (gnu_type, fields, 0, true);
 
                TYPE_OBJECT_RECORD_TYPE (gnu_old) = make_node (RECORD_TYPE);
                TYPE_NAME (TYPE_OBJECT_RECORD_TYPE (gnu_old))
-                 = concat_id_with_name (get_entity_name (gnat_desig_type),
+                 = concat_id_with_name (get_entity_name (gnat_desig_equiv),
                                         "XUT");
                TYPE_DUMMY_P (TYPE_OBJECT_RECORD_TYPE (gnu_old)) = 1;
              }
@@ -2962,43 +3304,57 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           to it.  If it is a reference to an incomplete or private type with a
           full view that is a record, make a dummy type node and get the
           actual type later when we have verified it is safe.  */
-       else if (!in_main_unit
-                && !present_gnu_tree (gnat_desig_type)
-                && Present (gnat_desig_full)
-                && !present_gnu_tree (gnat_desig_full)
-                && Is_Record_Type (gnat_desig_full))
+       else if ((! in_main_unit
+                 && ! present_gnu_tree (gnat_desig_equiv)
+                 && Present (gnat_desig_full)
+                 && ! present_gnu_tree (gnat_desig_full)
+                 && Is_Record_Type (gnat_desig_full))
+                /* Likewise if we are pointing to a record or array and we
+                   are to defer elaborating incomplete types.  We do this
+                   since this access type may be the full view of some
+                   private type.  Note that the unconstrained array case is
+                   handled above. */
+                || ((! in_main_unit || imported_p)
+                    && defer_incomplete_level != 0
+                    && ! present_gnu_tree (gnat_desig_equiv)
+                    && ((Is_Record_Type (gnat_desig_rep)
+                         || Is_Array_Type (gnat_desig_rep))))
+                /* If this is a reference from a limited_with type back to our
+                   main unit and there's a Freeze_Node for it, either we have
+                   already processed the declaration and made the dummy type,
+                   in which case we just reuse the latter, or we have not yet,
+                   in which case we make the dummy type and it will be reused
+                   when the declaration is processed.  In both cases, the
+                   pointer eventually created below will be automatically
+                   adjusted when the Freeze_Node is processed.  Note that the
+                   unconstrained array case is handled above. */
+                ||  (in_main_unit && is_from_limited_with
+                     && Present (Freeze_Node (gnat_desig_rep))))
          {
-           gnu_desig_type = make_dummy_type (gnat_desig_type);
+           gnu_desig_type = make_dummy_type (gnat_desig_equiv);
            made_dummy = true;
          }
 
-       /* Likewise if we are pointing to a record or array and we are to defer
-          elaborating incomplete types.  We do this since this access type
-          may be the full view of some private type.  Note that the
-          unconstrained array case is handled above. */
-       else if ((!in_main_unit || imported_p) && defer_incomplete_level != 0
-                && !present_gnu_tree (gnat_desig_type)
-                && ((Is_Record_Type (gnat_desig_type)
-                     || Is_Array_Type (gnat_desig_type))
-                    || (Present (gnat_desig_full)
-                        && (Is_Record_Type (gnat_desig_full)
-                            || Is_Array_Type (gnat_desig_full)))))
-         {
-           gnu_desig_type = make_dummy_type (gnat_desig_type);
-           made_dummy = true;
-         }
-       else if (gnat_desig_type == gnat_entity)
+       /* Otherwise handle the case of a pointer to itself.  */
+       else if (gnat_desig_equiv == gnat_entity)
          {
            gnu_type
-             = build_pointer_type_for_mode (make_node (VOID_TYPE),
-                                            p_mode,
+             = build_pointer_type_for_mode (void_type_node, p_mode,
                                             No_Strict_Aliasing (gnat_entity));
            TREE_TYPE (gnu_type) = TYPE_POINTER_TO (gnu_type) = gnu_type;
          }
+
+       /* If expansion is disabled, the equivalent type of a concurrent
+          type is absent, so build a dummy pointer type.  */
+       else if (type_annotate_only && No (gnat_desig_equiv))
+         gnu_type = ptr_void_type_node;
+
+       /* Finally, handle the straightforward case where we can just
+          elaborate our designated type and point to it.  */
        else
-         gnu_desig_type = gnat_to_gnu_type (gnat_desig_type);
+         gnu_desig_type = gnat_to_gnu_type (gnat_desig_equiv);
 
-       /* It is possible that the above call to gnat_to_gnu_type resolved our
+       /* It is possible that a call to gnat_to_gnu_type above resolved our
           type.  If so, just return it.  */
        if (present_gnu_tree (gnat_entity))
          {
@@ -3053,9 +3409,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           save our current definition, evaluate the actual type, and replace
           the tentative type we made with the actual one.  If we are to defer
           actually looking up the actual type, make an entry in the
-          deferred list.  */
-
-       if (!in_main_unit && made_dummy)
+          deferred list.  If this is from a limited with, we have to defer
+          to the end of the current spec in two cases: first if the
+          designated type is in the current unit and second if the access
+          type is.  */
+       if ((! in_main_unit || is_from_limited_with) && made_dummy)
          {
            tree gnu_old_type
              = TYPE_FAT_POINTER_P (gnu_type)
@@ -3071,10 +3429,18 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
            gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
                                         !Comes_From_Source (gnat_entity),
                                         debug_info_p, gnat_entity);
+           this_made_decl = true;
+           gnu_type = TREE_TYPE (gnu_decl);
            save_gnu_tree (gnat_entity, gnu_decl, false);
-           this_made_decl = saved = true;
+           saved = true;
+
+           if (defer_incomplete_level == 0
+               && ! (is_from_limited_with
+                     && (in_main_unit
+                         || In_Extended_Main_Code_Unit (gnat_entity))))
+             update_pointer_to (TYPE_MAIN_VARIANT (gnu_old_type),
+                                gnat_to_gnu_type (gnat_desig_equiv));
 
-           if (defer_incomplete_level == 0)
              /* Note that the call to gnat_to_gnu_type here might have
                 updated gnu_old_type directly, in which case it is not a
                 dummy type any more when we get into update_pointer_to.
@@ -3087,17 +3453,21 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                 Besides, variants of this non-dummy type might have been
                 created along the way. update_pointer_to is expected to
                 properly take care of those situations.  */
-             update_pointer_to (TYPE_MAIN_VARIANT (gnu_old_type),
-                                gnat_to_gnu_type (gnat_desig_type));
            else
              {
                struct incomplete *p
-                 = (struct incomplete *) xmalloc (sizeof (struct incomplete));
+                 = (struct incomplete *) xmalloc (sizeof
+                                                  (struct incomplete));
+               struct incomplete **head
+                 = (is_from_limited_with
+                    && (in_main_unit
+                        || In_Extended_Main_Code_Unit (gnat_entity))
+                    ? &defer_limited_with : &defer_incomplete_list);
 
                p->old_type = gnu_old_type;
-               p->full_type = gnat_desig_type;
-               p->next = defer_incomplete_list;
-               defer_incomplete_list = p;
+               p->full_type = gnat_desig_equiv;
+               p->next = *head;
+               *head = p;
              }
          }
       }
@@ -3105,11 +3475,14 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
     case E_Access_Protected_Subprogram_Type:
     case E_Anonymous_Access_Protected_Subprogram_Type:
-      if (type_annotate_only && No (Equivalent_Type (gnat_entity)))
-       gnu_type = build_pointer_type (void_type_node);
+      if (type_annotate_only && No (gnat_equiv_type))
+       gnu_type = ptr_void_type_node;
       else
-       /* The runtime representation is the equivalent type. */
-       gnu_type = gnat_to_gnu_type (Equivalent_Type (gnat_entity));
+       {
+         /* The runtime representation is the equivalent type. */
+         gnu_type = gnat_to_gnu_type (gnat_equiv_type);
+         maybe_present = 1;
+       }
 
       if (Is_Itype (Directly_Designated_Type (gnat_entity))
          && !present_gnu_tree (Directly_Designated_Type (gnat_entity))
@@ -3156,11 +3529,9 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
              p->next = defer_incomplete_list;
              defer_incomplete_list = p;
            }
-          else if (IN (Ekind (Base_Type
+         else if (!IN (Ekind (Base_Type
                              (Directly_Designated_Type (gnat_entity))),
                       Incomplete_Or_Private_Kind))
-           ;
-         else
            gnat_to_gnu_entity (Directly_Designated_Type (gnat_entity),
                                NULL_TREE, 0);
        }
@@ -3177,6 +3548,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                Is_Imported     Indicates that the subprogram has appeared in
                                an INTERFACE or IMPORT pragma. For now we
                                assume that the external language is C.
+               Is_Exported     Likewise but for an EXPORT pragma.
                Is_Inlined      True if the subprogram is to be inlined.
 
        In addition for function subprograms we have:
@@ -3185,7 +3557,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
        Each parameter is first checked by calling must_pass_by_ref on its
        type to determine if it is passed by reference.  For parameters which
-       are copied in, if they are Ada IN OUT or OUT parameters, their return
+       are copied in, if they are Ada In Out or Out parameters, their return
        value becomes part of a record which becomes the return type of the
        function (C function - note that this applies only to Ada procedures
        so there is no Ada return type). Additional code to store back the
@@ -3195,18 +3567,16 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        The intended result of the transformation can be seen from the
        equivalent source rewritings that follow:
 
-                                                   struct temp {int a,b};
-       procedure P (A,B: IN OUT ...) is            temp P (int A,B) {
-        ..                                            ..
-       end P;                                        return {A,B};
-                                                   }
-                              procedure call
+                                               struct temp {int a,b};
+       procedure P (A,B: In Out ...) is                temp P (int A,B)
+       begin                                   {
+        ..                                       ..
+       end P;                                    return {A,B};
+                                               }
 
-                                              {
-                                                  temp t;
-       P(X,Y);                                    t = P(X,Y);
-                                                  X = t.a , Y = t.b;
-                                              }
+                                               temp t;
+       P(X,Y);                                 t = P(X,Y);
+                                               X = t.a , Y = t.b;
 
        For subprogram types we need to perform mainly the same conversions to
        GCC form that are needed for procedures and function declarations.  The
@@ -3221,14 +3591,16 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           PARM_DECL nodes are chained through the TREE_CHAIN field, so this
           actually is the head of this parameter list.  */
        tree gnu_param_list = NULL_TREE;
+       /* Likewise for the stub associated with an exported procedure.  */
+       tree gnu_stub_param_list = NULL_TREE;
        /* The type returned by a function. If the subprogram is a procedure
           this type should be void_type_node.  */
        tree gnu_return_type = void_type_node;
-        /* List of fields in return type of procedure with copy in copy out
+       /* List of fields in return type of procedure with copy-in copy-out
           parameters.  */
-        tree gnu_field_list = NULL_TREE;
-       /* Non-null for subprograms containing  parameters passed by copy in
-          copy out (Ada IN OUT or OUT parameters not passed by reference),
+       tree gnu_field_list = NULL_TREE;
+       /* Non-null for subprograms containing parameters passed by copy-in
+          copy-out (Ada In Out or Out parameters not passed by reference),
           in which case it is the list of nodes used to specify the values of
           the in out/out parameters that are returned as a record upon
           procedure return.  The TREE_PURPOSE of an element of this list is
@@ -3239,9 +3611,12 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        /* If an import pragma asks to map this subprogram to a GCC builtin,
           this is the builtin DECL node.  */
        tree gnu_builtin_decl = NULL_TREE;
+       /* For the stub associated with an exported procedure.  */
+       tree gnu_stub_type = NULL_TREE, gnu_stub_name = NULL_TREE;
+       tree gnu_ext_name = create_concat_name (gnat_entity, NULL);
        Entity_Id gnat_param;
        bool inline_flag = Is_Inlined (gnat_entity);
-       bool public_flag = Is_Public (gnat_entity);
+       bool public_flag = Is_Public (gnat_entity) || imported_p;
        bool extern_flag
          = (Is_Public (gnat_entity) && !definition) || imported_p;
        bool pure_flag = Is_Pure (gnat_entity);
@@ -3249,8 +3624,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        bool returns_by_ref = false;
        bool returns_unconstrained = false;
        bool returns_by_target_ptr = false;
-       tree gnu_ext_name = create_concat_name (gnat_entity, 0);
        bool has_copy_in_out = false;
+       bool has_stub = false;
        int parmnum;
 
        if (kind == E_Subprogram_Type && !definition)
@@ -3271,7 +3646,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                                           gnu_expr, 0);
 
            /* Elaborate any Itypes in the parameters of this entity.  */
-           for (gnat_temp = First_Formal (gnat_entity);
+           for (gnat_temp = First_Formal_With_Extras (gnat_entity);
                 Present (gnat_temp);
                 gnat_temp = Next_Formal_With_Extras (gnat_temp))
              if (Is_Itype (Etype (gnat_temp)))
@@ -3311,8 +3686,14 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        else if (kind == E_Function
                 && Mechanism (gnat_entity) == By_Reference)
          {
-           gnu_return_type = copy_type (gnu_return_type);
            TREE_ADDRESSABLE (gnu_return_type) = 1;
+
+           /* We expect this bit to be reset by gigi shortly, so can avoid a
+              type node copy here.  This actually also prevents troubles with
+              the generation of debug information for the function, because
+              we might have issued such info for this type already, and would
+              be attaching a distinct type node to the function if we made a
+              copy here.  */
          }
 
        /* If we are supposed to return an unconstrained array,
@@ -3361,6 +3742,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           a function that returns that type.  This usage doesn't make
           sense anyway, so give an error here.  */
        if (TYPE_SIZE_UNIT (gnu_return_type)
+           && TREE_CONSTANT (TYPE_SIZE_UNIT (gnu_return_type))
            && TREE_OVERFLOW (TYPE_SIZE_UNIT (gnu_return_type)))
          {
            post_error ("cannot return type whose size overflows",
@@ -3376,203 +3758,90 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
           each.  While doing this, build a copy-out structure if
           we need one.  */
 
-       for (gnat_param = First_Formal (gnat_entity), parmnum = 0;
+       /* Loop over the parameters and get their associated GCC tree.
+          While doing this, build a copy-out structure if we need one.  */
+       for (gnat_param = First_Formal_With_Extras (gnat_entity), parmnum = 0;
             Present (gnat_param);
             gnat_param = Next_Formal_With_Extras (gnat_param), parmnum++)
          {
            tree gnu_param_name = get_entity_name (gnat_param);
            tree gnu_param_type = gnat_to_gnu_type (Etype (gnat_param));
            tree gnu_param, gnu_field;
-           bool by_ref_p = false;
-           bool by_descr_p = false;
-           bool by_component_ptr_p = false;
-           bool copy_in_copy_out_flag = false;
-           bool req_by_copy = false, req_by_ref = false;
+           bool copy_in_copy_out = false;
+           Mechanism_Type mech = Mechanism (gnat_param);
 
            /* Builtins are expanded inline and there is no real call sequence
-              involved. so the type expected by the underlying expander is
+              involved.  So the type expected by the underlying expander is
               always the type of each argument "as is".  */
            if (gnu_builtin_decl)
-             req_by_copy = 1;
-
+             mech = By_Copy;
+           /* Handle the first parameter of a valued procedure specially.  */
+           else if (Is_Valued_Procedure (gnat_entity) && parmnum == 0)
+             mech = By_Copy_Return;
            /* Otherwise, see if a Mechanism was supplied that forced this
               parameter to be passed one way or another.  */
-           else if (Is_Valued_Procedure (gnat_entity) && parmnum == 0)
-             req_by_copy = true;
-           else if (Mechanism (gnat_param) == Default)
+           else if (mech == Default
+                    || mech == By_Copy || mech == By_Reference)
              ;
-           else if (Mechanism (gnat_param) == By_Copy)
-             req_by_copy = true;
-           else if (Mechanism (gnat_param) == By_Reference)
-             req_by_ref = true;
-           else if (Mechanism (gnat_param) <= By_Descriptor)
-             by_descr_p = true;
-           else if (Mechanism (gnat_param) > 0)
+           else if (By_Descriptor_Last <= mech && mech <= By_Descriptor)
+             mech = By_Descriptor;
+           else if (mech > 0)
              {
                if (TREE_CODE (gnu_param_type) == UNCONSTRAINED_ARRAY_TYPE
                    || TREE_CODE (TYPE_SIZE (gnu_param_type)) != INTEGER_CST
                    || 0 < compare_tree_int (TYPE_SIZE (gnu_param_type),
-                                            Mechanism (gnat_param)))
-                 req_by_ref = true;
+                                            mech))
+                 mech = By_Reference;
                else
-                 req_by_copy = true;
+                 mech = By_Copy;
              }
            else
-             post_error ("unsupported mechanism for&", gnat_param);
-
-           /* If this is either a foreign function or if the
-              underlying type won't be passed by reference, strip off
-              possible padding type.  */
-           if (TREE_CODE (gnu_param_type) == RECORD_TYPE
-               && TYPE_IS_PADDING_P (gnu_param_type)
-               && (req_by_ref || Has_Foreign_Convention (gnat_entity)
-                   || (!must_pass_by_ref (TREE_TYPE (TYPE_FIELDS
-                                                     (gnu_param_type)))
-                       && (req_by_copy
-                           || !default_pass_by_ref (TREE_TYPE
-                                                     (TYPE_FIELDS
-                                                      (gnu_param_type)))))))
-             gnu_param_type = TREE_TYPE (TYPE_FIELDS (gnu_param_type));
-
-           /* If this is an IN parameter it is read-only, so make a variant
-              of the type that is read-only.
-
-              ??? However, if this is an unconstrained array, that type can
-              be very complex.  So skip it for now.  Likewise for any other
-              self-referential type.  */
-           if (Ekind (gnat_param) == E_In_Parameter
-               && TREE_CODE (gnu_param_type) != UNCONSTRAINED_ARRAY_TYPE
-               && !CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_param_type)))
-             gnu_param_type
-               = build_qualified_type (gnu_param_type,
-                                       (TYPE_QUALS (gnu_param_type)
-                                        | TYPE_QUAL_CONST));
-
-           /* For foreign conventions, pass arrays as a pointer to the
-              underlying type.  First check for unconstrained array and get
-              the underlying array.  Then get the component type and build
-              a pointer to it.  */
-           if (Has_Foreign_Convention (gnat_entity)
-               && TREE_CODE (gnu_param_type) == UNCONSTRAINED_ARRAY_TYPE)
-             gnu_param_type
-               = TREE_TYPE (TREE_TYPE (TYPE_FIELDS
-                                       (TREE_TYPE (gnu_param_type))));
-
-           if (by_descr_p)
-             gnu_param_type
-               = build_pointer_type
-                 (build_vms_descriptor (gnu_param_type,
-                                        Mechanism (gnat_param), gnat_entity));
-
-           else if (Has_Foreign_Convention (gnat_entity)
-                    && !req_by_copy
-                    && TREE_CODE (gnu_param_type) == ARRAY_TYPE)
              {
-               /* Strip off any multi-dimensional entries, then strip
-                  off the last array to get the component type.  */
-               while (TREE_CODE (TREE_TYPE (gnu_param_type)) == ARRAY_TYPE
-                      && TYPE_MULTI_ARRAY_P (TREE_TYPE (gnu_param_type)))
-                 gnu_param_type = TREE_TYPE (gnu_param_type);
-
-               by_component_ptr_p = true;
-               gnu_param_type = TREE_TYPE (gnu_param_type);
-
-               if (Ekind (gnat_param) == E_In_Parameter)
-                 gnu_param_type
-                   = build_qualified_type (gnu_param_type,
-                                           (TYPE_QUALS (gnu_param_type)
-                                            | TYPE_QUAL_CONST));
-
-               gnu_param_type = build_pointer_type (gnu_param_type);
+               post_error ("unsupported mechanism for&", gnat_param);
+               mech = Default;
              }
 
-           /* Fat pointers are passed as thin pointers for foreign
-              conventions.  */
-           else if (Has_Foreign_Convention (gnat_entity)
-                    && TYPE_FAT_POINTER_P (gnu_param_type))
-             gnu_param_type
-               = make_type_from_size (gnu_param_type,
-                                      size_int (POINTER_SIZE), false);
-
-           /* If we must pass or were requested to pass by reference, do so.
-              If we were requested to pass by copy, do so.
-              Otherwise, for foreign conventions, pass all in out parameters
-              or aggregates by reference.  For COBOL and Fortran, pass
-              all integer and FP types that way too.  For Convention Ada,
-              use the standard Ada default.  */
-           else if (must_pass_by_ref (gnu_param_type) || req_by_ref
-                    || (!req_by_copy
-                        && ((Has_Foreign_Convention (gnat_entity)
-                             && (Ekind (gnat_param) != E_In_Parameter
-                                 || AGGREGATE_TYPE_P (gnu_param_type)))
-                            || (((Convention (gnat_entity)
-                                  == Convention_Fortran)
-                                 || (Convention (gnat_entity)
-                                     == Convention_COBOL))
-                                && (INTEGRAL_TYPE_P (gnu_param_type)
-                                    || FLOAT_TYPE_P (gnu_param_type)))
-                            /* For convention Ada, see if we pass by reference
-                               by default.  */
-                            || (!Has_Foreign_Convention (gnat_entity)
-                                && default_pass_by_ref (gnu_param_type)))))
+           gnu_param
+             = gnat_to_gnu_param (gnat_param, mech, gnat_entity,
+                                  Has_Foreign_Convention (gnat_entity),
+                                  &copy_in_copy_out);
+
+           /* We are returned either a PARM_DECL or a type if no parameter
+              needs to be passed; in either case, adjust the type.  */
+           if (DECL_P (gnu_param))
+             gnu_param_type = TREE_TYPE (gnu_param);
+           else
              {
-               gnu_param_type = build_reference_type (gnu_param_type);
-               by_ref_p = true;
+               gnu_param_type = gnu_param;
+               gnu_param = NULL_TREE;
              }
 
-           else if (Ekind (gnat_param) != E_In_Parameter)
-             copy_in_copy_out_flag = true;
-
-           if (req_by_copy && (by_ref_p || by_component_ptr_p))
-             post_error ("?cannot pass & by copy", gnat_param);
-
-           /* If this is an OUT parameter that isn't passed by reference
-              and isn't a pointer or aggregate, we don't make a PARM_DECL
-              for it.  Instead, it will be a VAR_DECL created when we process
-              the procedure.  For the special parameter of Valued_Procedure,
-              never pass it in.
-
-              An exception is made to cover the RM-6.4.1 rule requiring "by
-              copy" out parameters with discriminants or implicit initial
-              values to be handled like in out parameters. These type are
-              normally built as aggregates, and hence passed by reference,
-              except for some packed arrays which end up encoded in special
-              integer types.
-
-              The exception we need to make is then for packed arrays of
-              records with discriminants or implicit initial values. We have
-              no light/easy way to check for the latter case, so we merely
-              check for packed arrays of records. This may lead to useless
-              copy-in operations, but in very rare cases only, as these would
-              be exceptions in a set of already exceptional situations.  */
-           if (Ekind (gnat_param) == E_Out_Parameter && !by_ref_p
-               && ((Is_Valued_Procedure (gnat_entity) && parmnum == 0)
-                   || (!by_descr_p
-                       && !POINTER_TYPE_P (gnu_param_type)
-                       && !AGGREGATE_TYPE_P (gnu_param_type)))
-               && !(Is_Array_Type (Etype (gnat_param))
-                    && Is_Packed (Etype (gnat_param))
-                    && Is_Composite_Type (Component_Type
-                                          (Etype (gnat_param)))))
-             gnu_param = NULL_TREE;
-           else
+           if (gnu_param)
              {
-               gnu_param
-                 = create_param_decl
-                   (gnu_param_name, gnu_param_type,
-                    by_ref_p || by_component_ptr_p
-                    || Ekind (gnat_param) == E_In_Parameter);
-
-               DECL_BY_REF_P (gnu_param) = by_ref_p;
-               DECL_BY_COMPONENT_PTR_P (gnu_param) = by_component_ptr_p;
-               DECL_BY_DESCRIPTOR_P (gnu_param) = by_descr_p;
-               DECL_POINTS_TO_READONLY_P (gnu_param)
-                 = (Ekind (gnat_param) == E_In_Parameter
-                    && (by_ref_p || by_component_ptr_p));
+               /* If it's an exported subprogram, we build a parameter list
+                  in parallel, in case we need to emit a stub for it.  */
+               if (Is_Exported (gnat_entity))
+                 {
+                   gnu_stub_param_list
+                     = chainon (gnu_param, gnu_stub_param_list);
+                   /* Change By_Descriptor parameter to By_Reference for
+                      the internal version of an exported subprogram.  */
+                   if (mech == By_Descriptor)
+                     {
+                       gnu_param
+                         = gnat_to_gnu_param (gnat_param, By_Reference,
+                                              gnat_entity, false,
+                                              &copy_in_copy_out);
+                       has_stub = true;
+                     }
+                   else
+                     gnu_param = copy_node (gnu_param);
+                 }
+
+               gnu_param_list = chainon (gnu_param, gnu_param_list);
                Sloc_to_locus (Sloc (gnat_param),
                               &DECL_SOURCE_LOCATION (gnu_param));
                save_gnu_tree (gnat_param, gnu_param, false);
-               gnu_param_list = chainon (gnu_param, gnu_param_list);
 
                /* If a parameter is a pointer, this function may modify
                   memory through it and thus shouldn't be considered
@@ -3580,11 +3849,11 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                   between two calls, so they can't be CSE'ed.  The latter
                   case also handles by-ref parameters.  */
                if (POINTER_TYPE_P (gnu_param_type)
-                   ||  TYPE_FAT_POINTER_P (gnu_param_type))
+                   || TYPE_FAT_POINTER_P (gnu_param_type))
                  pure_flag = false;
              }
 
-           if (copy_in_copy_out_flag)
+           if (copy_in_copy_out)
              {
                if (!has_copy_in_out)
                  {
@@ -3607,26 +3876,9 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
        /* Do not compute record for out parameters if subprogram is
           stubbed since structures are incomplete for the back-end.  */
-       if (gnu_field_list
-           && Convention (gnat_entity) != Convention_Stubbed)
-           {
-           /* If all types are not complete, defer emission of debug
-              information for this record types. Otherwise, we risk emitting
-              debug information for a dummy type contained in the fields
-              for that record.  */
-           finish_record_type (gnu_return_type, nreverse (gnu_field_list),
-                               false, defer_incomplete_level);
-
-           if (defer_incomplete_level)
-             {
-               debug_deferred = true;
-               defer_debug_level++;
-
-               defer_debug_incomplete_list
-                 = tree_cons (NULL_TREE, gnu_return_type,
-                              defer_debug_incomplete_list);
-             }
-         }
+       if (gnu_field_list && Convention (gnat_entity) != Convention_Stubbed)
+         finish_record_type (gnu_return_type, nreverse (gnu_field_list),
+                             0, false);
 
        /* If we have a CICO list but it has only one entry, we convert
           this function into a function that simply returns that one
@@ -3634,31 +3886,32 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        if (list_length (gnu_return_list) == 1)
          gnu_return_type = TREE_TYPE (TREE_PURPOSE (gnu_return_list));
 
-#ifdef TARGET_DLLIMPORT_DECL_ATTRIBUTES
-       if (Convention (gnat_entity) == Convention_Stdcall)
-         {
-           struct attrib *attr
-             = (struct attrib *) xmalloc (sizeof (struct attrib));
-
-           attr->next = attr_list;
-           attr->type = ATTR_MACHINE_ATTRIBUTE;
-           attr->name = get_identifier ("stdcall");
-           attr->args = NULL_TREE;
-           attr->error_point = gnat_entity;
-           attr_list = attr;
-         }
-#endif
+       if (Has_Stdcall_Convention (gnat_entity))
+         prepend_one_attribute_to
+           (&attr_list, ATTR_MACHINE_ATTRIBUTE,
+            get_identifier ("stdcall"), NULL_TREE,
+            gnat_entity);
 
-       /* Both lists ware built in reverse.  */
+       /* The lists have been built in reverse.  */
        gnu_param_list = nreverse (gnu_param_list);
+       if (has_stub)
+         gnu_stub_param_list = nreverse (gnu_stub_param_list);
        gnu_return_list = nreverse (gnu_return_list);
 
+       if (Ekind (gnat_entity) == E_Function)
+         Set_Mechanism (gnat_entity,
+                        (returns_by_ref || returns_unconstrained
+                         ? By_Reference : By_Copy));
        gnu_type
          = create_subprog_type (gnu_return_type, gnu_param_list,
                                 gnu_return_list, returns_unconstrained,
-                                returns_by_ref,
-                                Function_Returns_With_DSP (gnat_entity),
-                                returns_by_target_ptr);
+                                returns_by_ref, returns_by_target_ptr);
+
+       if (has_stub)
+         gnu_stub_type
+           = create_subprog_type (gnu_return_type, gnu_stub_param_list,
+                                  gnu_return_list, returns_unconstrained,
+                                  returns_by_ref, returns_by_target_ptr);
 
        /* A subprogram (something that doesn't return anything) shouldn't
           be considered Pure since there would be no reason for such a
@@ -3667,30 +3920,47 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        if (TREE_CODE (gnu_return_type) == VOID_TYPE)
          pure_flag = false;
 
+       /* The semantics of "pure" in Ada essentially matches that of "const"
+          in the back-end.  In particular, both properties are orthogonal to
+          the "nothrow" property.  But this is true only if the EH circuitry
+          is explicit in the internal representation of the back-end.  If we
+          are to completely hide the EH circuitry from it, we need to declare
+          that calls to pure Ada subprograms that can throw have side effects
+          since they can trigger an "abnormal" transfer of control flow; thus
+          they can be neither "const" nor "pure" in the back-end sense.  */
        gnu_type
          = build_qualified_type (gnu_type,
-                                 (TYPE_QUALS (gnu_type)
-                                  | (TYPE_QUAL_CONST * pure_flag)
-                                  | (TYPE_QUAL_VOLATILE * volatile_flag)));
+                                 TYPE_QUALS (gnu_type)
+                                 | (Exception_Mechanism == Back_End_Exceptions
+                                    ? TYPE_QUAL_CONST * pure_flag : 0)
+                                 | (TYPE_QUAL_VOLATILE * volatile_flag));
 
        Sloc_to_locus (Sloc (gnat_entity), &input_location);
 
-        /* If we have a builtin decl for that function, check the signatures
-           compatibilities.  If the signatures are compatible, use the builtin
-           decl.  If they are not, we expect the checker predicate to have
-           posted the appropriate errors, and just continue with what we have
-           so far.  */
-        if (gnu_builtin_decl)
-          {
-            tree gnu_builtin_type =  TREE_TYPE (gnu_builtin_decl);
-
-            if (compatible_signatures_p (gnu_type, gnu_builtin_type))
-              {
-                gnu_decl = gnu_builtin_decl;
-                gnu_type = gnu_builtin_type;
-                break;
-              }
-          }
+       if (has_stub)
+         gnu_stub_type
+           = build_qualified_type (gnu_stub_type,
+                                   TYPE_QUALS (gnu_stub_type)
+                                   | (Exception_Mechanism == Back_End_Exceptions
+                                      ? TYPE_QUAL_CONST * pure_flag : 0)
+                                   | (TYPE_QUAL_VOLATILE * volatile_flag));
+
+       /* If we have a builtin decl for that function, check the signatures
+          compatibilities.  If the signatures are compatible, use the builtin
+          decl.  If they are not, we expect the checker predicate to have
+          posted the appropriate errors, and just continue with what we have
+          so far.  */
+       if (gnu_builtin_decl)
+         {
+           tree gnu_builtin_type = TREE_TYPE (gnu_builtin_decl);
+
+           if (compatible_signatures_p (gnu_type, gnu_builtin_type))
+             {
+               gnu_decl = gnu_builtin_decl;
+               gnu_type = gnu_builtin_type;
+               break;
+             }
+         }
 
        /* If there was no specified Interface_Name and the external and
           internal names of the subprogram are the same, only use the
@@ -3717,7 +3987,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
            save_gnu_tree (gnat_entity, NULL_TREE, false);
 
-           gnu_type = build_reference_type (gnu_type);
+           /* Convert the type of the object to a reference type that can
+              alias everything as per 13.3(19).  */
+           gnu_type
+             = build_reference_type_for_mode (gnu_type, ptr_mode, true);
            if (gnu_address)
              gnu_address = convert (gnu_type, gnu_address);
 
@@ -3731,15 +4004,33 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
        else if (kind == E_Subprogram_Type)
          gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
                                       !Comes_From_Source (gnat_entity),
-                                      debug_info_p && !defer_incomplete_level,
-                                      gnat_entity);
+                                      debug_info_p, gnat_entity);
        else
          {
+           if (has_stub)
+             {
+               gnu_stub_name = gnu_ext_name;
+               gnu_ext_name = create_concat_name (gnat_entity, "internal");
+               public_flag = false;
+             }
+
            gnu_decl = create_subprog_decl (gnu_entity_id, gnu_ext_name,
                                            gnu_type, gnu_param_list,
                                            inline_flag, public_flag,
                                            extern_flag, attr_list,
                                            gnat_entity);
+           if (has_stub)
+             {
+               tree gnu_stub_decl
+                 = create_subprog_decl (gnu_entity_id, gnu_stub_name,
+                                        gnu_stub_type, gnu_stub_param_list,
+                                        inline_flag, true,
+                                        extern_flag, attr_list,
+                                        gnat_entity);
+               SET_DECL_FUNCTION_STUB (gnu_decl, gnu_stub_decl);
+             }
+
+           /* This is unrelated to the stub built right above.  */
            DECL_STUBBED_P (gnu_decl)
              = Convention (gnat_entity) == Convention_Stubbed;
          }
@@ -3747,71 +4038,81 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       break;
 
     case E_Incomplete_Type:
+    case E_Incomplete_Subtype:
     case E_Private_Type:
-    case E_Limited_Private_Type:
-    case E_Record_Type_With_Private:
     case E_Private_Subtype:
+    case E_Limited_Private_Type:
     case E_Limited_Private_Subtype:
+    case E_Record_Type_With_Private:
     case E_Record_Subtype_With_Private:
+      {
+       /* Get the "full view" of this entity.  If this is an incomplete
+          entity from a limited with, treat its non-limited view as the
+          full view.  Otherwise, use either the full view or the underlying
+          full view, whichever is present.  This is used in all the tests
+          below.  */
+       Entity_Id full_view
+         = (IN (Ekind (gnat_entity), Incomplete_Kind)
+            && From_With_Type (gnat_entity))
+           ? Non_Limited_View (gnat_entity)
+           : Present (Full_View (gnat_entity))
+             ? Full_View (gnat_entity)
+             : Underlying_Full_View (gnat_entity);
+
+       /* If this is an incomplete type with no full view, it must be a Taft
+          Amendment type, in which case we return a dummy type.  Otherwise,
+          just get the type from its Etype.  */
+       if (No (full_view))
+         {
+           if (kind == E_Incomplete_Type)
+             gnu_type = make_dummy_type (gnat_entity);
+           else
+             {
+               gnu_decl = gnat_to_gnu_entity (Etype (gnat_entity),
+                                              NULL_TREE, 0);
+               maybe_present = true;
+             }
+           break;
+         }
 
-      /* If this type does not have a full view in the unit we are
-        compiling, then just get the type from its Etype.  */
-      if (No (Full_View (gnat_entity)))
-       {
-         /* If this is an incomplete type with no full view, it must
-            be a Taft Amendement type, so just return a dummy type.  */
-         if (kind == E_Incomplete_Type)
-           gnu_type = make_dummy_type (gnat_entity);
-
-          else if (Present (Underlying_Full_View (gnat_entity)))
-             gnu_decl = gnat_to_gnu_entity (Underlying_Full_View (gnat_entity),
-                                           NULL_TREE, 0);
-         else
-           {
-             gnu_decl = gnat_to_gnu_entity (Etype (gnat_entity),
-                                            NULL_TREE, 0);
-             maybe_present = true;
-           }
-
-         break;
-       }
-
-      /* Otherwise, if we are not defining the type now, get the
-        type from the full view. But always get the type from the full
-        view for define on use types, since otherwise we won't see them! */
+       /* If we already made a type for the full view, reuse it.  */
+       else if (present_gnu_tree (full_view))
+         {
+           gnu_decl = get_gnu_tree (full_view);
+           break;
+         }
 
-      else if (!definition
-              || (Is_Itype (Full_View (gnat_entity))
+       /* Otherwise, if we are not defining the type now, get the type
+          from the full view.  But always get the type from the full view
+          for define on use types, since otherwise we won't see them!  */
+       else if (!definition
+                || (Is_Itype (full_view)
                   && No (Freeze_Node (gnat_entity)))
-              || (Is_Itype (gnat_entity)
-                  && No (Freeze_Node (Full_View (gnat_entity)))))
-       {
-         gnu_decl = gnat_to_gnu_entity (Full_View (gnat_entity),
-                                         NULL_TREE, 0);
-         maybe_present = true;
-         break;
-       }
+                || (Is_Itype (gnat_entity)
+                  && No (Freeze_Node (full_view))))
+         {
+           gnu_decl = gnat_to_gnu_entity (full_view, NULL_TREE, 0);
+           maybe_present = true;
+           break;
+         }
 
-      /* For incomplete types, make a dummy type entry which will be
-        replaced later.  */
-      gnu_type = make_dummy_type (gnat_entity);
+       /* For incomplete types, make a dummy type entry which will be
+          replaced later.  */
+       gnu_type = make_dummy_type (gnat_entity);
 
-      /* Save this type as the full declaration's type so we can do any needed
-        updates when we see it.  */
-      gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
-                                  !Comes_From_Source (gnat_entity),
-                                  debug_info_p, gnat_entity);
-      save_gnu_tree (Full_View (gnat_entity), gnu_decl, false);
-      break;
+       /* Save this type as the full declaration's type so we can do any
+          needed updates when we see it.  */
+       gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
+                                    !Comes_From_Source (gnat_entity),
+                                    debug_info_p, gnat_entity);
+       save_gnu_tree (full_view, gnu_decl, 0);
+       break;
+      }
 
       /* Simple class_wide types are always viewed as their root_type
         by Gigi unless an Equivalent_Type is specified.  */
     case E_Class_Wide_Type:
-      if (Present (Equivalent_Type (gnat_entity)))
-       gnu_type = gnat_to_gnu_type (Equivalent_Type (gnat_entity));
-      else
-       gnu_type = gnat_to_gnu_type (Root_Type (gnat_entity));
-
+      gnu_decl = gnat_to_gnu_entity (gnat_equiv_type, NULL_TREE, 0);
       maybe_present = true;
       break;
 
@@ -3819,10 +4120,10 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
     case E_Task_Subtype:
     case E_Protected_Type:
     case E_Protected_Subtype:
-      if (type_annotate_only && No (Corresponding_Record_Type (gnat_entity)))
+      if (type_annotate_only && No (gnat_equiv_type))
        gnu_type = void_type_node;
       else
-       gnu_type = gnat_to_gnu_type (Corresponding_Record_Type (gnat_entity));
+       gnu_type = gnat_to_gnu_type (gnat_equiv_type);
 
       maybe_present = true;
       break;
@@ -3859,7 +4160,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
     {
       if (Is_Tagged_Type (gnat_entity)
          || Is_Class_Wide_Equivalent_Type (gnat_entity))
-        TYPE_ALIGN_OK (gnu_type) = 1;
+       TYPE_ALIGN_OK (gnu_type) = 1;
 
       if (AGGREGATE_TYPE_P (gnu_type) && Is_By_Reference_Type (gnat_entity))
        TYPE_BY_REFERENCE_P (gnu_type) = 1;
@@ -3891,8 +4192,38 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       if (align != 0 || TREE_CODE (gnu_type) == UNCONSTRAINED_ARRAY_TYPE)
        ;
       else if (Known_Alignment (gnat_entity))
-       align = validate_alignment (Alignment (gnat_entity), gnat_entity,
-                                   TYPE_ALIGN (gnu_type));
+       {
+         align = validate_alignment (Alignment (gnat_entity), gnat_entity,
+                                     TYPE_ALIGN (gnu_type));
+
+         /* Warn on suspiciously large alignments.  This should catch
+            errors about the (alignment,byte)/(size,bit) discrepancy.  */
+         if (align > BIGGEST_ALIGNMENT && Has_Alignment_Clause (gnat_entity))
+           {
+             tree size;
+
+             /* If a size was specified, take it into account.  Otherwise
+                use the RM size for records as the type size has already
+                been adjusted to the alignment.  */
+             if (gnu_size)
+               size = gnu_size;
+             else if ((TREE_CODE (gnu_type) == RECORD_TYPE
+                       || TREE_CODE (gnu_type) == UNION_TYPE
+                       || TREE_CODE (gnu_type) == QUAL_UNION_TYPE)
+                      && !TYPE_IS_FAT_POINTER_P (gnu_type))
+               size = rm_size (gnu_type);
+             else
+               size = TYPE_SIZE (gnu_type);
+
+             /* Consider an alignment as suspicious if the alignment/size
+                ratio is greater or equal to the byte/bit ratio.  */
+             if (host_integerp (size, 1)
+                 && align >= TREE_INT_CST_LOW (size) * BITS_PER_UNIT)
+               post_error_ne ("?suspiciously large alignment specified for&",
+                              Expression (Alignment_Clause (gnat_entity)),
+                              gnat_entity);
+           }
+       }
       else if (Is_Atomic (gnat_entity) && !gnu_size
               && host_integerp (TYPE_SIZE (gnu_type), 1)
               && integer_pow2p (TYPE_SIZE (gnu_type)))
@@ -3990,6 +4321,7 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                 right now, we have to put in an explicit multiply and
                 divide by that value.  */
              if (!CONTAINS_PLACEHOLDER_P (DECL_FIELD_OFFSET (gnu_field)))
+               {
                DECL_FIELD_OFFSET (gnu_field)
                  = build_binary_op
                    (MULT_EXPR, sizetype,
@@ -4002,6 +4334,13 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
                      get_identifier ("OFFSET"),
                      definition, 0),
                     size_int (DECL_OFFSET_ALIGN (gnu_field) / BITS_PER_UNIT));
+
+               /* ??? The context of gnu_field is not necessarily gnu_type so
+                  the MULT_EXPR node built above may not be marked by the call
+                  to create_type_decl below.  Mark it manually for now.  */
+               if (global_bindings_p ())
+                 TREE_VISITED (DECL_FIELD_OFFSET (gnu_field)) = 1;
+               }
            }
 
       gnu_type = build_qualified_type (gnu_type,
@@ -4012,9 +4351,12 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
       if (Is_Atomic (gnat_entity))
        check_ok_for_atomic (gnu_type, gnat_entity, false);
 
-      if (Known_Alignment (gnat_entity))
+      if (Present (Alignment_Clause (gnat_entity)))
        TYPE_USER_ALIGN (gnu_type) = 1;
 
+      if (Universal_Aliasing (gnat_entity))
+       TYPE_UNIVERSAL_ALIASING_P (TYPE_MAIN_VARIANT (gnu_type)) = 1;
+
       if (!gnu_decl)
        gnu_decl = create_type_decl (gnu_entity_id, gnu_type, attr_list,
                                     !Comes_From_Source (gnat_entity),
@@ -4044,31 +4386,29 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
          Set_Esize (gnat_entity, annotate_value (gnu_size));
 
-          if (type_annotate_only && Is_Tagged_Type (gnat_entity))
+         if (type_annotate_only && Is_Tagged_Type (gnat_entity))
            {
              /* In this mode the tag and the parent components are not
                 generated by the front-end, so the sizes must be adjusted
                 explicitly now. */
+             int size_offset, new_size;
+
+             if (Is_Derived_Type (gnat_entity))
+               {
+                 size_offset
+                   = UI_To_Int (Esize (Etype (Base_Type (gnat_entity))));
+                 Set_Alignment (gnat_entity,
+                                Alignment (Etype (Base_Type (gnat_entity))));
+               }
+             else
+               size_offset = POINTER_SIZE;
 
-             int size_offset;
-             int new_size;
-
-             if (Is_Derived_Type (gnat_entity))
-              {
-                 size_offset
-                  = UI_To_Int (Esize (Etype (Base_Type (gnat_entity))));
-                 Set_Alignment (gnat_entity,
-                               Alignment (Etype (Base_Type (gnat_entity))));
-              }
-             else
-               size_offset = POINTER_SIZE;
-
-             new_size = UI_To_Int (Esize (gnat_entity)) + size_offset;
-             Set_Esize (gnat_entity,
-                       UI_From_Int (((new_size + (POINTER_SIZE - 1))
-                                     / POINTER_SIZE) * POINTER_SIZE));
-             Set_RM_Size (gnat_entity, Esize (gnat_entity));
-           }
+             new_size = UI_To_Int (Esize (gnat_entity)) + size_offset;
+             Set_Esize (gnat_entity,
+                        UI_From_Int (((new_size + (POINTER_SIZE - 1))
+                                      / POINTER_SIZE) * POINTER_SIZE));
+             Set_RM_Size (gnat_entity, Esize (gnat_entity));
+           }
        }
 
       if (Unknown_RM_Size (gnat_entity) && rm_size (gnu_type))
@@ -4079,7 +4419,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
     DECL_ARTIFICIAL (gnu_decl) = 1;
 
   if (!debug_info_p && DECL_P (gnu_decl)
-      && TREE_CODE (gnu_decl) != FUNCTION_DECL)
+      && TREE_CODE (gnu_decl) != FUNCTION_DECL
+      && No (Renamed_Object (gnat_entity)))
     DECL_IGNORED_P (gnu_decl) = 1;
 
   /* If we haven't already, associate the ..._DECL node that we just made with
@@ -4115,31 +4456,56 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
 
       if (TREE_CODE (gnu_scalar_type) == ENUMERAL_TYPE)
        {
-         TYPE_STUB_DECL (gnu_scalar_type) = gnu_decl;
-
          /* Since this has both a typedef and a tag, avoid outputting
             the name twice.  */
          DECL_ARTIFICIAL (gnu_decl) = 1;
-         rest_of_type_compilation (gnu_scalar_type, global_bindings_p ());
+         rest_of_type_decl_compilation (gnu_decl);
        }
     }
 
   /* If we deferred processing of incomplete types, re-enable it.  If there
      were no other disables and we have some to process, do so.  */
-  if (this_deferred && --defer_incomplete_level == 0 && defer_incomplete_list)
+  if (this_deferred && --defer_incomplete_level == 0)
     {
-      struct incomplete *incp = defer_incomplete_list;
-      struct incomplete *next;
+      if (defer_incomplete_list)
+       {
+         struct incomplete *incp, *next;
+
+         /* We are back to level 0 for the deferring of incomplete types.
+            But processing these incomplete types below may itself require
+            deferring, so preserve what we have and restart from scratch.  */
+         incp = defer_incomplete_list;
+         defer_incomplete_list = NULL;
 
-      defer_incomplete_list = NULL;
-      for (; incp; incp = next)
+         /* For finalization, however, all types must be complete so we
+            cannot do the same because deferred incomplete types may end up
+            referencing each other.  Process them all recursively first.  */
+         defer_finalize_level++;
+
+         for (; incp; incp = next)
+           {
+             next = incp->next;
+
+             if (incp->old_type)
+               update_pointer_to (TYPE_MAIN_VARIANT (incp->old_type),
+                                  gnat_to_gnu_type (incp->full_type));
+             free (incp);
+           }
+
+         defer_finalize_level--;
+       }
+
+      /* All the deferred incomplete types have been processed so we can
+        now proceed with the finalization of the deferred types.  */
+      if (defer_finalize_level == 0 && defer_finalize_list)
        {
-         next = incp->next;
+         unsigned int i;
+         tree t;
 
-         if (incp->old_type)
-           update_pointer_to (TYPE_MAIN_VARIANT (incp->old_type),
-                              gnat_to_gnu_type (incp->full_type));
-         free (incp);
+         for (i = 0; VEC_iterate (tree, defer_finalize_list, i, t); i++)
+           rest_of_type_decl_compilation_no_defer (t);
+
+         VEC_free (tree, heap, defer_finalize_list);
        }
     }
 
@@ -4158,31 +4524,6 @@ gnat_to_gnu_entity (Entity_Id gnat_entity, tree gnu_expr, int definition)
          }
     }
 
-   /* If there are no incomplete types and we have deferred emission
-      of debug information, check whether we have finished defining
-      all nested records.
-      If so, handle the list now.  */
-
-   if (debug_deferred)
-     defer_debug_level--;
-
-   if (defer_debug_incomplete_list
-       && !defer_incomplete_level
-       && !defer_debug_level)
-    {
-      tree c, n;
-
-      defer_debug_incomplete_list = nreverse (defer_debug_incomplete_list);
-
-      for (c = defer_debug_incomplete_list; c; c = n)
-       {
-         n = TREE_CHAIN (c);
-         write_record_type_debug_info (TREE_VALUE (c));
-       }
-
-      defer_debug_incomplete_list = 0;
-    }
-
   if (this_global)
     force_global--;
 
@@ -4208,6 +4549,322 @@ gnat_to_gnu_field_decl (Entity_Id gnat_entity)
 
   return gnu_field;
 }
+
+/* Wrap up compilation of DECL, a TYPE_DECL, possibly deferring it.
+   Every TYPE_DECL generated for a type definition must be passed
+   to this function once everything else has been done for it.  */
+
+void
+rest_of_type_decl_compilation (tree decl)
+{
+  /* We need to defer finalizing the type if incomplete types
+     are being deferred or if they are being processed.  */
+  if (defer_incomplete_level || defer_finalize_level)
+    VEC_safe_push (tree, heap, defer_finalize_list, decl);
+  else
+    rest_of_type_decl_compilation_no_defer (decl);
+}
+
+/* Same as above but without deferring the compilation.  This
+   function should not be invoked directly on a TYPE_DECL.  */
+
+static void
+rest_of_type_decl_compilation_no_defer (tree decl)
+{
+  const int toplev = global_bindings_p ();
+  tree t = TREE_TYPE (decl);
+
+  rest_of_decl_compilation (decl, toplev, 0);
+
+  /* Now process all the variants.  This is needed for STABS.  */
+  for (t = TYPE_MAIN_VARIANT (t); t; t = TYPE_NEXT_VARIANT (t))
+    {
+      if (t == TREE_TYPE (decl))
+       continue;
+
+      if (!TYPE_STUB_DECL (t))
+       {
+         TYPE_STUB_DECL (t) = build_decl (TYPE_DECL, DECL_NAME (decl), t);
+         DECL_ARTIFICIAL (TYPE_STUB_DECL (t)) = 1;
+       }
+
+      rest_of_type_compilation (t, toplev);
+    }
+}
+
+/* Finalize any From_With_Type incomplete types.  We do this after processing
+   our compilation unit and after processing its spec, if this is a body.  */
+
+void
+finalize_from_with_types (void)
+{
+  struct incomplete *incp = defer_limited_with;
+  struct incomplete *next;
+
+  defer_limited_with = 0;
+  for (; incp; incp = next)
+    {
+      next = incp->next;
+
+      if (incp->old_type != 0)
+       update_pointer_to (TYPE_MAIN_VARIANT (incp->old_type),
+                          gnat_to_gnu_type (incp->full_type));
+      free (incp);
+    }
+}
+
+/* Return the equivalent type to be used for GNAT_ENTITY, if it's a
+   kind of type (such E_Task_Type) that has a different type which Gigi
+   uses for its representation.  If the type does not have a special type
+   for its representation, return GNAT_ENTITY.  If a type is supposed to
+   exist, but does not, abort unless annotating types, in which case
+   return Empty.  If GNAT_ENTITY is Empty, return Empty.  */
+
+Entity_Id
+Gigi_Equivalent_Type (Entity_Id gnat_entity)
+{
+  Entity_Id gnat_equiv = gnat_entity;
+
+  if (No (gnat_entity))
+    return gnat_entity;
+
+  switch (Ekind (gnat_entity))
+    {
+    case E_Class_Wide_Subtype:
+      if (Present (Equivalent_Type (gnat_entity)))
+       gnat_equiv = Equivalent_Type (gnat_entity);
+      break;
+
+    case E_Access_Protected_Subprogram_Type:
+    case E_Anonymous_Access_Protected_Subprogram_Type:
+      gnat_equiv = Equivalent_Type (gnat_entity);
+      break;
+
+    case E_Class_Wide_Type:
+      gnat_equiv = ((Present (Equivalent_Type (gnat_entity)))
+                   ? Equivalent_Type (gnat_entity)
+                   : Root_Type (gnat_entity));
+      break;
+
+    case E_Task_Type:
+    case E_Task_Subtype:
+    case E_Protected_Type:
+    case E_Protected_Subtype:
+      gnat_equiv = Corresponding_Record_Type (gnat_entity);
+      break;
+
+    default:
+      break;
+    }
+
+  gcc_assert (Present (gnat_equiv) || type_annotate_only);
+  return gnat_equiv;
+}
+
+/* Return a GCC tree for a parameter corresponding to GNAT_PARAM and
+   using MECH as its passing mechanism, to be placed in the parameter
+   list built for GNAT_SUBPROG.  Assume a foreign convention for the
+   latter if FOREIGN is true.  Also set CICO to true if the parameter
+   must use the copy-in copy-out implementation mechanism.
+
+   The returned tree is a PARM_DECL, except for those cases where no
+   parameter needs to be actually passed to the subprogram; the type
+   of this "shadow" parameter is then returned instead.  */
+
+static tree
+gnat_to_gnu_param (Entity_Id gnat_param, Mechanism_Type mech,
+                  Entity_Id gnat_subprog, bool foreign, bool *cico)
+{
+  tree gnu_param_name = get_entity_name (gnat_param);
+  tree gnu_param_type = gnat_to_gnu_type (Etype (gnat_param));
+  bool in_param = (Ekind (gnat_param) == E_In_Parameter);
+  /* The parameter can be indirectly modified if its address is taken.  */
+  bool ro_param = in_param && !Address_Taken (gnat_param);
+  bool by_return = false, by_component_ptr = false, by_ref = false;
+  tree gnu_param;
+
+  /* Copy-return is used only for the first parameter of a valued procedure.
+     It's a copy mechanism for which a parameter is never allocated.  */
+  if (mech == By_Copy_Return)
+    {
+      gcc_assert (Ekind (gnat_param) == E_Out_Parameter);
+      mech = By_Copy;
+      by_return = true;
+    }
+
+  /* If this is either a foreign function or if the underlying type won't
+     be passed by reference, strip off possible padding type.  */
+  if (TREE_CODE (gnu_param_type) == RECORD_TYPE
+      && TYPE_IS_PADDING_P (gnu_param_type))
+    {
+      tree unpadded_type = TREE_TYPE (TYPE_FIELDS (gnu_param_type));
+
+      if (mech == By_Reference
+         || foreign
+         || (!must_pass_by_ref (unpadded_type)
+             && (mech == By_Copy || !default_pass_by_ref (unpadded_type))))
+       gnu_param_type = unpadded_type;
+    }
+
+  /* If this is a read-only parameter, make a variant of the type that is
+     read-only.  ??? However, if this is an unconstrained array, that type
+     can be very complex, so skip it for now.  Likewise for any other
+     self-referential type.  */
+  if (ro_param
+      && TREE_CODE (gnu_param_type) != UNCONSTRAINED_ARRAY_TYPE
+      && !CONTAINS_PLACEHOLDER_P (TYPE_SIZE (gnu_param_type)))
+    gnu_param_type = build_qualified_type (gnu_param_type,
+                                          (TYPE_QUALS (gnu_param_type)
+                                           | TYPE_QUAL_CONST));
+
+  /* For foreign conventions, pass arrays as pointers to the element type.
+     First check for unconstrained array and get the underlying array.  */
+  if (foreign && TREE_CODE (gnu_param_type) == UNCONSTRAINED_ARRAY_TYPE)
+    gnu_param_type
+      = TREE_TYPE (TREE_TYPE (TYPE_FIELDS (TREE_TYPE (gnu_param_type))));
+
+  /* VMS descriptors are themselves passed by reference.  */
+  if (mech == By_Descriptor)
+    gnu_param_type
+      = build_pointer_type (build_vms_descriptor (gnu_param_type,
+                                                 Mechanism (gnat_param),
+                                                 gnat_subprog));
+
+  /* Arrays are passed as pointers to element type for foreign conventions.  */
+  else if (foreign
+          && mech != By_Copy
+          && TREE_CODE (gnu_param_type) == ARRAY_TYPE)
+    {
+      /* Strip off any multi-dimensional entries, then strip
+        off the last array to get the component type.  */
+      while (TREE_CODE (TREE_TYPE (gnu_param_type)) == ARRAY_TYPE
+            && TYPE_MULTI_ARRAY_P (TREE_TYPE (gnu_param_type)))
+       gnu_param_type = TREE_TYPE (gnu_param_type);
+
+      by_component_ptr = true;
+      gnu_param_type = TREE_TYPE (gnu_param_type);
+
+      if (ro_param)
+       gnu_param_type = build_qualified_type (gnu_param_type,
+                                              (TYPE_QUALS (gnu_param_type)
+                                               | TYPE_QUAL_CONST));
+
+      gnu_param_type = build_pointer_type (gnu_param_type);
+    }
+
+  /* Fat pointers are passed as thin pointers for foreign conventions.  */
+  else if (foreign && TYPE_FAT_POINTER_P (gnu_param_type))
+    gnu_param_type
+      = make_type_from_size (gnu_param_type, size_int (POINTER_SIZE), 0);
+
+  /* If we must pass or were requested to pass by reference, do so.
+     If we were requested to pass by copy, do so.
+     Otherwise, for foreign conventions, pass In Out or Out parameters
+     or aggregates by reference.  For COBOL and Fortran, pass all
+     integer and FP types that way too.  For Convention Ada, use
+     the standard Ada default.  */
+  else if (must_pass_by_ref (gnu_param_type)
+          || mech == By_Reference
+          || (mech != By_Copy
+              && ((foreign
+                   && (!in_param || AGGREGATE_TYPE_P (gnu_param_type)))
+                  || (foreign
+                      && (Convention (gnat_subprog) == Convention_Fortran
+                          || Convention (gnat_subprog) == Convention_COBOL)
+                      && (INTEGRAL_TYPE_P (gnu_param_type)
+                          || FLOAT_TYPE_P (gnu_param_type)))
+                  || (!foreign
+                      && default_pass_by_ref (gnu_param_type)))))
+    {
+      gnu_param_type = build_reference_type (gnu_param_type);
+      by_ref = true;
+    }
+
+  /* Pass In Out or Out parameters using copy-in copy-out mechanism.  */
+  else if (!in_param)
+    *cico = true;
+
+  if (mech == By_Copy && (by_ref || by_component_ptr))
+    post_error ("?cannot pass & by copy", gnat_param);
+
+  /* If this is an Out parameter that isn't passed by reference and isn't
+     a pointer or aggregate, we don't make a PARM_DECL for it.  Instead,
+     it will be a VAR_DECL created when we process the procedure, so just
+     return its type.  For the special parameter of a valued procedure,
+     never pass it in.
+
+     An exception is made to cover the RM-6.4.1 rule requiring "by copy"
+     Out parameters with discriminants or implicit initial values to be
+     handled like In Out parameters.  These type are normally built as
+     aggregates, hence passed by reference, except for some packed arrays
+     which end up encoded in special integer types.
+
+     The exception we need to make is then for packed arrays of records
+     with discriminants or implicit initial values.  We have no light/easy
+     way to check for the latter case, so we merely check for packed arrays
+     of records.  This may lead to useless copy-in operations, but in very
+     rare cases only, as these would be exceptions in a set of already
+     exceptional situations.  */
+  if (Ekind (gnat_param) == E_Out_Parameter
+      && !by_ref
+      && (by_return
+         || (mech != By_Descriptor
+             && !POINTER_TYPE_P (gnu_param_type)
+             && !AGGREGATE_TYPE_P (gnu_param_type)))
+      && !(Is_Array_Type (Etype (gnat_param))
+          && Is_Packed (Etype (gnat_param))
+          && Is_Composite_Type (Component_Type (Etype (gnat_param)))))
+    return gnu_param_type;
+
+  gnu_param = create_param_decl (gnu_param_name, gnu_param_type,
+                                ro_param || by_ref || by_component_ptr);
+  DECL_BY_REF_P (gnu_param) = by_ref;
+  DECL_BY_COMPONENT_PTR_P (gnu_param) = by_component_ptr;
+  DECL_BY_DESCRIPTOR_P (gnu_param) = (mech == By_Descriptor);
+  DECL_POINTS_TO_READONLY_P (gnu_param)
+    = (ro_param && (by_ref || by_component_ptr));
+
+  /* If no Mechanism was specified, indicate what we're using, then
+     back-annotate it.  */
+  if (mech == Default)
+    mech = (by_ref || by_component_ptr) ? By_Reference : By_Copy;
+
+  Set_Mechanism (gnat_param, mech);
+  return gnu_param;
+}
+
+/* Return true if DISCR1 and DISCR2 represent the same discriminant.  */
+
+static bool
+same_discriminant_p (Entity_Id discr1, Entity_Id discr2)
+{
+  while (Present (Corresponding_Discriminant (discr1)))
+    discr1 = Corresponding_Discriminant (discr1);
+
+  while (Present (Corresponding_Discriminant (discr2)))
+    discr2 = Corresponding_Discriminant (discr2);
+
+  return
+    Original_Record_Component (discr1) == Original_Record_Component (discr2);
+}
+
+/* Return true if the array type specified by GNAT_TYPE and GNU_TYPE has
+   a non-aliased component in the back-end sense.  */
+
+static bool
+array_type_has_nonaliased_component (Entity_Id gnat_type, tree gnu_type)
+{
+  /* If the type below this is a multi-array type, then
+     this does not have aliased components.  */
+  if (TREE_CODE (TREE_TYPE (gnu_type)) == ARRAY_TYPE
+      && TYPE_MULTI_ARRAY_P (TREE_TYPE (gnu_type)))
+    return true;
+
+  if (Has_Aliased_Components (gnat_type))
+    return false;
+
+  return type_for_nonaliased_component_p (TREE_TYPE (gnu_type));
+}
 \f
 /* Given GNAT_ENTITY, elaborate all expressions that are required to
    be elaborated at the point of its definition, but do nothing else.  */
@@ -4242,7 +4899,7 @@ elaborate_entity (Entity_Id gnat_entity)
 
     case E_Record_Type:
       {
-        Node_Id full_definition = Declaration_Node (gnat_entity);
+       Node_Id full_definition = Declaration_Node (gnat_entity);
        Node_Id record_definition = Type_Definition (full_definition);
 
        /* If this is a record extension, go a level further to find the
@@ -4257,7 +4914,7 @@ elaborate_entity (Entity_Id gnat_entity)
     case E_Limited_Private_Subtype:
     case E_Record_Subtype_With_Private:
       if (Is_Constrained (gnat_entity)
-          && Has_Discriminants (Base_Type (gnat_entity))
+         && Has_Discriminants (Base_Type (gnat_entity))
          && Present (Discriminant_Constraint (gnat_entity)))
        {
          Node_Id gnat_discriminant_expr;
@@ -4359,7 +5016,7 @@ copy_alias_set (tree gnu_new_type, tree gnu_old_type)
 
 static tree
 substitution_list (Entity_Id gnat_subtype, Entity_Id gnat_type,
-                   tree gnu_list, bool definition)
+                  tree gnu_list, bool definition)
 {
   Entity_Id gnat_discrim;
   Node_Id gnat_value;
@@ -4385,73 +5042,6 @@ substitution_list (Entity_Id gnat_subtype, Entity_Id gnat_type,
   return gnu_list;
 }
 \f
-/* For the following two functions: for each GNAT entity, the GCC
-   tree node used as a dummy for that entity, if any.  */
-
-static GTY((length ("max_gnat_nodes"))) tree * dummy_node_table;
-
-/* Initialize the above table.  */
-
-void
-init_dummy_type (void)
-{
-  Node_Id gnat_node;
-
-  dummy_node_table = (tree *) ggc_alloc (max_gnat_nodes * sizeof (tree));
-
-  for (gnat_node = 0; gnat_node < max_gnat_nodes; gnat_node++)
-    dummy_node_table[gnat_node] = NULL_TREE;
-
-  dummy_node_table -= First_Node_Id;
-}
-
-/* Make a dummy type corresponding to GNAT_TYPE.  */
-
-tree
-make_dummy_type (Entity_Id gnat_type)
-{
-  Entity_Id gnat_underlying;
-  tree gnu_type;
-
-  /* Find a full type for GNAT_TYPE, taking into account any class wide
-     types.  */
-  if (Is_Class_Wide_Type (gnat_type) && Present (Equivalent_Type (gnat_type)))
-    gnat_type = Equivalent_Type (gnat_type);
-  else if (Ekind (gnat_type) == E_Class_Wide_Type)
-    gnat_type = Root_Type (gnat_type);
-
-  for (gnat_underlying = gnat_type;
-       (IN (Ekind (gnat_underlying), Incomplete_Or_Private_Kind)
-       && Present (Full_View (gnat_underlying)));
-       gnat_underlying = Full_View (gnat_underlying))
-    ;
-
-  /* If it there already a dummy type, use that one.  Else make one.  */
-  if (dummy_node_table[gnat_underlying])
-    return dummy_node_table[gnat_underlying];
-
-  /* If this is a record, make this a RECORD_TYPE or UNION_TYPE; else make
-     it a VOID_TYPE.  */
-  if (Is_Unchecked_Union (gnat_underlying))
-    {
-      gnu_type = make_node (UNION_TYPE);
-      TYPE_UNCHECKED_UNION_P (gnu_type) = 1;
-    }
-  else if (Is_Record_Type (gnat_underlying))
-    gnu_type = make_node (RECORD_TYPE);
-  else
-    gnu_type = make_node (ENUMERAL_TYPE);
-
-  TYPE_NAME (gnu_type) = get_entity_name (gnat_type);
-  TYPE_DUMMY_P (gnu_type) = 1;
-  if (AGGREGATE_TYPE_P (gnu_type))
-    TYPE_STUB_DECL (gnu_type) = build_decl (TYPE_DECL, NULL_TREE, gnu_type);
-
-  dummy_node_table[gnat_underlying] = gnu_type;
-
-  return gnu_type;
-}
-\f
 /* Return true if the size represented by GNU_SIZE can be handled by an
    allocation.  If STATIC_P is true, consider only what can be done with a
    static allocation.  */
@@ -4466,7 +5056,7 @@ allocatable_size_p (tree gnu_size, bool static_p)
      Storage_Error.  */
   if (!static_p)
     return !(TREE_CODE (gnu_size) == INTEGER_CST
-            && TREE_CONSTANT_OVERFLOW (gnu_size));
+            && TREE_OVERFLOW (gnu_size));
 
   /* Otherwise, we need to deal with both variable sizes and constant
      sizes that won't fit in a host int.  We use int instead of HOST_WIDE_INT
@@ -4478,6 +5068,27 @@ allocatable_size_p (tree gnu_size, bool static_p)
   return (int) our_size == our_size;
 }
 \f
+/* Prepend to ATTR_LIST an entry for an attribute with provided TYPE,
+   NAME, ARGS and ERROR_POINT.  */
+
+static void
+prepend_one_attribute_to (struct attrib ** attr_list,
+                         enum attr_type attr_type,
+                         tree attr_name,
+                         tree attr_args,
+                         Node_Id attr_error_point)
+{
+  struct attrib * attr = (struct attrib *) xmalloc (sizeof (struct attrib));
+
+  attr->type = attr_type;
+  attr->name = attr_name;
+  attr->args = attr_args;
+  attr->error_point = attr_error_point;
+
+  attr->next = *attr_list;
+  *attr_list = attr;
+}
+
 /* Prepend to ATTR_LIST the list of attributes for GNAT_ENTITY, if any.  */
 
 static void
@@ -4489,7 +5100,6 @@ prepend_attributes (Entity_Id gnat_entity, struct attrib ** attr_list)
        gnat_temp = Next_Rep_Item (gnat_temp))
     if (Nkind (gnat_temp) == N_Pragma)
       {
-       struct attrib *attr;
        tree gnu_arg0 = NULL_TREE, gnu_arg1 = NULL_TREE;
        Node_Id gnat_assoc = Pragma_Argument_Associations (gnat_temp);
        enum attr_type etype;
@@ -4513,7 +5123,7 @@ prepend_attributes (Entity_Id gnat_entity, struct attrib ** attr_list)
                                                  (First (gnat_assoc)))))));
          }
 
-       switch (Get_Pragma_Id (Chars (gnat_temp)))
+       switch (Get_Pragma_Id (Chars (Pragma_Identifier (gnat_temp))))
          {
          case Pragma_Machine_Attribute:
            etype = ATTR_MACHINE_ATTRIBUTE;
@@ -4543,23 +5153,16 @@ prepend_attributes (Entity_Id gnat_entity, struct attrib ** attr_list)
            continue;
          }
 
-       attr = (struct attrib *) xmalloc (sizeof (struct attrib));
-       attr->next = *attr_list;
-       attr->type = etype;
-       attr->name = gnu_arg0;
-
-       /* If we have an argument specified together with an attribute name,
-          make it a single TREE_VALUE entry in a list of arguments, as GCC
-          expects it.  */
-       if (gnu_arg1 != NULL_TREE)
-         attr->args = build_tree_list (NULL_TREE, gnu_arg1);
-       else
-         attr->args = NULL_TREE;
 
-       attr->error_point
-         = Present (Next (First (gnat_assoc)))
-           ? Expression (Next (First (gnat_assoc))) : gnat_temp;
-       *attr_list = attr;
+       /* Prepend to the list now.  Make a list of the argument we might
+          have, as GCC expects it.  */
+       prepend_one_attribute_to
+         (attr_list,
+          etype, gnu_arg0,
+          (gnu_arg1 != NULL_TREE)
+          ? build_tree_list (NULL_TREE, gnu_arg1) : NULL_TREE,
+          Present (Next (First (gnat_assoc)))
+          ? Expression (Next (First (gnat_assoc))) : gnat_temp);
       }
 }
 \f
@@ -4611,7 +5214,7 @@ maybe_variable (tree gnu_operand)
 
 static tree
 elaborate_expression (Node_Id gnat_expr, Entity_Id gnat_entity,
-                      tree gnu_name, bool definition, bool need_value,
+                     tree gnu_name, bool definition, bool need_value,
                      bool need_debug)
 {
   tree gnu_expr;
@@ -4621,7 +5224,7 @@ elaborate_expression (Node_Id gnat_expr, Entity_Id gnat_entity,
   if (present_gnu_tree (gnat_expr))
     return get_gnu_tree (gnat_expr);
 
-  /* If we don't need a value and this is static or a discriment, we
+  /* If we don't need a value and this is static or a discriminant, we
      don't need to do anything.  */
   else if (!need_value
           && (Is_OK_Static_Expression (gnat_expr)
@@ -4646,8 +5249,8 @@ elaborate_expression (Node_Id gnat_expr, Entity_Id gnat_entity,
 
 static tree
 elaborate_expression_1 (Node_Id gnat_expr, Entity_Id gnat_entity,
-                        tree gnu_expr, tree gnu_name, bool definition,
-                        bool need_debug)
+                       tree gnu_expr, tree gnu_name, bool definition,
+                       bool need_debug)
 {
   tree gnu_decl = NULL_TREE;
   /* Strip any conversions to see if the expression is a readonly variable.
@@ -4679,7 +5282,8 @@ elaborate_expression_1 (Node_Id gnat_expr, Entity_Id gnat_entity,
 
   expr_variable = (!CONSTANT_CLASS_P (gnu_expr)
                   && !(TREE_CODE (gnu_inner_expr) == VAR_DECL
-                       && TREE_READONLY (gnu_inner_expr))
+                       && (TREE_READONLY (gnu_inner_expr)
+                           || DECL_READONLY_ONCE_ELAB (gnu_inner_expr)))
                   && !CONTAINS_PLACEHOLDER_P (gnu_expr));
 
   /* If this is a static expression or contains a discriminant, we don't
@@ -4695,9 +5299,9 @@ elaborate_expression_1 (Node_Id gnat_expr, Entity_Id gnat_entity,
     gnu_decl
       = create_var_decl (create_concat_name (gnat_entity,
                                             IDENTIFIER_POINTER (gnu_name)),
-                        NULL_TREE, TREE_TYPE (gnu_expr), gnu_expr, true,
-                        Is_Public (gnat_entity), !definition, false, NULL,
-                        gnat_entity);
+                        NULL_TREE, TREE_TYPE (gnu_expr), gnu_expr,
+                        !need_debug, Is_Public (gnat_entity),
+                        !definition, false, NULL, gnat_entity);
 
   /* We only need to use this variable if we are in global context since GCC
      can do the right thing in the local case.  */
@@ -4709,107 +5313,194 @@ elaborate_expression_1 (Node_Id gnat_expr, Entity_Id gnat_entity,
     return maybe_variable (gnu_expr);
 }
 \f
-/* Create a record type that contains a field of TYPE with a starting bit
-   position so that it is aligned to ALIGN bits and is SIZE bytes long.  */
+/* Create a record type that contains a SIZE bytes long field of TYPE with a
+   starting bit position so that it is aligned to ALIGN bits, and leaving at
+   least ROOM bytes free before the field.  BASE_ALIGN is the alignment the
+   record is guaranteed to get.  */
 
 tree
-make_aligning_type (tree type, int align, tree size)
+make_aligning_type (tree type, unsigned int align, tree size,
+                   unsigned int base_align, int room)
 {
+  /* We will be crafting a record type with one field at a position set to be
+     the next multiple of ALIGN past record'address + room bytes.  We use a
+     record placeholder to express record'address.  */
+
   tree record_type = make_node (RECORD_TYPE);
-  tree place = build0 (PLACEHOLDER_EXPR, record_type);
-  tree size_addr_place = convert (sizetype,
-                                 build_unary_op (ADDR_EXPR, NULL_TREE,
-                                                 place));
+  tree record = build0 (PLACEHOLDER_EXPR, record_type);
+
+  tree record_addr_st
+    = convert (sizetype, build_unary_op (ADDR_EXPR, NULL_TREE, record));
+
+  /* The diagram below summarizes the shape of what we manipulate:
+
+                    <--------- pos ---------->
+                {  +------------+-------------+-----------------+
+      record  =>{  |############|     ...     | field (type)    |
+                {  +------------+-------------+-----------------+
+                  |<-- room -->|<- voffset ->|<---- size ----->|
+                  o            o
+                  |            |
+                  record_addr  vblock_addr
+
+     Every length is in sizetype bytes there, except "pos" which has to be
+     set as a bit position in the GCC tree for the record.  */
+
+  tree room_st = size_int (room);
+  tree vblock_addr_st = size_binop (PLUS_EXPR, record_addr_st, room_st);
+  tree voffset_st, pos, field;
+
   tree name = TYPE_NAME (type);
-  tree pos, field;
 
   if (TREE_CODE (name) == TYPE_DECL)
     name = DECL_NAME (name);
 
   TYPE_NAME (record_type) = concat_id_with_name (name, "_ALIGN");
 
-  /* The bit position is obtained by "and"ing the alignment minus 1
-     with the two's complement of the address and  multiplying
-     by the number of bits per unit.  Do all this in sizetype.  */
+  /* Compute VOFFSET and then POS.  The next byte position multiple of some
+     alignment after some address is obtained by "and"ing the alignment minus
+     1 with the two's complement of the address.   */
+
+  voffset_st = size_binop (BIT_AND_EXPR,
+                          size_diffop (size_zero_node, vblock_addr_st),
+                          ssize_int ((align / BITS_PER_UNIT) - 1));
+
+  /* POS = (ROOM + VOFFSET) * BIT_PER_UNIT, in bitsizetype.  */
+
   pos = size_binop (MULT_EXPR,
                    convert (bitsizetype,
-                            size_binop (BIT_AND_EXPR,
-                                        size_diffop (size_zero_node,
-                                                     size_addr_place),
-                                        ssize_int ((align / BITS_PER_UNIT)
-                                                   - 1))),
-                   bitsize_unit_node);
-
-  /* Create the field, with -1 as the 'addressable' indication to avoid the
+                            size_binop (PLUS_EXPR, room_st, voffset_st)),
+                    bitsize_unit_node);
+
+  /* Craft the GCC record representation.  We exceptionally do everything
+     manually here because 1) our generic circuitry is not quite ready to
+     handle the complex position/size expressions we are setting up, 2) we
+     have a strong simplifying factor at hand: we know the maximum possible
+     value of voffset, and 3) we have to set/reset at least the sizes in
+     accordance with this maximum value anyway, as we need them to convey
+     what should be "alloc"ated for this type.
+
+     Use -1 as the 'addressable' indication for the field to prevent the
      creation of a bitfield.  We don't need one, it would have damaging
      consequences on the alignment computation, and create_field_decl would
      make one without this special argument, for instance because of the
      complex position expression.  */
-  field = create_field_decl (get_identifier ("F"), type, record_type, 1, size,
-                            pos, -1);
 
-  finish_record_type (record_type, field, true, false);
-  TYPE_ALIGN (record_type) = BIGGEST_ALIGNMENT;
+  field = create_field_decl (get_identifier ("F"), type, record_type,
+                             1, size, pos, -1);
+  TYPE_FIELDS (record_type) = field;
+
+  TYPE_ALIGN (record_type) = base_align;
+  TYPE_USER_ALIGN (record_type) = 1;
+
   TYPE_SIZE (record_type)
     = size_binop (PLUS_EXPR,
-                 size_binop (MULT_EXPR, convert (bitsizetype, size),
-                             bitsize_unit_node),
-                 bitsize_int (align));
+                  size_binop (MULT_EXPR, convert (bitsizetype, size),
+                              bitsize_unit_node),
+                 bitsize_int (align + room * BITS_PER_UNIT));
   TYPE_SIZE_UNIT (record_type)
-    = size_binop (PLUS_EXPR, size, size_int (align / BITS_PER_UNIT));
+    = size_binop (PLUS_EXPR, size,
+                 size_int (room + align / BITS_PER_UNIT));
+
+  TYPE_MODE (record_type) = BLKmode;
+
   copy_alias_set (record_type, type);
   return record_type;
 }
 \f
-/* TYPE is a RECORD_TYPE, UNION_TYPE, or QUAL_UNION_TYPE, with BLKmode that's
-   being used as the field type of a packed record.  See if we can rewrite it
-   as a record that has a non-BLKmode type, which we can pack tighter.  If so,
-   return the new type.  If not, return the original type.  */
+/* Return the result of rounding T up to ALIGN.  */
+
+static inline unsigned HOST_WIDE_INT
+round_up_to_align (unsigned HOST_WIDE_INT t, unsigned int align)
+{
+  t += align - 1;
+  t /= align;
+  t *= align;
+  return t;
+}
+
+/* TYPE is a RECORD_TYPE, UNION_TYPE or QUAL_UNION_TYPE that is being used
+   as the field type of a packed record if IN_RECORD is true, or as the
+   component type of a packed array if IN_RECORD is false.  See if we can
+   rewrite it either as a type that has a non-BLKmode, which we can pack
+   tighter in the packed record case, or as a smaller type with BLKmode.
+   If so, return the new type.  If not, return the original type.  */
 
 static tree
-make_packable_type (tree type)
+make_packable_type (tree type, bool in_record)
 {
-  tree new_type = make_node (TREE_CODE (type));
-  tree field_list = NULL_TREE;
-  tree old_field;
+  unsigned HOST_WIDE_INT size = tree_low_cst (TYPE_SIZE (type), 1);
+  unsigned HOST_WIDE_INT new_size;
+  tree new_type, old_field, field_list = NULL_TREE;
 
-  /* Copy the name and flags from the old type to that of the new and set
-     the alignment to try for an integral type.  For QUAL_UNION_TYPE,
-     also copy the size.  */
+  /* No point in doing anything if the size is zero.  */
+  if (size == 0)
+    return type;
+
+  new_type = make_node (TREE_CODE (type));
+
+  /* Copy the name and flags from the old type to that of the new.  Note
+     that we rely on the pointer equality created here for TYPE_NAME at
+     the end of gnat_to_gnu.  */
   TYPE_NAME (new_type) = TYPE_NAME (type);
-  TYPE_JUSTIFIED_MODULAR_P (new_type)
-    = TYPE_JUSTIFIED_MODULAR_P (type);
+  TYPE_JUSTIFIED_MODULAR_P (new_type) = TYPE_JUSTIFIED_MODULAR_P (type);
   TYPE_CONTAINS_TEMPLATE_P (new_type) = TYPE_CONTAINS_TEMPLATE_P (type);
-
   if (TREE_CODE (type) == RECORD_TYPE)
     TYPE_IS_PADDING_P (new_type) = TYPE_IS_PADDING_P (type);
-  else if (TREE_CODE (type) == QUAL_UNION_TYPE)
+
+  /* If we are in a record and have a small size, set the alignment to
+     try for an integral mode.  Otherwise set it to try for a smaller
+     type with BLKmode.  */
+  if (in_record && size <= MAX_FIXED_MODE_SIZE)
     {
-      TYPE_SIZE (new_type) = TYPE_SIZE (type);
-      TYPE_SIZE_UNIT (new_type) = TYPE_SIZE_UNIT (type);
+      TYPE_ALIGN (new_type) = ceil_alignment (size);
+      new_size = round_up_to_align (size, TYPE_ALIGN (new_type));
+    }
+  else
+    {
+      unsigned HOST_WIDE_INT align;
+
+      /* Do not try to shrink the size if the RM size is not constant.  */
+      if (TYPE_CONTAINS_TEMPLATE_P (type)
+         || !host_integerp (TYPE_ADA_SIZE (type), 1))
+       return type;
+
+      /* Round the RM size up to a unit boundary to get the minimal size
+        for a BLKmode record.  Give up if it's already the size.  */
+      new_size = TREE_INT_CST_LOW (TYPE_ADA_SIZE (type));
+      new_size = round_up_to_align (new_size, BITS_PER_UNIT);
+      if (new_size == size)
+       return type;
+
+      align = new_size & -new_size;
+      TYPE_ALIGN (new_type) = MIN (TYPE_ALIGN (type), align);
     }
 
-  TYPE_ALIGN (new_type)
-    = ((HOST_WIDE_INT) 1
-       << (floor_log2 (tree_low_cst (TYPE_SIZE (type), 1) - 1) + 1));
+  TYPE_USER_ALIGN (new_type) = 1;
 
-  /* Now copy the fields, keeping the position and size.  */
+  /* Now copy the fields, keeping the position and size as we don't
+     want to propagate packedness downward.  But make an exception
+     for the last field in order to ditch the padding bits.  */
   for (old_field = TYPE_FIELDS (type); old_field;
        old_field = TREE_CHAIN (old_field))
     {
       tree new_field_type = TREE_TYPE (old_field);
-      tree new_field;
+      tree new_field, new_size;
 
       if (TYPE_MODE (new_field_type) == BLKmode
          && (TREE_CODE (new_field_type) == RECORD_TYPE
              || TREE_CODE (new_field_type) == UNION_TYPE
              || TREE_CODE (new_field_type) == QUAL_UNION_TYPE)
          && host_integerp (TYPE_SIZE (new_field_type), 1))
-       new_field_type = make_packable_type (new_field_type);
+       new_field_type = make_packable_type (new_field_type, true);
+
+      if (!TREE_CHAIN (old_field) && !TYPE_PACKED (type))
+       new_size = rm_size (new_field_type);
+      else
+       new_size = DECL_SIZE (old_field);
 
       new_field = create_field_decl (DECL_NAME (old_field), new_field_type,
-                                    new_type, TYPE_PACKED (type),
-                                    DECL_SIZE (old_field),
+                                    new_type, TYPE_PACKED (type), new_size,
                                     bit_position (old_field),
                                     !DECL_NONADDRESSABLE_P (old_field));
 
@@ -4825,9 +5516,40 @@ make_packable_type (tree type)
       field_list = new_field;
     }
 
-  finish_record_type (new_type, nreverse (field_list), true, true);
+  finish_record_type (new_type, nreverse (field_list), 2, true);
   copy_alias_set (new_type, type);
-  return TYPE_MODE (new_type) == BLKmode ? type : new_type;
+
+  /* If this is a padding record, we never want to make the size smaller
+     than what was specified.  For QUAL_UNION_TYPE, also copy the size.  */
+  if ((TREE_CODE (type) == RECORD_TYPE && TYPE_IS_PADDING_P (type))
+      || TREE_CODE (type) == QUAL_UNION_TYPE)
+    {
+      TYPE_SIZE (new_type) = TYPE_SIZE (type);
+      TYPE_SIZE_UNIT (new_type) = TYPE_SIZE_UNIT (type);
+    }
+  else
+    {
+      TYPE_SIZE (new_type) = bitsize_int (new_size);
+      TYPE_SIZE_UNIT (new_type)
+       = size_int ((new_size + BITS_PER_UNIT - 1) / BITS_PER_UNIT);
+    }
+
+  if (!TYPE_CONTAINS_TEMPLATE_P (type))
+    SET_TYPE_ADA_SIZE (new_type, TYPE_ADA_SIZE (type));
+
+  compute_record_mode (new_type);
+
+  /* Try harder to get a packable type if necessary, for example
+     in case the record itself contains a BLKmode field.  */
+  if (in_record && TYPE_MODE (new_type) == BLKmode)
+    TYPE_MODE (new_type)
+      = mode_for_size_tree (TYPE_SIZE (new_type), MODE_INT, 1);
+
+  /* If neither the mode nor the size has shrunk, return the old type.  */
+  if (TYPE_MODE (new_type) == BLKmode && new_size >= size)
+    return type;
+
+  return new_type;
 }
 \f
 /* Ensure that TYPE has SIZE and ALIGN.  Make and return a new padded type
@@ -4846,10 +5568,11 @@ make_packable_type (tree type)
 
 tree
 maybe_pad_type (tree type, tree size, unsigned int align,
-                Entity_Id gnat_entity, const char *name_trailer,
-                bool is_user_type, bool definition, bool same_rm_size)
+               Entity_Id gnat_entity, const char *name_trailer,
+               bool is_user_type, bool definition, bool same_rm_size)
 {
   tree orig_size = TYPE_SIZE (type);
+  unsigned int orig_align = align;
   tree record;
   tree field;
 
@@ -4858,7 +5581,6 @@ maybe_pad_type (tree type, tree size, unsigned int align,
      off the padding, since we will either be returning the inner type
      or repadding it.  If no size or alignment is specified, use that of
      the original padded type.  */
-
   if (TREE_CODE (type) == RECORD_TYPE && TYPE_IS_PADDING_P (type))
     {
       if ((!size
@@ -4883,7 +5605,6 @@ maybe_pad_type (tree type, tree size, unsigned int align,
      is not done here (and is only valid for bitfields anyway), show the size
      isn't changing.  Likewise, clear the alignment if it isn't being
      changed.  Then return if we aren't doing anything.  */
-
   if (size
       && (operand_equal_p (size, orig_size, 0)
          || (TREE_CODE (orig_size) == INTEGER_CST
@@ -4918,19 +5639,19 @@ maybe_pad_type (tree type, tree size, unsigned int align,
      BLKmode and a small constant size, try to make a form that has an
      integral mode.  That might allow this record to have an integral mode,
      which will be much more efficient.  There is no point in doing this if a
-     size is specified unless it is also smaller than the biggest alignment
+     size is specified unless it is also smaller than the maximum mode size
      and it is incorrect to do this if the size of the original type is not a
      multiple of the alignment.  */
   if (align != 0
       && TREE_CODE (type) == RECORD_TYPE
       && TYPE_MODE (type) == BLKmode
-      && host_integerp (orig_size, 1)
-      && compare_tree_int (orig_size, BIGGEST_ALIGNMENT) <= 0
+      && TREE_CODE (orig_size) == INTEGER_CST
+      && compare_tree_int (orig_size, MAX_FIXED_MODE_SIZE) <= 0
       && (!size
          || (TREE_CODE (size) == INTEGER_CST
-             && compare_tree_int (size, BIGGEST_ALIGNMENT) <= 0))
-      && tree_low_cst (orig_size, 1) % align == 0)
-    type = make_packable_type (type);
+             && compare_tree_int (size, MAX_FIXED_MODE_SIZE) <= 0))
+      && value_factor_p (orig_size, align))
+    type = make_packable_type (type, true);
 
   field  = create_field_decl (get_identifier ("F"), type, record, 0,
                              NULL_TREE, bitsize_zero_node, 1);
@@ -4943,10 +5664,14 @@ maybe_pad_type (tree type, tree size, unsigned int align,
        : TYPE_SIZE_UNIT (type));
 
   TYPE_ALIGN (record) = align;
+  if (orig_align)
+    TYPE_USER_ALIGN (record) = align;
+
   TYPE_IS_PADDING_P (record) = 1;
   TYPE_VOLATILE (record)
     = Present (gnat_entity) && Treat_As_Volatile (gnat_entity);
-  finish_record_type (record, field, true, false);
+  /* Do not finalize it until after the auxiliary record is built.  */
+  finish_record_type (record, field, 1, true);
 
   /* Keep the RM_Size of the padded record as that of the old record
      if requested.  */
@@ -4955,16 +5680,18 @@ maybe_pad_type (tree type, tree size, unsigned int align,
   /* Unless debugging information isn't being written for the input type,
      write a record that shows what we are a subtype of and also make a
      variable that indicates our size, if variable. */
-  if (TYPE_NAME (record) && AGGREGATE_TYPE_P (type)
+  if (TYPE_NAME (record)
+      && AGGREGATE_TYPE_P (type)
       && (TREE_CODE (TYPE_NAME (type)) != TYPE_DECL
          || !DECL_IGNORED_P (TYPE_NAME (type))))
     {
       tree marker = make_node (RECORD_TYPE);
-      tree name = (TREE_CODE (TYPE_NAME (record)) == TYPE_DECL
-                  ? DECL_NAME (TYPE_NAME (record))
-                  : TYPE_NAME (record));
+      tree name = TYPE_NAME (record);
       tree orig_name = TYPE_NAME (type);
 
+      if (TREE_CODE (name) == TYPE_DECL)
+       name = DECL_NAME (name);
+
       if (TREE_CODE (orig_name) == TYPE_DECL)
        orig_name = DECL_NAME (orig_name);
 
@@ -4973,20 +5700,26 @@ maybe_pad_type (tree type, tree size, unsigned int align,
                          create_field_decl (orig_name, integer_type_node,
                                             marker, 0, NULL_TREE, NULL_TREE,
                                             0),
-                         false, false);
+                         0, false);
 
       if (size && TREE_CODE (size) != INTEGER_CST && definition)
        create_var_decl (concat_id_with_name (name, "XVZ"), NULL_TREE,
-                        sizetype, TYPE_SIZE (record), false, false, false,
+                        bitsizetype, TYPE_SIZE (record), false, false, false,
                         false, NULL, gnat_entity);
     }
 
-  type = record;
+  rest_of_record_type_compilation (record);
 
+  /* If the size was widened explicitly, maybe give a warning.  Take the
+     original size as the maximum size of the input if there was an
+     unconstrained record involved and round it up to the specified alignment,
+     if one was specified.  */
   if (CONTAINS_PLACEHOLDER_P (orig_size))
     orig_size = max_size (orig_size, true);
 
-  /* If the size was widened explicitly, maybe give a warning.  */
+  if (align)
+    orig_size = round_up (orig_size, align);
+
   if (size && Present (gnat_entity)
       && !operand_equal_p (size, orig_size, 0)
       && !(TREE_CODE (size) == INTEGER_CST
@@ -4996,7 +5729,7 @@ maybe_pad_type (tree type, tree size, unsigned int align,
       Node_Id gnat_error_node = Empty;
 
       if (Is_Packed_Array_Type (gnat_entity))
-       gnat_entity = Associated_Node_For_Itype (gnat_entity);
+       gnat_entity = Original_Array_Type (gnat_entity);
 
       if ((Ekind (gnat_entity) == E_Component
           || Ekind (gnat_entity) == E_Discriminant)
@@ -5019,7 +5752,7 @@ maybe_pad_type (tree type, tree size, unsigned int align,
                            size_diffop (size, orig_size));
     }
 
-  return type;
+  return record;
 }
 \f
 /* Given a GNU tree and a GNAT list of choices, generate an expression to test
@@ -5052,7 +5785,7 @@ choices_to_gnu (tree operand, Node_Id choices)
 
          break;
 
-        case N_Subtype_Indication:
+       case N_Subtype_Indication:
          gnat_temp = Range_Expression (Constraint (choice));
          low = gnat_to_gnu (Low_Bound (gnat_temp));
          high = gnat_to_gnu (High_Bound (gnat_temp));
@@ -5063,15 +5796,15 @@ choices_to_gnu (tree operand, Node_Id choices)
                                                operand, low),
                               build_binary_op (LE_EXPR, integer_type_node,
                                                operand, high));
-          break;
+         break;
 
        case N_Identifier:
-        case N_Expanded_Name:
+       case N_Expanded_Name:
          /* This represents either a subtype range, an enumeration
             literal, or a constant  Ekind says which.  If an enumeration
-             literal or constant, fall through to the next case.  */
+            literal or constant, fall through to the next case.  */
          if (Ekind (Entity (choice)) != E_Enumeration_Literal
-              && Ekind (Entity (choice)) != E_Constant)
+             && Ekind (Entity (choice)) != E_Constant)
            {
              tree type = gnat_to_gnu_type (Entity (choice));
 
@@ -5109,87 +5842,136 @@ choices_to_gnu (tree operand, Node_Id choices)
   return result;
 }
 \f
+/* Adjust PACKED setting as passed to gnat_to_gnu_field for a field of
+   type FIELD_TYPE to be placed in RECORD_TYPE.  Return the result.  */
+
+static int
+adjust_packed (tree field_type, tree record_type, int packed)
+{
+  /* If the field contains an item of variable size, we cannot pack it
+     because we cannot create temporaries of non-fixed size in case
+     we need to take the address of the field.  See addressable_p and
+     the notes on the addressability issues for further details.  */
+  if (is_variable_size (field_type))
+    return 0;
+
+  /* If the alignment of the record is specified and the field type
+     is over-aligned, request Storage_Unit alignment for the field.  */
+  if (packed == -2)
+    {
+      if (TYPE_ALIGN (field_type) > TYPE_ALIGN (record_type))
+       return -1;
+      else
+       return 0;
+    }
+
+  return packed;
+}
+
 /* Return a GCC tree for a field corresponding to GNAT_FIELD to be
    placed in GNU_RECORD_TYPE.
 
-   PACKED is 1 if the enclosing record is packed and -1 if the enclosing
-   record has a Component_Alignment of Storage_Unit.
+   PACKED is 1 if the enclosing record is packed, -1 if the enclosing
+   record has Component_Alignment of Storage_Unit, -2 if the enclosing
+   record has a specified alignment.
 
    DEFINITION is true if this field is for a record being defined.  */
 
 static tree
 gnat_to_gnu_field (Entity_Id gnat_field, tree gnu_record_type, int packed,
-                   bool definition)
+                  bool definition)
 {
   tree gnu_field_id = get_entity_name (gnat_field);
   tree gnu_field_type = gnat_to_gnu_type (Etype (gnat_field));
-  tree gnu_pos = 0;
-  tree gnu_size = 0;
-  tree gnu_field;
+  tree gnu_field, gnu_size, gnu_pos;
   bool needs_strict_alignment
     = (Is_Aliased (gnat_field) || Strict_Alignment (Etype (gnat_field))
        || Treat_As_Volatile (gnat_field));
 
-  /* If this field requires strict alignment or contains an item of
-     variable sized, pretend it isn't packed.  */
-  if (needs_strict_alignment || is_variable_size (gnu_field_type))
+  /* If this field requires strict alignment, we cannot pack it because
+     it would very likely be under-aligned in the record.  */
+  if (needs_strict_alignment)
     packed = 0;
+  else
+    packed = adjust_packed (gnu_field_type, gnu_record_type, packed);
 
-  /* For packed records, this is one of the few occasions on which we use
-     the official RM size for discrete or fixed-point components, instead
-     of the normal GNAT size stored in Esize. See description in Einfo:
-     "Handling of Type'Size Values" for further details.  */
-
-  if (packed == 1)
-    gnu_size = validate_size (RM_Size (Etype (gnat_field)), gnu_field_type,
-                             gnat_field, FIELD_DECL, false, true);
-
+  /* If a size is specified, use it.  Otherwise, if the record type is packed,
+     use the official RM size.  See "Handling of Type'Size Values" in Einfo
+     for further details.  */
   if (Known_Static_Esize (gnat_field))
     gnu_size = validate_size (Esize (gnat_field), gnu_field_type,
                              gnat_field, FIELD_DECL, false, true);
+  else if (packed == 1)
+    gnu_size = validate_size (RM_Size (Etype (gnat_field)), gnu_field_type,
+                             gnat_field, FIELD_DECL, false, true);
+  else
+    gnu_size = NULL_TREE;
+
+  /* If we have a specified size that's smaller than that of the field type,
+     or a position is specified, and the field type is also a record that's
+     BLKmode, see if we can get either an integral mode form of the type or
+     a smaller BLKmode form.  If we can, show a size was specified for the
+     field if there wasn't one already, so we know to make this a bitfield
+     and avoid making things wider.
+
+     Doing this is first useful if the record is packed because we may then
+     place the field at a non-byte-aligned position and so achieve tighter
+     packing.
+
+     This is in addition *required* if the field shares a byte with another
+     field and the front-end lets the back-end handle the references, because
+     GCC does not handle BLKmode bitfields properly.
+
+     We avoid the transformation if it is not required or potentially useful,
+     as it might entail an increase of the field's alignment and have ripple
+     effects on the outer record type.  A typical case is a field known to be
+     byte aligned and not to share a byte with another field.
+
+     Besides, we don't even look the possibility of a transformation in cases
+     known to be in error already, for instance when an invalid size results
+     from a component clause.  */
 
-  /* If we are packing this record, have a specified size that's smaller than
-     that of the field type, or a position is specified, and the field type is
-     also a record that's BLKmode and with a small constant size, see if we
-     can get a better form of the type that allows more packing.  If we can,
-     show a size was specified for it if there wasn't one so we know to make
-     this a bitfield and avoid making things wider.  */
   if (TREE_CODE (gnu_field_type) == RECORD_TYPE
       && TYPE_MODE (gnu_field_type) == BLKmode
       && host_integerp (TYPE_SIZE (gnu_field_type), 1)
-      && compare_tree_int (TYPE_SIZE (gnu_field_type), BIGGEST_ALIGNMENT) <= 0
       && (packed == 1
          || (gnu_size
-             && tree_int_cst_lt (gnu_size, TYPE_SIZE (gnu_field_type)))
-         || Present (Component_Clause (gnat_field))))
+             && (tree_int_cst_lt (gnu_size, TYPE_SIZE (gnu_field_type))
+                 || Present (Component_Clause (gnat_field))))))
     {
       /* See what the alternate type and size would be.  */
-      tree gnu_packable_type = make_packable_type (gnu_field_type);
+      tree gnu_packable_type = make_packable_type (gnu_field_type, true);
+
+      bool has_byte_aligned_clause
+       = Present (Component_Clause (gnat_field))
+         && (UI_To_Int (Component_Bit_Offset (gnat_field))
+             % BITS_PER_UNIT == 0);
 
       /* Compute whether we should avoid the substitution.  */
-      int reject =
-        /* There is no point subtituting if there is no change.  */
-        (gnu_packable_type == gnu_field_type
-         ||
-         /* The size of an aliased field must be an exact multiple of the
-            type's alignment, which the substitution might increase.  Reject
-            substitutions that would so invalidate a component clause when the
-            specified position is byte aligned, as the change would have no
-            real benefit from the packing standpoint anyway.  */
-         (Is_Aliased (gnat_field)
-          && Present (Component_Clause (gnat_field))
-          && UI_To_Int (Component_Bit_Offset (gnat_field)) % BITS_PER_UNIT == 0
-          && tree_low_cst (gnu_size, 1) % TYPE_ALIGN (gnu_packable_type) != 0)
-         );
+      bool reject
+       /* There is no point substituting if there is no change...  */
+       = (gnu_packable_type == gnu_field_type)
+        /* ... nor when the field is known to be byte aligned and not to
+           share a byte with another field.  */
+         || (has_byte_aligned_clause
+             && value_factor_p (gnu_size, BITS_PER_UNIT))
+        /* The size of an aliased field must be an exact multiple of the
+           type's alignment, which the substitution might increase.  Reject
+           substitutions that would so invalidate a component clause when the
+           specified position is byte aligned, as the change would have no
+           real benefit from the packing standpoint anyway.  */
+         || (Is_Aliased (gnat_field)
+             && has_byte_aligned_clause
+             && !value_factor_p (gnu_size, TYPE_ALIGN (gnu_packable_type)));
 
       /* Substitute unless told otherwise.  */
       if (!reject)
-        {
-          gnu_field_type = gnu_packable_type;
+       {
+         gnu_field_type = gnu_packable_type;
 
-          if (gnu_size == 0)
-            gnu_size = rm_size (gnu_field_type);
-        }
+         if (!gnu_size)
+           gnu_size = rm_size (gnu_field_type);
+       }
     }
 
   /* If we are packing the record and the field is BLKmode, round the
@@ -5226,38 +6008,32 @@ gnat_to_gnu_field (Entity_Id gnat_field, tree gnu_record_type, int packed,
         consistent with the alignment.  */
       if (needs_strict_alignment)
        {
-         tree gnu_rounded_size = round_up (rm_size (gnu_field_type),
-                                           TYPE_ALIGN (gnu_field_type));
-
          TYPE_ALIGN (gnu_record_type)
            = MAX (TYPE_ALIGN (gnu_record_type), TYPE_ALIGN (gnu_field_type));
 
-         /* If Atomic, the size must match exactly that of the field.  */
-         if ((Is_Atomic (gnat_field) || Is_Atomic (Etype (gnat_field)))
+         if (gnu_size
              && !operand_equal_p (gnu_size, TYPE_SIZE (gnu_field_type), 0))
            {
-             post_error_ne_tree
-               ("atomic field& must be natural size of type{ (^)}",
-                Last_Bit (Component_Clause (gnat_field)), gnat_field,
-                TYPE_SIZE (gnu_field_type));
+             if (Is_Atomic (gnat_field) || Is_Atomic (Etype (gnat_field)))
+               post_error_ne_tree
+                 ("atomic field& must be natural size of type{ (^)}",
+                  Last_Bit (Component_Clause (gnat_field)), gnat_field,
+                  TYPE_SIZE (gnu_field_type));
+
+             else if (Is_Aliased (gnat_field))
+               post_error_ne_tree
+                 ("size of aliased field& must be ^ bits",
+                  Last_Bit (Component_Clause (gnat_field)), gnat_field,
+                  TYPE_SIZE (gnu_field_type));
 
-             gnu_size = NULL_TREE;
-           }
+             else if (Strict_Alignment (Etype (gnat_field)))
+               post_error_ne_tree
+                 ("size of & with aliased or tagged components not ^ bits",
+                  Last_Bit (Component_Clause (gnat_field)), gnat_field,
+                  TYPE_SIZE (gnu_field_type));
 
-         /* If Aliased, the size must match exactly the rounded size.  We
-            used to be more accommodating here and accept greater sizes, but
-            fully supporting this case on big-endian platforms would require
-            switching to a more involved layout for the field.  */
-         else if (Is_Aliased (gnat_field)
-                  && gnu_size
-                  && ! operand_equal_p (gnu_size, gnu_rounded_size, 0))
-           {
-             post_error_ne_tree
-               ("size of aliased field& must be ^ bits",
-                Last_Bit (Component_Clause (gnat_field)), gnat_field,
-                gnu_rounded_size);
              gnu_size = NULL_TREE;
-           }
+           }
 
          if (!integer_zerop (size_binop
                              (TRUNC_MOD_EXPR, gnu_pos,
@@ -5280,6 +6056,7 @@ gnat_to_gnu_field (Entity_Id gnat_field, tree gnu_record_type, int packed,
   ("position of & with aliased or tagged components not multiple of ^ bits",
                   First_Bit (Component_Clause (gnat_field)), gnat_field,
                   TYPE_ALIGN (gnu_field_type));
+
              else
                gcc_unreachable ();
 
@@ -5300,6 +6077,9 @@ gnat_to_gnu_field (Entity_Id gnat_field, tree gnu_record_type, int packed,
       gnu_size = TYPE_SIZE (gnu_field_type);
     }
 
+  else
+    gnu_pos = NULL_TREE;
+
   /* We need to make the size the maximum for the type if it is
      self-referential and an unconstrained type.  In that case, we can't
      pack the field since we can't make a copy to align it.  */
@@ -5312,11 +6092,8 @@ gnat_to_gnu_field (Entity_Id gnat_field, tree gnu_record_type, int packed,
       packed = 0;
     }
 
-  /* If no size is specified (or if there was an error), don't specify a
-     position.  */
-  if (!gnu_size)
-    gnu_pos = NULL_TREE;
-  else
+  /* If a size is specified, adjust the field's type to it.  */
+  if (gnu_size)
     {
       /* If the field's type is justified modular, we would need to remove
         the wrapper to (better) meet the layout requirements.  However we
@@ -5337,6 +6114,10 @@ gnat_to_gnu_field (Entity_Id gnat_field, tree gnu_record_type, int packed,
                                       "PAD", false, definition, true);
     }
 
+  /* Otherwise (or if there was an error), don't specify a position.  */
+  else
+    gnu_pos = NULL_TREE;
+
   gcc_assert (TREE_CODE (gnu_field_type) != RECORD_TYPE
              || !TYPE_CONTAINS_TEMPLATE_P (gnu_field_type));
 
@@ -5362,18 +6143,17 @@ is_variable_size (tree type)
 {
   tree field;
 
-  /* We need not be concerned about this at all if we don't have
-     strict alignment.  */
-  if (!STRICT_ALIGNMENT)
-    return false;
-  else if (!TREE_CONSTANT (TYPE_SIZE (type)))
+  if (!TREE_CONSTANT (TYPE_SIZE (type)))
     return true;
-  else if (TREE_CODE (type) == RECORD_TYPE && TYPE_IS_PADDING_P (type)
-          && !TREE_CONSTANT (DECL_SIZE (TYPE_FIELDS (type))))
+
+  if (TREE_CODE (type) == RECORD_TYPE
+      && TYPE_IS_PADDING_P (type)
+      && !TREE_CONSTANT (DECL_SIZE (TYPE_FIELDS (type))))
     return true;
-  else if (TREE_CODE (type) != RECORD_TYPE
-          && TREE_CODE (type) != UNION_TYPE
-          && TREE_CODE (type) != QUAL_UNION_TYPE)
+
+  if (TREE_CODE (type) != RECORD_TYPE
+      && TREE_CODE (type) != UNION_TYPE
+      && TREE_CODE (type) != QUAL_UNION_TYPE)
     return false;
 
   for (field = TYPE_FIELDS (type); field; field = TREE_CHAIN (field))
@@ -5383,6 +6163,19 @@ is_variable_size (tree type)
   return false;
 }
 \f
+/* qsort comparer for the bit positions of two record components.  */
+
+static int
+compare_field_bitpos (const PTR rt1, const PTR rt2)
+{
+  const_tree const field1 = * (const_tree const *) rt1;
+  const_tree const field2 = * (const_tree const *) rt2;
+  const int ret
+    = tree_int_cst_compare (bit_position (field1), bit_position (field2));
+
+  return ret ? ret : (int) (DECL_UID (field1) - DECL_UID (field2));
+}
+
 /* Return a GCC tree for a record type given a GNAT Component_List and a chain
    of GCC trees for fields that are in the record and have already been
    processed.  When called from gnat_to_gnu_entity during the processing of a
@@ -5390,8 +6183,9 @@ is_variable_size (tree type)
    the chain.  The other calls to this function are recursive calls from
    itself for the Component_List of a variant and the chain is empty.
 
-   PACKED is 1 if this is for a record with "pragma pack" and -1 is this is
-   for a record type with "pragma component_alignment (storage_unit)".
+   PACKED is 1 if this is for a packed record, -1 if this is for a record
+   with Component_Alignment of Storage_Unit, -2 if this is for a record
+   with a specified alignment.
 
    DEFINITION is true if we are defining this record.
 
@@ -5407,22 +6201,24 @@ is_variable_size (tree type)
    ALL_REP, if true, means a rep clause was found for all the fields.  This
    simplifies the logic since we know we're not in the mixed case.
 
-   DEFER_DEBUG, if true, means that the debugging routines should not be
-   called when finishing constructing the record type.
+   DO_NOT_FINALIZE, if true, means that the record type is expected to be
+   modified afterwards so it will not be sent to the back-end for finalization.
+
+   UNCHECKED_UNION, if true, means that we are building a type for a record
+   with a Pragma Unchecked_Union.
 
    The processing of the component list fills in the chain with all of the
    fields of the record and then the record type is finished.  */
 
 static void
 components_to_record (tree gnu_record_type, Node_Id component_list,
-                      tree gnu_field_list, int packed, bool definition,
-                      tree *p_gnu_rep_list, bool cancel_alignment,
-                     bool all_rep, bool defer_debug)
+                     tree gnu_field_list, int packed, bool definition,
+                     tree *p_gnu_rep_list, bool cancel_alignment,
+                     bool all_rep, bool do_not_finalize, bool unchecked_union)
 {
   Node_Id component_decl;
   Entity_Id gnat_field;
   Node_Id variant_part;
-  Node_Id variant;
   tree gnu_our_rep_list = NULL_TREE;
   tree gnu_field, gnu_last;
   bool layout_with_rep = false;
@@ -5430,7 +6226,6 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
 
   /* For each variable within each component declaration create a GCC field
      and add it to the list, skipping any pragmas in the list.  */
-
   if (Present (Component_Items (component_list)))
     for (component_decl = First_Non_Pragma (Component_Items (component_list));
         Present (component_decl);
@@ -5449,66 +6244,65 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
               instead of after them as is the case for all other fields.
               Ignore field of void type if only annotating.  */
            if (Chars (gnat_field) == Name_uTag)
-              gnu_field_list = chainon (gnu_field_list, gnu_field);
+             gnu_field_list = chainon (gnu_field_list, gnu_field);
            else
-              {
+             {
                TREE_CHAIN (gnu_field) = gnu_field_list;
                gnu_field_list = gnu_field;
              }
          }
 
          save_gnu_tree (gnat_field, gnu_field, false);
-        }
+      }
 
   /* At the end of the component list there may be a variant part.  */
   variant_part = Variant_Part (component_list);
 
-  /* If this is an unchecked union, each variant must have exactly one
-     component, each of which becomes one component of this union.  */
-  if (TREE_CODE (gnu_record_type) == UNION_TYPE
-      && TYPE_UNCHECKED_UNION_P (gnu_record_type)
-      && Present (variant_part))
-    for (variant = First_Non_Pragma (Variants (variant_part));
-        Present (variant);
-        variant = Next_Non_Pragma (variant))
-      {
-       component_decl
-         = First_Non_Pragma (Component_Items (Component_List (variant)));
-       gnat_field = Defining_Entity (component_decl);
-       gnu_field = gnat_to_gnu_field (gnat_field, gnu_record_type, packed,
-                                      definition);
-       TREE_CHAIN (gnu_field) = gnu_field_list;
-       gnu_field_list = gnu_field;
-       save_gnu_tree (gnat_field, gnu_field, false);
-      }
-
   /* We create a QUAL_UNION_TYPE for the variant part since the variants are
      mutually exclusive and should go in the same memory.  To do this we need
      to treat each variant as a record whose elements are created from the
      component list for the variant.  So here we create the records from the
-     lists for the variants and put them all into the QUAL_UNION_TYPE.  */
-  else if (Present (variant_part))
+     lists for the variants and put them all into the QUAL_UNION_TYPE.
+     If this is an Unchecked_Union, we make a UNION_TYPE instead or
+     use GNU_RECORD_TYPE if there are no fields so far.  */
+  if (Present (variant_part))
     {
       tree gnu_discriminant = gnat_to_gnu (Name (variant_part));
       Node_Id variant;
-      tree gnu_union_type = make_node (QUAL_UNION_TYPE);
-      tree gnu_union_field;
-      tree gnu_variant_list = NULL_TREE;
       tree gnu_name = TYPE_NAME (gnu_record_type);
       tree gnu_var_name
-       = concat_id_with_name
-         (get_identifier (Get_Name_String (Chars (Name (variant_part)))),
-          "XVN");
+       = concat_id_with_name (get_identifier (Get_Name_String
+                                              (Chars (Name (variant_part)))),
+                              "XVN");
+      tree gnu_union_type;
+      tree gnu_union_name;
+      tree gnu_union_field;
+      tree gnu_variant_list = NULL_TREE;
 
       if (TREE_CODE (gnu_name) == TYPE_DECL)
        gnu_name = DECL_NAME (gnu_name);
 
-      TYPE_NAME (gnu_union_type)
-       = concat_id_with_name (gnu_name, IDENTIFIER_POINTER (gnu_var_name));
-      TYPE_PACKED (gnu_union_type) = TYPE_PACKED (gnu_record_type);
+      gnu_union_name = concat_id_with_name (gnu_name,
+                                           IDENTIFIER_POINTER (gnu_var_name));
+
+      /* Reuse an enclosing union if all fields are in the variant part
+        and there is no representation clause on the record, to match
+        the layout of C unions.  There is an associated check below.  */
+      if (!gnu_field_list
+         && TREE_CODE (gnu_record_type) == UNION_TYPE
+         && !TYPE_PACKED (gnu_record_type))
+       gnu_union_type = gnu_record_type;
+      else
+       {
+         gnu_union_type
+           = make_node (unchecked_union ? UNION_TYPE : QUAL_UNION_TYPE);
+
+         TYPE_NAME (gnu_union_type) = gnu_union_name;
+         TYPE_PACKED (gnu_union_type) = TYPE_PACKED (gnu_record_type);
+       }
 
       for (variant = First_Non_Pragma (Variants (variant_part));
-           Present (variant);
+          Present (variant);
           variant = Next_Non_Pragma (variant))
        {
          tree gnu_variant_type = make_node (RECORD_TYPE);
@@ -5518,7 +6312,7 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
          Get_Variant_Encoding (variant);
          gnu_inner_name = get_identifier (Name_Buffer);
          TYPE_NAME (gnu_variant_type)
-           = concat_id_with_name (TYPE_NAME (gnu_union_type),
+           = concat_id_with_name (gnu_union_name,
                                   IDENTIFIER_POINTER (gnu_inner_name));
 
          /* Set the alignment of the inner type in case we need to make
@@ -5537,35 +6331,56 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
                = TYPE_SIZE_UNIT (gnu_record_type);
            }
 
+         /* Create the record type for the variant.  Note that we defer
+            finalizing it until after we are sure to actually use it.  */
          components_to_record (gnu_variant_type, Component_List (variant),
                                NULL_TREE, packed, definition,
                                &gnu_our_rep_list, !all_rep_and_size, all_rep,
-                               false);
+                               true, unchecked_union);
 
          gnu_qual = choices_to_gnu (gnu_discriminant,
                                     Discrete_Choices (variant));
 
          Set_Present_Expr (variant, annotate_value (gnu_qual));
-         gnu_field = create_field_decl (gnu_inner_name, gnu_variant_type,
-                                        gnu_union_type, 0,
-                                        (all_rep_and_size
-                                         ? TYPE_SIZE (gnu_record_type) : 0),
-                                        (all_rep_and_size
-                                         ? bitsize_zero_node : 0),
-                                        0);
 
-         DECL_INTERNAL_P (gnu_field) = 1;
-         DECL_QUALIFIER (gnu_field) = gnu_qual;
+         /* If this is an Unchecked_Union and we have exactly one field,
+            use this field directly to match the layout of C unions.  */
+         if (unchecked_union
+             && TYPE_FIELDS (gnu_variant_type)
+             && !TREE_CHAIN (TYPE_FIELDS (gnu_variant_type)))
+           gnu_field = TYPE_FIELDS (gnu_variant_type);
+         else
+           {
+             /* Deal with packedness like in gnat_to_gnu_field.  */
+             int field_packed
+               = adjust_packed (gnu_variant_type, gnu_record_type, packed);
+
+             /* Finalize the record type now.  We used to throw away
+                empty records but we no longer do that because we need
+                them to generate complete debug info for the variant;
+                otherwise, the union type definition will be lacking
+                the fields associated with these empty variants.  */
+             rest_of_record_type_compilation (gnu_variant_type);
+
+             gnu_field = create_field_decl (gnu_inner_name, gnu_variant_type,
+                                            gnu_union_type, field_packed,
+                                            (all_rep_and_size
+                                             ? TYPE_SIZE (gnu_variant_type)
+                                             : 0),
+                                            (all_rep_and_size
+                                             ? bitsize_zero_node : 0),
+                                            0);
+
+             DECL_INTERNAL_P (gnu_field) = 1;
+
+             if (!unchecked_union)
+               DECL_QUALIFIER (gnu_field) = gnu_qual;
+           }
+
          TREE_CHAIN (gnu_field) = gnu_variant_list;
          gnu_variant_list = gnu_field;
        }
 
-      /* We use to delete the empty variants from the end. However,
-         we no longer do that because we need them to generate complete
-         debugging information for the variant record.  Otherwise,
-         the union type definition will be missing the fields associated
-         to these empty variants.  */
-
       /* Only make the QUAL_UNION_TYPE if there are any non-empty variants.  */
       if (gnu_variant_list)
        {
@@ -5577,7 +6392,18 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
            }
 
          finish_record_type (gnu_union_type, nreverse (gnu_variant_list),
-                             all_rep_and_size, false);
+                             all_rep_and_size ? 1 : 0, false);
+
+         /* If GNU_UNION_TYPE is our record type, it means we must have an
+            Unchecked_Union with no fields.  Verify that and, if so, just
+            return.  */
+         if (gnu_union_type == gnu_record_type)
+           {
+             gcc_assert (unchecked_union
+                         && !gnu_field_list
+                         && !gnu_our_rep_list);
+             return;
+           }
 
          gnu_union_field
            = create_field_decl (gnu_var_name, gnu_union_type, gnu_record_type,
@@ -5598,7 +6424,6 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
 
      ??? Note: if we then reorder them, debugging information will be wrong,
      but there's nothing that can be done about this at the moment.  */
-
   for (gnu_field = gnu_field_list, gnu_last = NULL_TREE; gnu_field; )
     {
       if (DECL_FIELD_OFFSET (gnu_field))
@@ -5623,27 +6448,22 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
 
   /* If we have any items in our rep'ed field list, it is not the case that all
      the fields in the record have rep clauses, and P_REP_LIST is nonzero,
-     set it and ignore the items.  Otherwise, sort the fields by bit position
-     and put them into their own record if we have any fields without
-     rep clauses. */
+     set it and ignore the items.  */
   if (gnu_our_rep_list && p_gnu_rep_list && !all_rep)
     *p_gnu_rep_list = chainon (*p_gnu_rep_list, gnu_our_rep_list);
   else if (gnu_our_rep_list)
     {
+      /* Otherwise, sort the fields by bit position and put them into their
+        own record if we have any fields without rep clauses. */
       tree gnu_rep_type
        = (gnu_field_list ? make_node (RECORD_TYPE) : gnu_record_type);
       int len = list_length (gnu_our_rep_list);
       tree *gnu_arr = (tree *) alloca (sizeof (tree) * len);
       int i;
 
-      /* Set/abuse DECL_FCONTEXT to increasing integers so we have a
-        stable sort.  */
       for (i = 0, gnu_field = gnu_our_rep_list; gnu_field;
           gnu_field = TREE_CHAIN (gnu_field), i++)
-       {
-         gnu_arr[i] = gnu_field;
-         DECL_FCONTEXT (gnu_field) = size_int (i);
-       }
+       gnu_arr[i] = gnu_field;
 
       qsort (gnu_arr, len, sizeof (tree), compare_field_bitpos);
 
@@ -5655,12 +6475,11 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
          TREE_CHAIN (gnu_arr[i]) = gnu_our_rep_list;
          gnu_our_rep_list = gnu_arr[i];
          DECL_CONTEXT (gnu_arr[i]) = gnu_rep_type;
-         DECL_FCONTEXT (gnu_arr[i]) = NULL_TREE;
        }
 
       if (gnu_field_list)
        {
-         finish_record_type (gnu_rep_type, gnu_our_rep_list, true, false);
+         finish_record_type (gnu_rep_type, gnu_our_rep_list, 1, false);
          gnu_field = create_field_decl (get_identifier ("REP"), gnu_rep_type,
                                         gnu_record_type, 0, 0, 0, 1);
          DECL_INTERNAL_P (gnu_field) = 1;
@@ -5677,26 +6496,7 @@ components_to_record (tree gnu_record_type, Node_Id component_list,
     TYPE_ALIGN (gnu_record_type) = 0;
 
   finish_record_type (gnu_record_type, nreverse (gnu_field_list),
-                     layout_with_rep, defer_debug);
-}
-\f
-/* Called via qsort from the above.  Returns -1, 1, depending on the
-   bit positions and ordinals of the two fields.  */
-
-static int
-compare_field_bitpos (const PTR rt1, const PTR rt2)
-{
-  tree *t1 = (tree *) rt1;
-  tree *t2 = (tree *) rt2;
-
-  if (tree_int_cst_equal (bit_position (*t1), bit_position (*t2)))
-    return
-      (tree_int_cst_lt (DECL_FCONTEXT (*t1), DECL_FCONTEXT (*t2))
-       ? -1 : 1);
-  else if (tree_int_cst_lt (bit_position (*t1), bit_position (*t2)))
-    return -1;
-  else
-    return 1;
+                     layout_with_rep ? 1 : 0, do_not_finalize);
 }
 \f
 /* Given GNU_SIZE, a GCC tree representing a size, return a Uint to be
@@ -5711,10 +6511,22 @@ annotate_value (tree gnu_size)
   Node_Ref_Or_Val ops[3], ret;
   int i;
   int size;
+  struct tree_int_map **h = NULL;
 
   /* See if we've already saved the value for this node.  */
-  if (EXPR_P (gnu_size) && TREE_COMPLEXITY (gnu_size))
-    return (Node_Ref_Or_Val) TREE_COMPLEXITY (gnu_size);
+  if (EXPR_P (gnu_size))
+    {
+      struct tree_int_map in;
+      if (!annotate_value_cache)
+        annotate_value_cache = htab_create_ggc (512, tree_int_map_hash,
+                                               tree_int_map_eq, 0);
+      in.base.from = gnu_size;
+      h = (struct tree_int_map **)
+           htab_find_slot (annotate_value_cache, &in, INSERT);
+
+      if (*h)
+       return (Node_Ref_Or_Val) (*h)->to;
+    }
 
   /* If we do not return inside this switch, TCODE will be set to the
      code to use for a Create_Node operand and LEN (set above) will be
@@ -5739,7 +6551,7 @@ annotate_value (tree gnu_size)
          bool adjust = false;
          tree temp;
 
-         if (TREE_CONSTANT_OVERFLOW (negative_size))
+         if (TREE_OVERFLOW (negative_size))
            {
              negative_size
                = size_binop (MINUS_EXPR, bitsize_zero_node,
@@ -5804,6 +6616,7 @@ annotate_value (tree gnu_size)
     case TRUTH_OR_EXPR:                tcode = Truth_Or_Expr; break;
     case TRUTH_XOR_EXPR:       tcode = Truth_Xor_Expr; break;
     case TRUTH_NOT_EXPR:       tcode = Truth_Not_Expr; break;
+    case BIT_AND_EXPR:         tcode = Bit_And_Expr; break;
     case LT_EXPR:              tcode = Lt_Expr; break;
     case LE_EXPR:              tcode = Le_Expr; break;
     case GT_EXPR:              tcode = Gt_Expr; break;
@@ -5828,7 +6641,15 @@ annotate_value (tree gnu_size)
     }
 
   ret = Create_Node (tcode, ops[0], ops[1], ops[2]);
-  TREE_COMPLEXITY (gnu_size) = ret;
+
+  /* Save the result in the cache.  */
+  if (h)
+    {
+      *h = ggc_alloc (sizeof (struct tree_int_map));
+      (*h)->base.from = gnu_size;
+      (*h)->to = ret;
+    }
+
   return ret;
 }
 
@@ -5861,7 +6682,7 @@ annotate_rep (Entity_Id gnat_entity, tree gnu_type)
        gnu_entry = purpose_member (gnat_to_gnu_field_decl (gnat_field),
                                    gnu_list);
 
-        if (gnu_entry)
+       if (gnu_entry)
          {
            if (type_annotate_only && Is_Tagged_Type (gnat_entity))
              {
@@ -5869,14 +6690,14 @@ annotate_rep (Entity_Id gnat_entity, tree gnu_type)
                   generated, so we add the appropriate offset to each
                   component.  For a component appearing in the current
                   extension, the offset is the size of the parent.  */
-            if (Is_Derived_Type (gnat_entity)
+           if (Is_Derived_Type (gnat_entity)
                && Original_Record_Component (gnat_field) == gnat_field)
              parent_offset
                = UI_To_gnu (Esize (Etype (Base_Type (gnat_entity))),
                             bitsizetype);
            else
              parent_offset = bitsize_int (POINTER_SIZE);
-          }
+             }
 
          Set_Component_Bit_Offset
            (gnat_field,
@@ -5890,8 +6711,7 @@ annotate_rep (Entity_Id gnat_entity, tree gnu_type)
            Set_Esize (gnat_field,
                       annotate_value (DECL_SIZE (TREE_PURPOSE (gnu_entry))));
          }
-       else if (type_annotate_only
-                && Is_Tagged_Type (gnat_entity)
+       else if (Is_Tagged_Type (gnat_entity)
                 && Is_Derived_Type (gnat_entity))
          {
            /* If there is no gnu_entry, this is an inherited component whose
@@ -5915,7 +6735,7 @@ annotate_rep (Entity_Id gnat_entity, tree gnu_type)
 
 static tree
 compute_field_positions (tree gnu_type, tree gnu_list, tree gnu_pos,
-                         tree gnu_bitpos, unsigned int offset_align)
+                        tree gnu_bitpos, unsigned int offset_align)
 {
   tree gnu_field;
   tree gnu_result = gnu_list;
@@ -5960,12 +6780,18 @@ compute_field_positions (tree gnu_type, tree gnu_list, tree gnu_pos,
 
 static tree
 validate_size (Uint uint_size, tree gnu_type, Entity_Id gnat_object,
-               enum tree_code kind, bool component_p, bool zero_ok)
+              enum tree_code kind, bool component_p, bool zero_ok)
 {
   Node_Id gnat_error_node;
-  tree type_size
-    = kind == VAR_DECL ? TYPE_SIZE (gnu_type) : rm_size (gnu_type);
-  tree size;
+  tree type_size, size;
+
+  if (kind == VAR_DECL
+      /* If a type needs strict alignment, a component of this type in
+        a packed record cannot be packed and thus uses the type size.  */
+      || (kind == TYPE_DECL && Strict_Alignment (gnat_object)))
+    type_size = TYPE_SIZE (gnu_type);
+  else
+    type_size = rm_size (gnu_type);
 
   /* Find the node to use for errors.  */
   if ((Ekind (gnat_object) == E_Component
@@ -6228,32 +7054,28 @@ make_type_from_size (tree type, tree size_tree, bool biased_p)
 static unsigned int
 validate_alignment (Uint alignment, Entity_Id gnat_entity, unsigned int align)
 {
-  Node_Id gnat_error_node = gnat_entity;
+  unsigned int max_allowed_alignment = get_target_maximum_allowed_alignment ();
   unsigned int new_align;
-
-#ifndef MAX_OFILE_ALIGNMENT
-#define MAX_OFILE_ALIGNMENT BIGGEST_ALIGNMENT
-#endif
-
-  if (Present (Alignment_Clause (gnat_entity)))
-    gnat_error_node = Expression (Alignment_Clause (gnat_entity));
+  Node_Id gnat_error_node;
 
   /* Don't worry about checking alignment if alignment was not specified
      by the source program and we already posted an error for this entity.  */
-
   if (Error_Posted (gnat_entity) && !Has_Alignment_Clause (gnat_entity))
     return align;
 
-  /* Within GCC, an alignment is an integer, so we must make sure a
-     value is specified that fits in that range.  Also, alignments of
-     more than MAX_OFILE_ALIGNMENT can't be supported.  */
+  /* Post the error on the alignment clause if any.  */
+  if (Present (Alignment_Clause (gnat_entity)))
+    gnat_error_node = Expression (Alignment_Clause (gnat_entity));
+  else
+    gnat_error_node = gnat_entity;
 
-  if (! UI_Is_In_Int_Range (alignment)
-      || ((new_align = UI_To_Int (alignment))
-          > MAX_OFILE_ALIGNMENT / BITS_PER_UNIT))
+  /* Within GCC, an alignment is an integer, so we must make sure a value is
+     specified that fits in that range.  Also, there is an upper bound to
+     alignments we can support/allow.  */
+  if (!UI_Is_In_Int_Range (alignment)
+      || ((new_align = UI_To_Int (alignment)) > max_allowed_alignment))
     post_error_ne_num ("largest supported alignment for& is ^",
-                      gnat_error_node, gnat_entity,
-                      MAX_OFILE_ALIGNMENT / BITS_PER_UNIT);
+                      gnat_error_node, gnat_entity, max_allowed_alignment);
   else if (!(Present (Alignment_Clause (gnat_entity))
             && From_At_Mod (Alignment_Clause (gnat_entity)))
           && new_align * BITS_PER_UNIT < align)
@@ -6261,10 +7083,22 @@ validate_alignment (Uint alignment, Entity_Id gnat_entity, unsigned int align)
                       gnat_error_node, gnat_entity,
                       align / BITS_PER_UNIT);
   else
-    align = MAX (align, new_align == 0 ? 1 : new_align * BITS_PER_UNIT);
+    {
+      new_align = (new_align > 0 ? new_align * BITS_PER_UNIT : 1);
+      if (new_align > align)
+       align = new_align;
+    }
 
   return align;
 }
+
+/* Return the smallest alignment not less than SIZE.  */
+
+static unsigned int
+ceil_alignment (unsigned HOST_WIDE_INT size)
+{
+  return (unsigned int) 1 << (floor_log2 (size - 1) + 1);
+}
 \f
 /* Verify that OBJECT, a type or decl, is something we can implement
    atomically.  If not, give an error for GNAT_ENTITY.  COMP_P is true
@@ -6323,10 +7157,11 @@ check_ok_for_atomic (tree object, Entity_Id gnat_entity, bool comp_p)
        gnat_node = Next_Rep_Item (gnat_node))
     {
       if (!comp_p && Nkind (gnat_node) == N_Pragma
-         && Get_Pragma_Id (Chars (gnat_node)) == Pragma_Atomic)
+         && (Get_Pragma_Id (Chars (Pragma_Identifier (gnat_node)))
+              == Pragma_Atomic))
        gnat_error_point = First (Pragma_Argument_Associations (gnat_node));
       else if (comp_p && Nkind (gnat_node) == N_Pragma
-              && (Get_Pragma_Id (Chars (gnat_node))
+              && (Get_Pragma_Id (Chars (Pragma_Identifier (gnat_node)))
                   == Pragma_Atomic_Components))
        gnat_error_point = First (Pragma_Argument_Associations (gnat_node));
     }
@@ -6367,14 +7202,13 @@ compatible_signatures_p (tree ftype1, tree ftype2)
   return 1;
 }
 \f
-/* Given a type T, a FIELD_DECL F, and a replacement value R, return a new type
-   with all size expressions that contain F updated by replacing F with R.
-   This is identical to GCC's substitute_in_type except that it knows about
-   TYPE_INDEX_TYPE.  If F is NULL_TREE, always make a new RECORD_TYPE, even if
+/* Given a type T, a FIELD_DECL F, and a replacement value R, return a new
+   type with all size expressions that contain F updated by replacing F
+   with R.  If F is NULL_TREE, always make a new RECORD_TYPE, even if
    nothing has changed.  */
 
 tree
-gnat_substitute_in_type (tree t, tree f, tree r)
+substitute_in_type (tree t, tree f, tree r)
 {
   tree new = t;
   tree tem;
@@ -6384,7 +7218,6 @@ gnat_substitute_in_type (tree t, tree f, tree r)
     case INTEGER_TYPE:
     case ENUMERAL_TYPE:
     case BOOLEAN_TYPE:
-    case CHAR_TYPE:
       if (CONTAINS_PLACEHOLDER_P (TYPE_MIN_VALUE (t))
          || CONTAINS_PLACEHOLDER_P (TYPE_MAX_VALUE (t)))
        {
@@ -6397,7 +7230,7 @@ gnat_substitute_in_type (tree t, tree f, tree r)
          new = build_range_type (TREE_TYPE (t), low, high);
          if (TYPE_INDEX_TYPE (t))
            SET_TYPE_INDEX_TYPE
-             (new, gnat_substitute_in_type (TYPE_INDEX_TYPE (t), f, r));
+             (new, substitute_in_type (TYPE_INDEX_TYPE (t), f, r));
          return new;
        }
 
@@ -6424,7 +7257,7 @@ gnat_substitute_in_type (tree t, tree f, tree r)
       return t;
 
     case COMPLEX_TYPE:
-      tem = gnat_substitute_in_type (TREE_TYPE (t), f, r);
+      tem = substitute_in_type (TREE_TYPE (t), f, r);
       if (tem == TREE_TYPE (t))
        return t;
 
@@ -6439,8 +7272,8 @@ gnat_substitute_in_type (tree t, tree f, tree r)
 
     case ARRAY_TYPE:
       {
-       tree component = gnat_substitute_in_type (TREE_TYPE (t), f, r);
-       tree domain = gnat_substitute_in_type (TYPE_DOMAIN (t), f, r);
+       tree component = substitute_in_type (TREE_TYPE (t), f, r);
+       tree domain = substitute_in_type (TYPE_DOMAIN (t), f, r);
 
        if (component == TREE_TYPE (t) && domain == TYPE_DOMAIN (t))
          return t;
@@ -6451,6 +7284,7 @@ gnat_substitute_in_type (tree t, tree f, tree r)
        TYPE_CONVENTION_FORTRAN_P (new) = TYPE_CONVENTION_FORTRAN_P (t);
        layout_type (new);
        TYPE_ALIGN (new) = TYPE_ALIGN (t);
+       TYPE_USER_ALIGN (new) = TYPE_USER_ALIGN (t);
 
        /* If we had bounded the sizes of T by a constant, bound the sizes of
           NEW by the same constant.  */
@@ -6489,7 +7323,7 @@ gnat_substitute_in_type (tree t, tree f, tree r)
            tree new_field = copy_node (field);
 
            TREE_TYPE (new_field)
-             = gnat_substitute_in_type (TREE_TYPE (new_field), f, r);
+             = substitute_in_type (TREE_TYPE (new_field), f, r);
 
            if (DECL_HAS_REP_P (field) && !DECL_INTERNAL_P (field))
              field_has_rep = true;
@@ -6630,32 +7464,28 @@ rm_size (tree gnu_type)
 tree
 create_concat_name (Entity_Id gnat_entity, const char *suffix)
 {
+  Entity_Kind kind = Ekind (gnat_entity);
+
   const char *str = (!suffix ? "" : suffix);
   String_Template temp = {1, strlen (str)};
   Fat_Pointer fp = {str, &temp};
 
   Get_External_Name_With_Suffix (gnat_entity, fp);
 
-#ifdef TARGET_DLLIMPORT_DECL_ATTRIBUTES
   /* A variable using the Stdcall convention (meaning we are running
      on a Windows box) live in a DLL. Here we adjust its name to use
      the jump-table, the _imp__NAME contains the address for the NAME
      variable. */
-  {
-    Entity_Kind kind = Ekind (gnat_entity);
-    const char *prefix = "_imp__";
-    int plen = strlen (prefix);
+  if ((kind == E_Variable || kind == E_Constant)
+      && Has_Stdcall_Convention (gnat_entity))
+    {
+      const char *prefix = "_imp__";
+      int k, plen = strlen (prefix);
 
-    if ((kind == E_Variable || kind == E_Constant)
-       && Convention (gnat_entity) == Convention_Stdcall)
-      {
-       int k;
-       for (k = 0; k <= Name_Len; k++)
-         Name_Buffer [Name_Len - k + plen] = Name_Buffer [Name_Len - k];
-       strncpy (Name_Buffer, prefix, plen);
-      }
-  }
-#endif
+      for (k = 0; k <= Name_Len; k++)
+       Name_Buffer [Name_Len - k + plen] = Name_Buffer [Name_Len - k];
+      strncpy (Name_Buffer, prefix, plen);
+    }
 
   return get_identifier (Name_Buffer);
 }
@@ -6680,8 +7510,7 @@ concat_id_with_name (tree gnu_id, const char *suffix)
 {
   int len = IDENTIFIER_LENGTH (gnu_id);
 
-  strncpy (Name_Buffer, IDENTIFIER_POINTER (gnu_id),
-           IDENTIFIER_LENGTH (gnu_id));
+  strncpy (Name_Buffer, IDENTIFIER_POINTER (gnu_id), len);
   strncpy (Name_Buffer + len, "___", 3);
   len += 3;
   strcpy (Name_Buffer + len, suffix);