Code Review for hotspot

Prepared by:sgehwolf on Mon Nov 23 11:31:00 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:4da64841abd7
Summary of changes: 622 lines changed: 525 ins; 52 del; 45 mod; 4338 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/cppInterpreter_zero.cpp

@    rev 9408 : 8008243: Zero: implement fast-bytecodes
|\
| \
| |\
| | \
| | |\
| | | \
| | | |\
| | | | \
| | | | |\
| | | | | \
| | | | | |\
| | | | | | \
| | | | | | |\
| | | | | | | \
| | | | | | | |\
| | | | | | | | \
| | | | | | | | |\
| | | | | | | | | \
| | | | | | | | | |\
| | | | | | | | | | \
| | | | | | | | | | |\
| | | | | | | | | | | \
| | | | | | | | | | | |\
| | | | | | | | | | | | \
| | | | | | | | | | | | |\
| | | | | | | | | | | | | \
| | | | | | | | | | | | | |\
| | | | | | | | | | | | | | \
| | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 9046 : 8136525: Generate interpreter entries only once and avoid unnecessary jump to jump
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: coleenp, twisti, aph
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 8082 : 8076421: Fix Zero Interpreter bugs in class redefinition and template interpreter changes
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: metadata_do walking interpreted frames was wrong and generate_Reference_get is not necessarily an accessor method.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: sgehwolf, dholmes
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 7032 : 8057696: java -version triggers assertion for slowdebug zero builds
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: The change as introduced with JDK-8003426 removed some zero code in cppInterpreter_zero.cpp which called generate_normal_entry() in that same file with a value of synchronized == false unconditionally. However, since the new code calls generate_normal_entry() with either true or false, the assertion in this method is no longer valid.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: coleenp
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Contributed-by: sgehwolf@redhat.com
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---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 6860 : 8003426: Remove UseFastAccessors and UseFastEmptyMethods except for zero
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: These options have been long disabled in Xmixed mode because they prevent these small methods from being inlined and are subject to bit rot, and we don't need more macro assembler code to maintain and change if the constant pool cache format changes.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: simonis, kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---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 6286 : 8032410: compiler/uncommontrap/TestStackBangRbp.java times out on Solaris-Sparc V9
| | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: make compiled code bang the stack by the worst case size of the interpreter frame at deoptimization points.
| | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti, kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5899 : Merge
| | | | | | | | | | | | | | | | | | | | | | | | | | |/|
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5865 : Merge
| | | | | | | | | | | | | | | | | | | | | | | | | | | |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-----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 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 5604 : 8026328: Setting a breakpoint on invokedynamic crashes the JVM
| | | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: jrose, roland
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 5110 : 8022956: Clang: enable return type warnings on BSD
| | | | | | | | | | | | | | | | | | | | | | | |/   Reviewed-by: coleenp, sla
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4630 : 8013067: Zero builds are broken after 8010862.
| | | | | | | | | | | | | | | | | | | | | | |/   Summary: Fixed broken Zero build.
| | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: twisti, coleenp, kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4292 : 8009761: Deoptimization on sparc doesn't set Llast_SP correctly in the interpreter frames it creates
| | | | | | | | | | | | | | | | | | | | | |/   Summary: deoptimization doesn't set up callee frames so that they restore caller frames correctly.
| | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: kvn
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 4107 : 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
| | | | | | | | | | | | | | | | | | | | |/   Summary: Rename INCLUDE_ALTERNATE_GCS to INCLUDE_ALL_GCS and replace SERIALGC with INCLUDE_ALL_GCS.
| | | | | | | | | | | | | | | | | | | | |    Reviewed-by: coleenp, stefank
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 3879 : 8003868: fix shark for latest HotSpot and LLVM
| | | | | | | | | | | | | | | | | | | |/   Reviewed-by: twisti
| | | | | | | | | | | | | | | | | | | |    Contributed-by: Roman Kennke <rkennke@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---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 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 3339 : 7167406: (Zero) Fix for InvokeDynamic needed
| | | | | | | | | | | | | | | |/   Reviewed-by: chrisphi, dholmes
| | | | | | | | | | | | | | | |    Contributed-by: Andrew Dinn <adinn@redhat.com>
+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2543 : 7055355: JSR 292: crash while throwing WrongMethodTypeException
| | | | | | | | | | | | | | |/   Reviewed-by: jrose, twisti, bdelsart
+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2466 : 7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
| | | | | | | | | | | | | |/   Reviewed-by: kvn, coleenp
+-+-+-+-+-+-+-+-+-+-+-+---o  rev 2358 : Merge
| | | | | | | | | | | | |/
+-+-+-+-+-+-+-+-+-+-+---o  rev 2346 : 7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error
| | | | | | | | | | | |    Summary: A referent object that is only weakly reachable at the start of concurrent marking but is re-attached to the strongly reachable object graph during marking may not be marked as live. This can cause the reference object to be processed prematurely and leave dangling pointers to the referent object. Implement a read barrier for the java.lang.ref.Reference::referent field by intrinsifying the Reference.get() method, and intercepting accesses though JNI, reflection, and Unsafe, so that when a non-null referent object is read it is also logged in an SATB buffer.
| | | | | | | | | | | |    Reviewed-by: kvn, iveresov, never, tonyp, dholmes
+-+-+-+-+-+-+-+-+-+---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 2223 : 7017732: move static fields into Class to prepare for perm gen removal
| | | | | | | | | |/   Reviewed-by: kvn, coleenp, twisti, stefank
+-+-+-+-+-+-+-+---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 1525 : 6960550: Missing semicolon in Zero
| | | | | | | |/   Summary: There is a missing semicolon in cppInterpreter_zero.cpp.
| | | | | | | |    Reviewed-by: twisti
| | | | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+-+---o  rev 1472 : 6941466: Oracle rebranding changes for Hotspot repositories
| | | | | | |/   Summary: Change all the Sun copyrights to Oracle copyright
| | | | | | |    Reviewed-by: ohair
+-+-+-+-+---o  rev 1434 : 6951784: Zero deoptimizer changes
| | | | | |/|  Summary: The way Zero currently handles deoptimization can lead to methods being freed while they are still being executed.
| | | | | | |  Reviewed-by: twisti
| | | | | | |  Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+-+---o  rev 1425 : 6939182: Zero JNI handles fix
| | | | | |/   Summary: Zero will exit with an error when invoked with -Xcheck:jni.
| | | | | |    Reviewed-by: twisti, kamg
| | | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+-+---o  rev 1379 : 6941224: Improved stack overflow handling for Zero
| | | | |/   Summary: Adding stack overflow checking to Shark brought to light a bunch of deficiencies in Zero's stack overflow code.
| | | | |    Reviewed-by: twisti
| | | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+-+---o  rev 1345 : 6939180: Zero locking fix
| | | |/   Summary: When Zero is running with Shark enabled threads can be left with their _do_not_unlock_if_synchronized flag incorrectly set.
| | | |    Reviewed-by: twisti
| | | |    Contributed-by: Gary Benson <gbenson@redhat.com>
+-+---o  rev 1256 : 6926048: Improve Zero performance
| | |/   Summary: Make Zero figure out result types in a similar way to C++ interpreter implementation.
| | |    Reviewed-by: kvn
| | |    Contributed-by: gbenson@redhat.com
+---o  rev 1139 : 6908267: Zero fails to unlock synchronized native methods on exception
| |/   Reviewed-by: never
| |    Contributed-by: Gary Benson <gbenson@redhat.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
2 lines changed: 0 ins; 0 del; 2 mod; 1005 unchg

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

@    rev 9408 : 8008243: Zero: implement fast-bytecodes
|\
| \
| |\
| | \
| | |\
| | | \
| | | |\
| | | | \
| | | | |\
| | | | | \
| | | | | |\
| | | | | | \
| | | | | | |\
| | | | | | | \
| | | | | | | |\
| | | | | | | | \
| | | | | | | | |\
| | | | | | | | | \
| | | | | | | | | |\
| | | | | | | | | | \
| | | | | | | | | | |\
| | | | | | | | | | | \
| | | | | | | | | | | |\
| | | | | | | | | | | | \
| | | | | | | | | | | | |\
| | | | | | | | | | | | | \
| | | | | | | | | | | | | |\
| | | | | | | | | | | | | | \
| | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | \
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |\
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+---o  rev 9254 : 8141138: Zero fails to build
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |/   Summary: Adjust to recent JDK-9 changes.
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | |    Reviewed-by: coleenp
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
620 lines changed: 525 ins; 52 del; 43 mod; 3333 unchg

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