cp-tree.h (struct lang_decl_flags): Remove needs_final_overrider.

	* cp-tree.h (struct lang_decl_flags): Remove
	needs_final_overrider.
	(DECL_NEEDS_FINAL_OVERRIDER_P): Remove.
	* decl.c (duplicate_decls): Do not copy DECL_NEEDS_FINAL_OVERRIDER_P.
	* class.c (finish_struct_bits): Correct comment about
	CLASSTYPE_PURE_VIRTUALS.
	* search.c (get_pure_virtuals): Remove useless loop.

From-SVN: r87541

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index e623148e275ef8922926becd55db77bd9e88ced6..a2bdd496aaeed42f0ce5cf992374e0a7749b93d7 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,13 @@
+2004-09-15  Nathan Sidwell  <nathan@codesourcery.com>
+
+       * cp-tree.h (struct lang_decl_flags): Remove
+       needs_final_overrider.
+       (DECL_NEEDS_FINAL_OVERRIDER_P): Remove.
+       * decl.c (duplicate_decls): Do not copy DECL_NEEDS_FINAL_OVERRIDER_P.
+       * class.c (finish_struct_bits): Correct comment about
+       CLASSTYPE_PURE_VIRTUALS.
+       * search.c (get_pure_virtuals): Remove useless loop.
+
 2004-09-14  Mark Mitchell  <mark@codesourcery.com>
 
        PR c++/17324

diff --git a/gcc/cp/class.c b/gcc/cp/class.c
index 999551b435d622e3648d637b64104be56be53daf..d31295a87a637b269c846e5fe13b734d86f26bf8 100644 (file)
--- a/gcc/cp/class.c
+++ b/gcc/cp/class.c
@@ -1439,8 +1439,8 @@ finish_struct_bits (tree t)
     }
 
   if (BINFO_N_BASE_BINFOS (TYPE_BINFO (t)) && TYPE_POLYMORPHIC_P (t))
-    /* For a class w/o baseclasses, `finish_struct' has set
-       CLASS_TYPE_ABSTRACT_VIRTUALS correctly (by definition).
+    /* For a class w/o baseclasses, 'finish_struct' has set
+       CLASSTYPE_PURE_VIRTUALS correctly (by definition).
        Similarly for a class whose base classes do not have vtables.
        When neither of these is true, we might have removed abstract
        virtuals (by providing a definition), added some (by declaring
@@ -1448,7 +1448,7 @@ finish_struct_bits (tree t)
        recalculate what's really an abstract virtual at this point (by
        looking in the vtables).  */
     get_pure_virtuals (t);
-
+  
   /* If this type has a copy constructor or a destructor, force its
      mode to be BLKmode, and force its TREE_ADDRESSABLE bit to be
      nonzero.  This will cause it to be passed by invisible reference

diff --git a/gcc/cp/cp-tree.h b/gcc/cp/cp-tree.h
index aa25658c18599a4def02c5dc1b54f50691844bab..5d788af0d1bb9df8873a68acbb4f64d6af78df76 100644 (file)
--- a/gcc/cp/cp-tree.h
+++ b/gcc/cp/cp-tree.h
@@ -1522,16 +1522,15 @@ struct lang_decl_flags GTY(())
   unsigned use_template : 2;
   unsigned nonconverting : 1;
   unsigned not_really_extern : 1;
-  unsigned needs_final_overrider : 1;
   unsigned initialized_in_class : 1;
   unsigned assignment_operator_p : 1;
-
   unsigned u1sel : 1;
+ 
   unsigned u2sel : 1;
   unsigned can_be_full : 1;
   unsigned this_thunk_p : 1;
   unsigned repo_available_p : 1;
-  unsigned dummy : 3;
+  unsigned dummy : 4;
 
   union lang_decl_u {
     /* In a FUNCTION_DECL for which DECL_THUNK_P holds, this is
@@ -1883,11 +1882,6 @@ struct lang_decl GTY(())
 #define DECL_PURE_VIRTUAL_P(NODE) \
   (DECL_LANG_SPECIFIC (NODE)->decl_flags.pure_virtual)
 
-/* Nonzero for FUNCTION_DECL means that this member function
-   must be overridden by derived classes.  */
-#define DECL_NEEDS_FINAL_OVERRIDER_P(NODE) \
-  (DECL_LANG_SPECIFIC (NODE)->decl_flags.needs_final_overrider)
-
 /* True (in a FUNCTION_DECL) if NODE is a virtual function that is an
    invalid overrider for a function from a base class.  Once we have
    complained about an invalid overrider we avoid complaining about it

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 97b5a1a03579bdc04ba820505c5bba55d9e37fdc..6128598f8b800c185df7ac27c6ca5721b01aa712 100644 (file)
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -1582,7 +1582,6 @@ duplicate_decls (tree newdecl, tree olddecl)
       DECL_STATIC_DESTRUCTOR (newdecl) |= DECL_STATIC_DESTRUCTOR (olddecl);
       DECL_PURE_VIRTUAL_P (newdecl) |= DECL_PURE_VIRTUAL_P (olddecl);
       DECL_VIRTUAL_P (newdecl) |= DECL_VIRTUAL_P (olddecl);
-      DECL_NEEDS_FINAL_OVERRIDER_P (newdecl) |= DECL_NEEDS_FINAL_OVERRIDER_P (olddecl);
       DECL_THIS_STATIC (newdecl) |= DECL_THIS_STATIC (olddecl);
       if (DECL_OVERLOADED_OPERATOR_P (olddecl) != ERROR_MARK)
        SET_OVERLOADED_OPERATOR_CODE

diff --git a/gcc/cp/search.c b/gcc/cp/search.c
index 906491ed02c13c6717315072d9b8247acf996cd3..99a945352e1f96ea7971df5a730053d127e7187c 100644 (file)
--- a/gcc/cp/search.c
+++ b/gcc/cp/search.c
@@ -1908,20 +1908,6 @@ get_pure_virtuals (tree type)
 
   /* Put the pure virtuals in dfs order.  */
   CLASSTYPE_PURE_VIRTUALS (type) = nreverse (CLASSTYPE_PURE_VIRTUALS (type));
-
-  for (vbases = CLASSTYPE_VBASECLASSES (type), ix = 0;
-       VEC_iterate (tree, vbases, ix, binfo); ix++)
-    {
-      tree virtuals;
-      
-      for (virtuals = BINFO_VIRTUALS (binfo); virtuals;
-          virtuals = TREE_CHAIN (virtuals))
-       {
-         tree base_fndecl = BV_FN (virtuals);
-         if (DECL_NEEDS_FINAL_OVERRIDER_P (base_fndecl))
-           error ("`%#D' needs a final overrider", base_fndecl);
-       }
-    }
 }
 \f
 /* DEPTH-FIRST SEARCH ROUTINES.  */