class.c (layout_vtable_decl): Fix off by one error on build_index_type.

	* class.c (layout_vtable_decl): Fix off by one error on
	build_index_type.
	(build_vtt): Likewise.
	(build_ctor_vtbl_group): Likewise.

From-SVN: r42903

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 8f5ebaa69ea8f24c89784ad85d00afbe7312a336..94658acc1cea1f8c1a9fa92db12fc6f189e93b1b 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,10 @@
+2001-06-05  Nathan Sidwell  <nathan@codesourcery.com>
+
+       * class.c (layout_vtable_decl): Fix off by one error on
+       build_index_type.
+       (build_vtt): Likewise.
+       (build_ctor_vtbl_group): Likewise.
+
 2001-06-05  Nathan Sidwell  <nathan@codesourcery.com>
 
        * class.c (maybe_indent_hierarchy): New function.

diff --git a/gcc/cp/class.c b/gcc/cp/class.c
index 7a7f69b0b7d301eaf1258797e2fd532aea1abe79..afe2da403f7ab59d31d0eec0b128576ef01236de 100644 (file)
--- a/gcc/cp/class.c
+++ b/gcc/cp/class.c
@@ -2446,13 +2446,11 @@ layout_vtable_decl (binfo, n)
      tree binfo;
      int n;
 {
-  tree itype;
   tree atype;
   tree vtable;
 
-  itype = size_int (n);
   atype = build_cplus_array_type (vtable_entry_type, 
-                                 build_index_type (itype));
+                                 build_index_type (size_int (n - 1)));
   layout_type (atype);
 
   /* We may have to grow the vtable.  */
@@ -4856,7 +4854,7 @@ layout_virtual_bases (t, offsets)
   TYPE_ALIGN (t) = MAX (TYPE_ALIGN (t), BITS_PER_UNIT);
 
   /* Go through the virtual bases, allocating space for each virtual
-     base that is not already a primary base class.  Under the new
+     base that is not already a primary base class.  Under the old
      ABI, these are allocated according to a depth-first left-to-right
      postorder traversal; in the new ABI, inheritance graph order is
      used instead.  */
@@ -7149,7 +7147,7 @@ build_vtt (t)
     return;
 
   /* Figure out the type of the VTT.  */
-  type = build_index_type (size_int (list_length (inits)));
+  type = build_index_type (size_int (list_length (inits) - 1));
   type = build_cplus_array_type (const_ptr_type_node, type);
                                 
   /* Now, build the VTT object itself.  */
@@ -7468,7 +7466,7 @@ build_ctor_vtbl_group (binfo, t)
   inits = TREE_VALUE (list);
 
   /* Figure out the type of the construction vtable.  */
-  type = build_index_type (size_int (list_length (inits)));
+  type = build_index_type (size_int (list_length (inits) - 1));
   type = build_cplus_array_type (vtable_entry_type, type);
   TREE_TYPE (vtbl) = type;