src/hotspot/share/oops/instanceKlass.cpp

*** 2471,2481 ****
    }
  }
  
  // returns true IFF is_in_error_state() has been changed as a result of this call.
  bool InstanceKlass::check_sharing_error_state() {
!   assert(DumpSharedSpaces, "should only be called during dumping");
    bool old_state = is_in_error_state();
  
    if (!is_in_error_state()) {
      bool bad = false;
      for (InstanceKlass* sup = java_super(); sup; sup = sup->java_super()) {
--- 2471,2481 ----
    }
  }
  
  // returns true IFF is_in_error_state() has been changed as a result of this call.
  bool InstanceKlass::check_sharing_error_state() {
!   Arguments::assert_is_dumping_archive();
    bool old_state = is_in_error_state();
  
    if (!is_in_error_state()) {
      bool bad = false;
      for (InstanceKlass* sup = java_super(); sup; sup = sup->java_super()) {
*** 3574,3586 ****
    // Verify that klass is present in ClassLoaderData
    guarantee(class_loader_data()->contains_klass(this),
              "this class isn't found in class loader data");
  
    // Verify vtables
!   if (is_linked()) {
!     // $$$ This used to be done only for m/s collections.  Doing it
!     // always seemed a valid generalization.  (DLD -- 6/00)
      vtable().verify(st);
    }
  
    // Verify first subklass
    if (subklass() != NULL) {
--- 3574,3584 ----
    // Verify that klass is present in ClassLoaderData
    guarantee(class_loader_data()->contains_klass(this),
              "this class isn't found in class loader data");
  
    // Verify vtables
!   if (is_linked() && !is_in_error_state()) {
      vtable().verify(st);
    }
  
    // Verify first subklass
    if (subklass() != NULL) {