Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8009595: The UseSplitVerifier option needs to be deprecated.
authorhseigel
Tue, 26 Mar 2013 09:06:16 -0400
changeset 16445 cb44d5b3ad89
parent 16443 dd0dcfac0c61
child 16446 7d1c5acb4866
8009595: The UseSplitVerifier option needs to be deprecated. Summary: Put UseSplitVerifier option on the deprecated list. Reviewed-by: dcubed, kmo, acorn
hotspot/src/share/vm/classfile/classFileParser.cpp file | annotate | diff | comparison | revisions
hotspot/src/share/vm/classfile/verifier.cpp file | annotate | diff | comparison | revisions
hotspot/src/share/vm/runtime/arguments.cpp file | annotate | diff | comparison | revisions
hotspot/src/share/vm/runtime/globals.hpp file | annotate | diff | comparison | revisions 
--- a/hotspot/src/share/vm/classfile/classFileParser.cpp	Mon Mar 25 09:36:15 2013 -0700
+++ b/hotspot/src/share/vm/classfile/classFileParser.cpp	Tue Mar 26 09:06:16 2013 -0400
@@ -2196,8 +2196,7 @@
                                       true,     // is LVTT
                                       CHECK_(nullHandle));
           lvtt_cnt++;
-        } else if (UseSplitVerifier &&
-                   _major_version >= Verifier::STACKMAP_ATTRIBUTE_MAJOR_VERSION &&
+        } else if (_major_version >= Verifier::STACKMAP_ATTRIBUTE_MAJOR_VERSION &&
                    _cp->symbol_at(code_attribute_name_index) == vmSymbols::tag_stack_map_table()) {
           // Stack map is only needed by the new verifier in JDK1.5.
           if (parsed_stackmap_attribute) {
--- a/hotspot/src/share/vm/classfile/verifier.cpp	Mon Mar 25 09:36:15 2013 -0700
+++ b/hotspot/src/share/vm/classfile/verifier.cpp	Tue Mar 26 09:06:16 2013 -0400
@@ -127,8 +127,7 @@
     if (TraceClassInitialization) {
       tty->print_cr("Start class verification for: %s", klassName);
     }
-    if (UseSplitVerifier &&
-        klass->major_version() >= STACKMAP_ATTRIBUTE_MAJOR_VERSION) {
+    if (klass->major_version() >= STACKMAP_ATTRIBUTE_MAJOR_VERSION) {
       ClassVerifier split_verifier(klass, THREAD);
       split_verifier.verify_class(THREAD);
       exception_name = split_verifier.result();
--- a/hotspot/src/share/vm/runtime/arguments.cpp	Mon Mar 25 09:36:15 2013 -0700
+++ b/hotspot/src/share/vm/runtime/arguments.cpp	Tue Mar 26 09:06:16 2013 -0400
@@ -260,6 +260,7 @@
   { "CMSRevisitStackSize",           JDK_Version::jdk(8), JDK_Version::jdk(9) },
   { "PrintRevisitStats",             JDK_Version::jdk(8), JDK_Version::jdk(9) },
   { "UseVectoredExceptions",         JDK_Version::jdk(8), JDK_Version::jdk(9) },
+  { "UseSplitVerifier",              JDK_Version::jdk(8), JDK_Version::jdk(9) },
 #ifdef PRODUCT
   { "DesiredMethodLimit",
                            JDK_Version::jdk_update(7, 2), JDK_Version::jdk(8) },
--- a/hotspot/src/share/vm/runtime/globals.hpp	Mon Mar 25 09:36:15 2013 -0700
+++ b/hotspot/src/share/vm/runtime/globals.hpp	Tue Mar 26 09:06:16 2013 -0400
@@ -679,9 +679,6 @@
   product(bool, UseCompilerSafepoints, true,                                \
           "Stop at safepoints in compiled code")                            \
                                                                             \
-  product(bool, UseSplitVerifier, true,                                     \
-          "use split verifier with StackMapTable attributes")               \
-                                                                            \
   product(bool, FailOverToOldVerifier, true,                                \
           "fail over to old verifier when split verifier fails")            \
                                                                             \
jdk-sandbox