Code Review for hotspot

Prepared by:sgehwolf on Mon Nov 2 16:56:38 CET 2015
Workspace:/home/sgehwolf/Documents/openjdk/upstream-sources/openjdk9-hs-rt/hotspot
Compare against: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot
Compare against version:b390353dd19e
Summary of changes: 7 lines changed: 0 ins; 5 del; 2 mod; 3585 unchg
Patch of changes: hotspot.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/zero/vm/compiledIC_zero.cpp 

@    rev 9248 : 8141138: Zero fails to build
|\   Summary: Adjust to recent JDK-9 changes.
| \
| |\
+---o  rev 8813 : 8130309: Need to bailout cleanly if creation of stubs fails when codecache is out of space
| |/   Summary: Check for failed expansion of stub section in code buffer and bailout.
| |    Reviewed-by: kvn, adinn, dlong, roland, twisti
| o  rev 6647 : 8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
|/   Reviewed-by: vlivanov, ehelin
o  rev 4565 : 8003853: specify offset of IC load in java_to_interp stub
|  Summary: refactored code to allow platform-specific differences
|  Reviewed-by: dlong, twisti
|  Contributed-by: Goetz Lindenmaier <goetz.lindenmaier@sap.com>
1 line changed: 0 ins; 0 del; 1 mod; 94 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/zero/vm/relocInfo_zero.cpp 

@    rev 9248 : 8141138: Zero fails to build
|\   Summary: Adjust to recent JDK-9 changes.
| \
| |\
| | \
| | |\
| | | \
| | | |\
| | | | \
| | | | |\
| | | | | \
| | | | | |\
| | | | | | \
| | | | | | |\
+-+-+-+-+-+---o  rev 5753 : 8029233: Update copyright year to match last edit in jdk8 hotspot repository for 2013
| | | | | | |/   Summary: Copyright year updated for files modified during 2013
| | | | | | |    Reviewed-by: twisti, iveresov
+-+-+-+-+---o  rev 5110 : 8022956: Clang: enable return type warnings on BSD
| | | | | |/   Reviewed-by: coleenp, sla
+-+-+-+---o  rev 4879 : 8017308: Remove unused breakpoint relocation type
| | | | |/   Summary: remove unused breakpoint relocation type
| | | | |    Reviewed-by: kvn
+-+-+---o  rev 3802 : 8000780: make Zero build and run with JDK8
| | | |/   Reviewed-by: coleenp, dholmes, twisti
| | | |    Contributed-by: Roman Kennke <rkennke@redhat.com>
+-+---o  rev 2294 : 7032458: Zero and Shark fixes
| | |/   Reviewed-by: twisti
| | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+---o  rev 1879 : 6989984: Use standard include model for Hospot
| |/   Summary: Replaced MakeDeps and the includeDB files with more standardized solutions.
| |    Reviewed-by: coleenp, kvn, kamg
| o  rev 1472 : 6941466: Oracle rebranding changes for Hotspot repositories
|/   Summary: Change all the Sun copyrights to Oracle copyright
|    Reviewed-by: ohair
o  rev 1010 : 6890308: integrate zero assembler hotspot changes
|  Reviewed-by: never
|  Contributed-by: gbenson@redhat.com
5 lines changed: 0 ins; 5 del; 0 mod; 64 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/interpreter/bytecodeInterpreter.cpp 

@    rev 9248 : 8141138: Zero fails to build
|\   Summary: Adjust to recent JDK-9 changes.
| \
| |\
| | \
| | |\
| | | \
| | | |\
| | | | \
| | | | |\
| | | | | \
| | | | | |\
| | | | | | \
| | | | | | |\
| | | | | | | \
| | | | | | | |\
| | | | | | | | \
| | | | | | | | |\
| | | | | | | | | \
| | | | | | | | | |\
| | | | | | | | | | \
| | | | | | | | | | |\
| | | | | | | | | | | \
| | | | | | | | | | | |\
| | | | | | | | | | | | \
| | | | | | | | | | | | |\
| | | | | | | | | | | | | \
| | | | | | | | | | | | | |\
| | | | | | | | | | | | | | \
| | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 9212 : 8139203: Consistent naming for klass type predicates
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   8138923: Remove oop coupling with InstanceKlass subclasses
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Summary: Renamed oop_is_instance and friends, removed the functions in oop that dug down into InstanceKlass.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: jrose, lfoltan, stefank
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 9203 : 8139163: InstanceKlass::cast passes through NULL
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Reduce raw (InstanceKlass*) casts and InstanceKlass::cast, which no long allows null
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti, kbarrett
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 9098 : 8136577: Make AbortVMOnException available in product builds
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: coleenp
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 9013 : 8080775: Better argument formatting for assert() and friends
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: kbarrett, pliden
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 8410 : 8079792: GC directory structure cleanup
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: brutisso, stefank, david
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 8070 : 8076492: Make common code from template interpreter code
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Move case statement out of assembly code
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: minqi, sspitsyn, dholmes
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 8064 : 8076181: bytecodeInterpreter.cpp refers to unknown labels.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: coleenp, dholmes
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 8023 : 8074345: Enable RewriteBytecodes when VM runs with CDS
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Enable bytecode rewriting when CDS turned on.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: coleenp, iklam
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: yumin.qi@oracle.com, ioi.lam@oracle.com
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 7975 : 8074543: Missing symbol "objArrayOopDesc::obj_at" when buiding with CPP Interpreter
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: stefank
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: thomas.stuefe@gmail.com
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 7698 : 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Changed interpreter and compilation policies to allow using CompileThresholdScaling on a per-method level
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: jrose, kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 7044 : 8048721: -XX:+PrintCompilation prints negative bci for non entrant OSR methods
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Removed 'InvalidOSREntryBci' and checking nmethod::_state instead to determine if an osr method is non-entrant.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn, vlivanov, drchase
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 6892 : 8055231: ZERO variant build is broken
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Fix zero build.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: coleenp
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Severin Gehwolf <sgehwolf@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 6668 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: stefank, coleenp, dholmes
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 6639 : 8044775: Improve usage of umbrella header atomic.inline.hpp.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: stefank, kvn
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 6392 : 8042195: Introduce umbrella header orderAccess.inline.hpp.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: dholmes, kvn, stefank, twisti
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 6389 : 8042443: Fix C++-Interpreter after "8036956: remove EnableInvokeDynamic flag"
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Insert missing bracket to make the interpreter build again
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: sla
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Volker Simonis <volker.simonis@gmail.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 6351 : 8036956: remove EnableInvokeDynamic flag
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: The EnableInvokeDynamic flag and all support code is removed because it is not longer used in JDK 9.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn, twisti
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Tobias Hartmann <tobias.hartmann@oracle.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 6083 : 8036146: make CPP interpreter build again
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: fix build of CPP interpreter on x86 and sparc
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5992 : 8035392: cppInterpreter: fix message of NPE
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: kvn
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: axel.siebenborn@sap.com
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5899 : Merge
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/|
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5895 : 8029101: PPC64 (part 211): ordering of Independent Reads of Independent Writes
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: dholmes, kvn
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Contributed-by: martin.doerr@sap.com
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5889 : 8029957: PPC64 (part 213): cppInterpreter: memory ordering for object initialization
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Summary: Add StoreStore barriers after object initialization and after constructor calls in the C++ interpreter.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5876 : 8028514: PPC64: Fix C++ Interpreter after '7195622: CheckUnhandledOops has limited usefulness now'
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Summary: fix CPP-interpreter after CheckUnhandledOops was re-enabled in the fastdebug build
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn, dholmes, lfoltan
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 5863 : 8024468: PPC64 (part 201): cppInterpreter: implement bytecode profiling
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Summary: Implement profiling for c2 jit compilation. Also enable new cppInterpreter features.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5860 : 8024469: PPC64 (part 202): cppInterpreter: support for OSR.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Summary: Call OSR migration with last java frame.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5854 : 8023033: PPC64 (part 13): basic changes for AIX
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Summary: Added AIX includes alpha-sorted before BSD. Fix compilation issues with xlC in shared code. Basic shared platform dependend adaption (vm_version etc.).
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn, dholmes, stefank
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5847 : Merge
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 5844 : 8020121: PPC64: fix build in cppInterpreter after 8019519
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 5843 : 8019519: PPC64 (part 105): C interpreter: implement support for jvmti early return.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: sspitsyn, kvn
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 5842 : 8019518: PPC64 (part 103): cppInterpreter: implement support for compressed Oops
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn, coleenp
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | o  rev 5841 : 8019517: PPC64 (part 102): cppInterpreter: implement G1 support
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |  Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-----o  rev 5839 : Merge
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-------o  rev 5838 : 8017317: PPC64 (part 7): cppInterpreter: implement support for biased locking
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn, dholmes
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5753 : 8029233: Update copyright year to match last edit in jdk8 hotspot repository for 2013
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Copyright year updated for files modified during 2013
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti, iveresov
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4915 : 8019382: PPC64: Fix bytecodeInterpreter to compile with '-Wunused-value'
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: cast the offending expressions to (void)
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn, coleenp
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4884 : 8017531: 8010460 changes broke bytecodeInterpreter.cpp
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Replace _indy by _jsr292 and also fix VERIFY_OOP macros.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4790 : 8010460: Interpreter on some platforms loads ConstMethod::_max_stack and misses extra stack slots for JSR 292
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: ConstMethod::max_stack() doesn't account for JSR 292 appendix.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4630 : 8013067: Zero builds are broken after 8010862.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Fixed broken Zero build.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti, coleenp, kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4100 : 6518907: cleanup IA64 specific code in Hotspot
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: removed unused IA64 specific code
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti, kvn, dholmes
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 3843 : 8001471: Klass::cast() does nothing
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Remove function Klass::cast() and calls to it.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: dholmes, coleenp
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 3802 : 8000780: make Zero build and run with JDK8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: coleenp, dholmes, twisti
| | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Roman Kennke <rkennke@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 3707 : 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
| | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Capitalize these metadata types (and objArrayKlass)
| | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: stefank, twisti, kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 3602 : 6964458: Reimplement class meta-data storage to use native memory
| | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Remove PermGen, allocate meta-data in metaspace linked to class loaders, rewrite GC walking, rewrite and rename metadata to be C++ classes
| | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: jmasa, stefank, never, coleenp, kvn, brutisso, mgerdin, dholmes, jrose, twisti, roland
| | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: jmasa <jon.masamitsu@oracle.com>, stefank <stefan.karlsson@oracle.com>, mgerdin <mikael.gerdin@oracle.com>, never <tom.rodriguez@oracle.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 3534 : 7023639: JSR 292 method handle invocation needs a fast path for compiled code
| | | | | | | | | | | | | | | | | | | | | | | | | |/   6984705: JSR 292 method handle creation should not go through JNI
| | | | | | | | | | | | | | | | | | | | | | | | | |    Summary: remove assembly code for JDK 7 chained method handles
| | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: jrose, twisti, kvn, mhaupt
| | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: John Rose <john.r.rose@oracle.com>, Christian Thalinger <christian.thalinger@oracle.com>, Michael Haupt <michael.haupt@oracle.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2721 : 7089790: integrate bsd-port changes
| | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: kvn, twisti, jrose
| | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Kurt Miller <kurt@intricatesoftware.com>, Greg Lewis <glewis@eyesbeyond.com>, Jung-uk Kim <jkim@freebsd.org>, Christos Zoulas <christos@zoulas.com>, Landon Fuller <landonf@plausible.coop>, The FreeBSD Foundation <board@freebsdfoundation.org>, Michael Franz <mvfranz@gmail.com>, Roger Hoover <rhoover@apple.com>, Alexander Strange <astrange@apple.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2327 : 7035870: JSR 292: Zero support
| | | | | | | | | | | | | | | | | | | | | | | |/   Summary: This adds support for JSR 292 to Zero.
| | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti
| | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---+---o  rev 2294 : 7032458: Zero and Shark fixes
| | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: twisti
| | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2244 : Merge
| | | | | | | | | | | | | | | | | | | | | |/
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2242 : 7030207: Zero tweak to remove accidentally incorporated code
| | | | | | | | | | | | | | | | | | | | |    Summary: IcedTea contains a now-unmaintained ARM-specific interpreter and part of that interpreter was accidentally incorporated in one of the webrevs when Zero was initially imported.
| | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti
| | | | | | | | | | | | | | | | | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2223 : 7017732: move static fields into Class to prepare for perm gen removal
| | | | | | | | | | | | | | | | | | | |/   Reviewed-by: kvn, coleenp, twisti, stefank
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2132 : Merge
| | | | | | | | | | | | | | | | | | |/|
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2128 : 7018673: Zero: 6953144, 6990754 and 7009756 made some changes which broke Zero
| | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn, chrisphi, twisti
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2073 : 7016023: Enable building ARM and PPC from src/closed repository
| | | | | | | | | | | | | | | | | |/   Reviewed-by: dholmes, bdelsart
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2027 : 4926272: methodOopDesc::method_from_bcp is unsafe
| | | | | | | | | | | | | | | | |/   Reviewed-by: coleenp, jrose, kvn, dcubed
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 1889 : 7004217: Remove IA64 workaround re-introduced with CR6953477
| | | | | | | | | | | | | | | |/   Summary: gcc bug worksaround for IA64 no longer needed
| | | | | | | | | | | | | | | |    Reviewed-by: andrew
+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 1879 : 6989984: Use standard include model for Hospot
| | | | | | | | | | | | | | |/   Summary: Replaced MakeDeps and the includeDB files with more standardized solutions.
| | | | | | | | | | | | | | |    Reviewed-by: coleenp, kvn, kamg
+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 1649 : 6977640: Zero and Shark fixes
| | | | | | | | | | | | | |/   Summary: A number of fixes for Zero and Shark.
| | | | | | | | | | | | | |    Reviewed-by: twisti
| | | | | | | | | | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+---o  rev 1601 : 6953477: Increase portability and flexibility of building Hotspot
| | | | | | | | | | | | |/   Summary: A collection of portability improvements including shared code support for PPC, ARM platforms, software floating point, cross compilation support and improvements in error crash detail.
| | | | | | | | | | | | |    Reviewed-by: phh, never, coleenp, dholmes
+-+-+-+-+-+-+-+-+-+-+---o  rev 1472 : 6941466: Oracle rebranding changes for Hotspot repositories
| | | | | | | | | | | |/   Summary: Change all the Sun copyrights to Oracle copyright
| | | | | | | | | | | |    Reviewed-by: ohair
+-+-+-+-+-+-+-+-+-+---o  rev 1433 : Merge
| | | | | | | | | | |/|
| | | | | | | | | | | o  rev 1429 : 6949423: remove tagged stack interpreter for Zero
| | | | | | | | | | | |  Summary: Missed Zero changes for 6943304.
| | | | | | | | | | | |  Reviewed-by: twisti
| | | | | | | | | | | |  Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+-+-+-+-+---o  rev 1426 : 6943304: remove tagged stack interpreter
| | | | | | | | | | |    Reviewed-by: coleenp, never, gbenson
+-+-+-+-+-+-+-+-+---o  rev 1410 : 6888954: argument formatting for assert() and friends
| | | | | | | | | |/   Reviewed-by: kvn, twisti, apetrusenko, never, dcubed
+-+-+-+-+-+-+-+---o  rev 1347 : 6939845: zero needs fallback path in C++ interpreter for platform dependent fast bytecodes
| | | | | | | | |/   Reviewed-by: never
| | | | | | | | |    Contributed-by: ed@camswl.com
+-+-+-+-+-+-+---o  rev 1078 : 6896043: first round of zero fixes
| | | | | | | |/   Reviewed-by: kvn
| | | | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+---o  rev 1010 : 6890308: integrate zero assembler hotspot changes
| | | | | | |/   Reviewed-by: never
| | | | | | |    Contributed-by: gbenson@redhat.com
+-+-+-+-+---o  rev 628 : Merge
| | | | | |/|
+-+-+-+---o |  rev 605 : 6810672: Comment typos
| | | | |/ /   Summary: I have collected some typos I have found while looking at the code.
| | | | | |    Reviewed-by: kvn, never
+-+-+-+---o  rev 579 : 6814575: Update copyright year
| | | | |/   Summary: Update copyright for files that have been modified in 2009, up to 03/09
| | | | |    Reviewed-by: katleman, tbell, ohair
+-+-+---o  rev 520 : 6791168: Fix invalid code in bytecodeInterpreter that can cause gcc ICE
| | | |/   Summary: Fix compilation errors from latest gcc in CC_INTERP including offending missing void* cast.
| | | |    Reviewed-by: xlu
+-+---o  rev 196 : 6719955: Update copyright year
| | |/   Summary: Update copyright year for files that have been modified in 2008
| | |    Reviewed-by: ohair, tbell
+---o  rev 167 : 6696264: assert("narrow oop can never be zero") for GCBasher & ParNewGC
| |/   Summary: decouple set_klass() with zeroing the gap when compressed.
| |    Reviewed-by: kvn, ysr, jrose
| o  rev 123 : 6688137: c++ interpreter fails on 64bit sparc
|/   Summary: Misc. 64bit and endian fixes for sparc
|    Reviewed-by: never, kvn, rasbold
|    Contributed-by: volker.simonis@gmail.com
o  rev 0 : Initial load
1 line changed: 0 ins; 0 del; 1 mod; 3427 unchg

This code review page was prepared using ../../webrev.ksh (vers 25.6-hg+openjdk.java.net).