Code Review for hotspot

Prepared by:andrew on Tue Jan 10 05:38:19 GMT 2017
Workspace:/home/andrew/projects/openjdk/upstream/jdk6/hotspot
Compare against: ssh://hg.openjdk.java.net/jdk6/jdk6/hotspot
Compare against version:4128
Summary of changes: 2738 lines changed: 1915 ins; 234 del; 589 mod; 88847 unchg
Changeset: hotspot.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw .hgtags

rev 4142 : Added tag jdk6-b41 for changeset 1ae05a34e052
1 line changed: 1 ins; 0 del; 0 mod; 464 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw agent/src/share/classes/sun/jvm/hotspot/oops/ConstMethod.java

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
66 lines changed: 55 ins; 10 del; 1 mod; 413 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw agent/src/share/classes/sun/jvm/hotspot/oops/GenerateOopMap.java

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
22 lines changed: 2 ins; 0 del; 20 mod; 2296 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw agent/src/share/classes/sun/jvm/hotspot/oops/Method.java

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
9 lines changed: 8 ins; 1 del; 0 mod; 359 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw agent/src/share/classes/sun/jvm/hotspot/tools/jcore/ClassWriter.java

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
15 lines changed: 3 ins; 0 del; 12 mod; 716 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw agent/src/share/classes/sun/jvm/hotspot/ui/classbrowser/HTMLGenerator.java

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
47 lines changed: 18 ins; 16 del; 13 mod; 2136 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/sparc/vm/c1_LIRAssembler_sparc.cpp

rev 4139 : 8160591: Improve internal array handling
Reviewed-by: kvn
Contributed-by: Xiang Yuan <xiang.yuan@linaro.org>, Zoltan Majo <zoltan.majo@oracle.com>
21 lines changed: 21 ins; 0 del; 0 mod; 3409 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/x86/vm/c1_LIRAssembler_x86.cpp

rev 4139 : 8160591: Improve internal array handling
Reviewed-by: kvn
Contributed-by: Xiang Yuan <xiang.yuan@linaro.org>, Zoltan Majo <zoltan.majo@oracle.com>
17 lines changed: 17 ins; 0 del; 0 mod; 3764 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/x86/vm/stubGenerator_x86_32.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 2362 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/x86/vm/stubGenerator_x86_64.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 3137 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/zero/vm/cppInterpreter_zero.cpp

rev 4141 : 8078628: Zero build fails with pre-compiled headers disabled
Summary: Add missing includes to make Zero builds without pre-compiled headers work
Reviewed-by: omajid
2 lines changed: 1 ins; 0 del; 1 mod; 1609 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/zero/vm/entry_zero.hpp

rev 4141 : 8078628: Zero build fails with pre-compiled headers disabled
Summary: Add missing includes to make Zero builds without pre-compiled headers work
Reviewed-by: omajid
3 lines changed: 2 ins; 0 del; 1 mod; 78 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/cpu/zero/vm/nativeInst_zero.cpp

rev 4141 : 8078628: Zero build fails with pre-compiled headers disabled
Summary: Add missing includes to make Zero builds without pre-compiled headers work
Reviewed-by: omajid
2 lines changed: 1 ins; 0 del; 1 mod; 59 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/os/solaris/vm/dtraceJSDT_solaris.cpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
42 lines changed: 1 ins; 39 del; 2 mod; 652 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/c1/c1_IR.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
3 lines changed: 0 ins; 0 del; 3 mod; 1300 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/ci/ciMethod.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
8 lines changed: 0 ins; 1 del; 7 mod; 1225 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/ci/ciUtilities.hpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
2 lines changed: 0 ins; 0 del; 2 mod; 112 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/classFileParser.cpp

rev 4133 : 8159503: Amend Annotation Actions
Reviewed-by: rprotacio
rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
53 lines changed: 20 ins; 7 del; 26 mod; 4384 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/classFileParser.hpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
2 lines changed: 0 ins; 0 del; 2 mod; 293 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/stackMapFrame.cpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
115 lines changed: 63 ins; 4 del; 48 mod; 304 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/stackMapFrame.hpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
89 lines changed: 72 ins; 6 del; 11 mod; 221 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/stackMapTable.cpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
36 lines changed: 7 ins; 7 del; 22 mod; 406 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/stackMapTable.hpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
5 lines changed: 1 ins; 1 del; 3 mod; 183 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/stackMapTableFormat.hpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
rev 4137 : 8159511: Stack map validation
Reviewed-by: acorn, mschoene
Contributed-by: harold.seigel@oracle.com
129 lines changed: 62 ins; 19 del; 48 mod; 856 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/systemDictionary.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
1 line changed: 0 ins; 1 del; 0 mod; 2956 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/verificationType.cpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
27 lines changed: 4 ins; 4 del; 19 mod; 120 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/verificationType.hpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
3 lines changed: 0 ins; 0 del; 3 mod; 311 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/verifier.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
rev 4137 : 8159511: Stack map validation
Reviewed-by: acorn, mschoene
Contributed-by: harold.seigel@oracle.com
673 lines changed: 429 ins; 53 del; 191 mod; 2296 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/classfile/verifier.hpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
190 lines changed: 168 ins; 2 del; 20 mod; 269 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/code/dependencies.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 1665 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/code/nmethod.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
2 lines changed: 1 ins; 0 del; 1 mod; 2801 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/gc_implementation/parallelScavenge/psPromotionManager.hpp

rev 4129 : 8169448: OpenJDK 6 fails to build without pre-compiled headers
Summary: psPromotionManager.hpp is missing an import for psOldGen
Reviewed-by: omajid
1 line changed: 1 ins; 0 del; 0 mod; 202 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/interpreter/interpreterRuntime.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
1 line changed: 0 ins; 1 del; 0 mod; 1271 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/cardTableModRefBS.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 729 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/dump.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
3 lines changed: 0 ins; 3 del; 0 mod; 1595 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/oopFactory.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
5 lines changed: 3 ins; 0 del; 2 mod; 180 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/oopFactory.hpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
2 lines changed: 2 ins; 0 del; 0 mod; 115 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/memory/universe.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
3 lines changed: 2 ins; 0 del; 1 mod; 1671 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/constMethodKlass.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
28 lines changed: 8 ins; 11 del; 9 mod; 268 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/constMethodKlass.hpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
2 lines changed: 1 ins; 0 del; 1 mod; 97 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/constMethodOop.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
42 lines changed: 38 ins; 0 del; 4 mod; 127 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/constMethodOop.hpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
44 lines changed: 26 ins; 14 del; 4 mod; 298 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/generateOopMap.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
19 lines changed: 1 ins; 2 del; 16 mod; 2542 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/methodOop.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
rev 4135 : 7181632: nsk classLoad001_14 failure and CompileTheWorld crash after 7178145.
Summary: Need to copy the inlined exception table to the new constMethodOop during method rewriting.
Reviewed-by: coleenp, dholmes
26 lines changed: 11 ins; 7 del; 8 mod; 1605 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/oops/methodOop.hpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
70 lines changed: 66 ins; 4 del; 0 mod; 838 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/opto/memnode.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 4284 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/prims/forte.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 652 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/prims/jvm.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
rev 4138 : 8159515: Improve indy validation
Reviewed-by: jrose, hseigel, vlivanov, bmoloden, ctornqvi, mschoene
rev 4140 : 8162419: closed/com/oracle/jfr/runtime/TestVMInfoEvent.sh failing after JDK-8155968
Summary: Under error conditions, always return -1 and perform null termination regardless of the behavior of underlying vsnprintf() implementation.
Reviewed-by: dholmes, cjplummer
22 lines changed: 12 ins; 2 del; 8 mod; 4585 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/prims/jvmtiClassFileReconstituter.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
10 lines changed: 0 ins; 1 del; 9 mod; 742 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/prims/jvmtiRedefineClasses.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
11 lines changed: 0 ins; 6 del; 5 mod; 3477 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/prims/methodHandleWalk.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
5 lines changed: 0 ins; 3 del; 2 mod; 2084 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/arguments.cpp

rev 4130 : 8155968: Update command line options
Reviewed-by: gthornbr, hseigel, mschoene
Contributed-by: gerard.ziemski@oracle.com
24 lines changed: 8 ins; 1 del; 15 mod; 3390 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/deoptimization.cpp

rev 4140 : 8162419: closed/com/oracle/jfr/runtime/TestVMInfoEvent.sh failing after JDK-8155968
Summary: Under error conditions, always return -1 and perform null termination regardless of the behavior of underlying vsnprintf() implementation.
Reviewed-by: dholmes, cjplummer
4 lines changed: 0 ins; 4 del; 0 mod; 1982 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/globals.hpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
3 lines changed: 3 ins; 0 del; 0 mod; 3969 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/java.cpp

rev 4140 : 8162419: closed/com/oracle/jfr/runtime/TestVMInfoEvent.sh failing after JDK-8155968
Summary: Under error conditions, always return -1 and perform null termination regardless of the behavior of underlying vsnprintf() implementation.
Reviewed-by: dholmes, cjplummer
17 lines changed: 11 ins; 0 del; 6 mod; 758 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/relocator.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
18 lines changed: 0 ins; 1 del; 17 mod; 753 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/sharedRuntime.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
rev 4132 : 8157176: Improved classfile parsing
Reviewed-by: pliden
4 lines changed: 0 ins; 2 del; 2 mod; 2956 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/signature.cpp

rev 4132 : 8157176: Improved classfile parsing
Reviewed-by: pliden
43 lines changed: 42 ins; 0 del; 1 mod; 460 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/runtime/vmStructs.cpp

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
7 lines changed: 6 ins; 1 del; 0 mod; 3267 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/services/diagnosticArgument.cpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 273 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/utilities/exceptions.hpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
5 lines changed: 0 ins; 0 del; 5 mod; 287 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/utilities/ostream.cpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
44 lines changed: 42 ins; 0 del; 2 mod; 1136 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/utilities/ostream.hpp

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
18 lines changed: 16 ins; 0 del; 2 mod; 298 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/vm/utilities/taskqueue.hpp

rev 4131 : 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
8015265: revise the fix for 8007037
Reviewed-by: sspitsyn, dholmes, dcubed, coleenp
Contributed-by: jeremymanson@google.com, calvin.cheung@oracle.com
24 lines changed: 18 ins; 0 del; 6 mod; 800 unchg

------ ------ ------ ------ --- New ----- Raw agent/src/share/classes/sun/jvm/hotspot/oops/ExceptionTableElement.java

rev 4134 : 7178145: Change constMethodOop::_exception_table to optionally inlined u2 table.
Summary: Change constMethodOop::_exception_table to optionally inlined u2 table.
Reviewed-by: bdelsart, coleenp, kamg
81 lines changed: 81 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/runtime/7116786/Test7116786.java

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
486 lines changed: 486 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw test/runtime/7116786/testcases.jar

rev 4136 : 7116786: RFE: Detailed information on VerifyErrors
Summary: Provide additional detail in VerifyError messages
Reviewed-by: sspitsyn, acorn
74 lines changed: 74 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/andrew/bin/webrev (vers 25.16-hg+openjdk.java.net).