8258839: Remove JVM option ExitVMOnVerifyError

Reviewed-by: iignatyev, dcubed, coleenp

Author: Harold Seigel

src/hotspot/share/c1/c1_Runtime1.cpp

@@ -530,14 +530,10 @@ JRT_ENTRY_NO_ASYNC(static address, exception_handler_for_pc_helper(JavaThread* t
  assert(exception_frame.is_deoptimized_frame(), "must be deopted");
  pc = exception_frame.pc();
  }
-#ifdef ASSERT
  assert(exception.not_null(), "NULL exceptions should be handled by throw_exception");
- // Check that exception is a subclass of Throwable, otherwise we have a VerifyError
- if (!(exception->is_a(SystemDictionary::Throwable_klass()))) {
- if (ExitVMOnVerifyError) vm_exit(-1);
- ShouldNotReachHere();
- }
-#endif
+ // Check that exception is a subclass of Throwable
+ assert(exception->is_a(SystemDictionary::Throwable_klass()),
+ "Exception not subclass of Throwable");
 
  // debugging support
  // tracing

src/hotspot/share/interpreter/interpreterRuntime.cpp

@@ -490,14 +490,10 @@ JRT_ENTRY(address, InterpreterRuntime::exception_handler_for_exception(JavaThrea
  should_repeat = false;
 
  // assertions
-#ifdef ASSERT
  assert(h_exception.not_null(), "NULL exceptions should be handled by athrow");
- // Check that exception is a subclass of Throwable, otherwise we have a VerifyError
- if (!(h_exception->is_a(SystemDictionary::Throwable_klass()))) {
- if (ExitVMOnVerifyError) vm_exit(-1);
- ShouldNotReachHere();
- }
-#endif
+ // Check that exception is a subclass of Throwable.
+ assert(h_exception->is_a(SystemDictionary::Throwable_klass()),
+ "Exception not subclass of Throwable");
 
  // tracing
  if (log_is_enabled(Info, exceptions)) {

src/hotspot/share/jvmci/jvmciRuntime.cpp

@@ -257,15 +257,11 @@ JRT_ENTRY_NO_ASYNC(static address, exception_handler_for_pc_helper(JavaThread* t
  assert(exception_frame.is_deoptimized_frame(), "must be deopted");
  pc = exception_frame.pc();
  }
-#ifdef ASSERT
  assert(exception.not_null(), "NULL exceptions should be handled by throw_exception");
  assert(oopDesc::is_oop(exception()), "just checking");
- // Check that exception is a subclass of Throwable, otherwise we have a VerifyError
- if (!(exception->is_a(SystemDictionary::Throwable_klass()))) {
- if (ExitVMOnVerifyError) vm_exit(-1);
- ShouldNotReachHere();
- }
-#endif
+ // Check that exception is a subclass of Throwable
+ assert(exception->is_a(SystemDictionary::Throwable_klass()),
+ "Exception not subclass of Throwable");
 
  // debugging support
  // tracing

src/hotspot/share/runtime/globals.hpp

@@ -1143,10 +1143,6 @@ const intx ObjectAlignmentInBytes = 8;
  notproduct(bool, PrintSymbolTableSizeHistogram, false, \
  "print histogram of the symbol table") \
  \
- notproduct(bool, ExitVMOnVerifyError, false, \
- "standard exit from VM if bytecode verify error " \
- "(only in debug mode)") \
- \
  product(ccstr, AbortVMOnException, NULL, DIAGNOSTIC, \
  "Call fatal if this exception is thrown. Example: " \
  "java -XX:AbortVMOnException=java.lang.NullPointerException Foo") \