Reviewed-by: mgerdin, tschatzl, jmasa

Summary: Add G1BiasedArray<T> that is an array where each element represents a fixed-sized subdivision of the heap. Use this abstraction to refactor the HeapRegionSeq class.
Reviewed-by: brutisso

8025088: Missing cases for JVM_CONSTANT_MethodHandleInError cause crash if debugger steps into error-tagged method handle
Summary: Need to refetch each method from InstanceKlass after all safepoints.  Removed leaky PreviousVersionInfo code.
Reviewed-by: coleenp, sspitsyn

Reviewed-by: sla, dsamersoff
Contributed-by: NYasumasa Suenaga <yasu@ysfactory.dip.jp>

Reviewed-by: coleenp, mgerdin, jmasa

Summary: reference_map[] array had uninitialized junk that was causing a bogus bootstrap method to be found.
Reviewed-by: hseigel, dcubed, sspitsyn

Summary: Need to refetch each method from InstanceKlass after all safepoints.  Removed leaky PreviousVersionInfo code.
Reviewed-by: dcubed, sspitsyn

Reviewed-by: coleenp, brutisso, mgerdin, jmasa

Summary: SymbolTable and StringTable can make calls to GC_locker::is_active() outside a safepoint. This isn't safe because the GC_locker active state (lock count) is only updated at a safepoint and only remains valid as long as _needs_gc is true. However, outside a safepoint_needs_gc can change to false at any time, which makes it impossible to do a correct call to is_active() in that context. In this case these calls can just be removed since the input argument to basic_add() should never be on the heap and so there's no need to check the GC_locker state. This change also adjusts the assert() in is_active() to makes sure all calls to this function are always done under a safepoint.
Reviewed-by: brutisso, dcubed
Contributed-by: per.liden@oracle.com

Summary: Make sure CDS is off by default when running server compiler.
Reviewed-by: dholmes, coleenp

Reviewed-by: alanb, chegar, dholmes, ksrini
Contributed-by: NMike Duigou <mike.duigou@oracle.com>

Summary: Add -XX:+VerifyStringTableAtExit option and code to verify StringTable invariants.
Reviewed-by: rdurbin, sspitsyn, coleenp

Summary: Check that the _java_thread parameter is valid when it is possible that the JavaThread has exited after the initial checks were made in generated/jvmtifiles/jvmtiEnter.cpp: jvmti_GetCurrentContendedMonitor()
Reviewed-by: dcubed, dsamersoff

Summary: Provide correct number of visited nmethods to Tracing
Reviewed-by: kvn, iveresov

Reviewed-by: stefank, dholmes, coleenp

Reviewed-by: sla, coleenp, ctornqvi, dsamersoff

Summary: Incorrect usage of idx_t where bm_word_t is appropriate.
Reviewed-by: tschatzl, brutisso
Contributed-by: NDan Horak <dhorak@redhat.com>

Summary: made changes to easily identify current log file in rotation. Parameterize the input with %t for time replacement in file name.
Reviewed-by: ccheung, tschatzl, tamao, zgu
Contributed-by: yumin.qi@oracle.com

Reviewed-by: kamg, minqi

8024396: VM crashing with assert(!UseLargePages || UseParallelOldGC || use_large_pages) failed: Wrong alignment to use large pages
Summary: Loosen wrong assert for UseParallelOldGC to UseParallelGC
Reviewed-by: stefank, brutisso

Summary: Enhance method resolution and resulting data structures, plus some refactoring.
Reviewed-by: twisti, acorn, jrose

Reviewed-by: kvn, coleenp

8024346: ~CautiouslyPreserveExceptionMark - assert(!_thread->has_pending_exception()) failed: unexpected exception generated
Summary: Pending exceptions must be handled properly after a call to the JVMTI merge_cp_and_rewrite
Reviewed-by: coleenp, dholmes
Contributed-by: serguei.spitsyn@oracle.com

8024345: 'assert(_value != NULL) failed: resolving NULL _value' from VM_RedefineClasses::set_new_constant_pool
Summary: The OOME's in the JVMTI merge_cp_and_rewrite and set_new_constant_pool must be handled correctly
Reviewed-by: coleenp, dholmes
Contributed-by: serguei.spitsyn@oracle.com

8017230: Internal Error (jvmtiRedefineClasses.cpp:1662): guarantee(false) failed: insert_space_at() failed
Summary: Handle pending exceptions instead of firing a guarantee()
Reviewed-by: coleenp, dholmes
Contributed-by: serguei.spitsyn@oracle.com

Reviewed-by: allwin, acorn, dcubed, dholmes, egahlin

Reviewed-by: twisti, kvn, sla

Reviewed-by: brutisso, tschatzl

Reviewed-by: brutisso, mgerdin, coleenp

Reviewed-by: jmasa, brutisso

Reviewed-by: jwilhelm, brutisso, tschatzl

Reviewed-by: brutisso, mgerdin, coleenp, jmasa

8024275: During CTW: assert(sig_bt[member_arg_pos] == T_OBJECT) failed: dispatch argument must be an object
Reviewed-by: kvn, vlivanov

Summary: We already have GrowableArray, use it instead of LRG_List
Reviewed-by: kvn

Reviewed-by: brutisso, ehelin

Reviewed-by: stefank, hseigel, mgerdin

Summary: Minor cleanup of static agent code from 8014135
Reviewed-by: dcubed, sspitsyn

Summary: Use conservative assumptions of required alignment for the various garbage collector components into account when determining the maximum heap size that supports compressed oops. Using this conservative value avoids several circular dependencies in the calculation.
Reviewed-by: stefank, dholmes

Summary: Correctly calculate the initialization value for the shift between object start and bitmap bit in the G1 mark bitmaps.
Reviewed-by: tonyp

Summary: Use best-fit block-splitting freelist allocation from the block freelist.
Reviewed-by: jmasa, stefank