Reviewed-by: tschatzl, ehelin, brutisso, coleenp, roland, iveresov
Contributed-by: stefan.karlsson@oracle.com, mikael.gerdin@oracle.com

Summary: Change parent of G1OffsetTableContigSpace to CompactibleSpace, reimplement missing functionality
Reviewed-by: stefank, jmasa, tschatzl

Reviewed-by: stefank, brutisso, tschatzl

Summary: Rename save_marks to record_top_and_timestamp and remove som unused but related methods
Reviewed-by: stefank, ehelin

Reviewed-by: tschatzl, stefank

Reviewed-by: stefank, tschatzl

Reviewed-by: mgerdin, tschatzl

Reviewed-by: ehelin, mgerdin

Summary: Get rid of some duplicated implementations and change nonsense implementations to pure virtuals
Reviewed-by: stefank, jmasa

Summary: Only declare the functions where they are actually needed.
Reviewed-by: tschatzl, stefank

Reviewed-by: brutisso, tschatzl, stefank

Reviewed-by: tschatzl, stefank

Summary: Allocate temporary BitMaps in the VMThread's resource area
Reviewed-by: stefank, sjohanss

Reviewed-by: tschatzl, coleenp

Reviewed-by: tschatzl, brutisso

Reviewed-by: tschatzl, mgerdin, jmasa

Reviewed-by: mgerdin, brutisso

Reviewed-by: mgerdin, brutisso, tschatzl

Summary: Refactor the strong root processing to avoid using a boolean in addition to the ScanOption enum.
Reviewed-by: stefank, tschatzl, ehelin, jmasa

Summary: When G1 is run with -XX:-G1DeferredRSUpdate, the VM crashes because of wrong initialization order of member variables. The change makes the initalization explicit, not relying on initialization order any more.
Reviewed-by: brutisso, mgerdin

Summary: After JDK-8035400 there were several opportunities to fix the visibility of several members of the G1ParScanThreadState class.
Reviewed-by: brutisso, mgerdin

Summary: Extract the G1ParScanThreadState class from G1CollectedHeap.?pp into its own files.
Reviewed-by: brutisso, mgerdin

Summary: Clean up usage of idx_t and uintptr_t when using it in conjunction with BitMap::set_map(), casting to the appropriate type. Fixes compilation on S390.
Reviewed-by: tschatzl
Contributed-by: NDan Horak <dhorak@redhat.com>

Summary: Changed faulty error message in the verification code
Reviewed-by: stefank, tschatzl
Contributed-by: NAndreas Sjoberg <andreas.sjoberg@oracle.com>

Summary: A few locations in the code use sizeof(this) which returns the size of the pointer instead of sizeof(classname) which returns the size of the sum of its members. This change fixes these errors and adds a few tests.
Reviewed-by: mgerdin, brutisso

Summary: Card re-dirtying code for verification and actual redirtying uses two different, almost completely identical card closures. Also the verification code still assumes a perm gen.
Reviewed-by: brutisso, jmasa

Summary: Parallelize card redirtying to decrease the time it takes.
Reviewed-by: brutisso

Summary: G1 does not retire allocation buffers after reference processing work when -XX:+ParallelRefProcEnabled is enabled. This causes wrong calculation of PLAB sizes, as the amount of space wasted is not updated correctly.
Reviewed-by: brutisso

Summary: The mentioned method is never used and out of date. So it is removed.
Reviewed-by: mgerdin, brutisso

Summary: After changes to the PerRegionTable where all these PRTs are linked together in an additional field, simplify iterating over all PRTs by using these links instead of walki
Reviewed-by: mgerdin, jwilhelm, brutisso

Summary: Instead of using a manually managed array for the in_cset_fast_test array, use a G1BiasedArray instance.
Reviewed-by: brutisso, mgerdin

Reviewed-by: brutisso, sjohanss

Summary: Removed all write barriers during reference processing and added explicit write barriers when iterating through the discovered list.
Reviewed-by: pliden, jmasa, tschatzl

Reviewed-by: tschatzl, mgerdin

Reviewed-by: brutisso, tschatzl

Reviewed-by: brutisso, ehelin

Reviewed-by: twisti, stefank

8043029: Change 8037816 breaks HS build with older GCC versions which don't support diagnostic pragmas
8043164: Format warning in traceStream.hpp
Summary: Backport of main fix + two corrections, enables clang compilation, turns on format attributes, corrects/mutes warnings
Reviewed-by: kvn, coleenp, iveresov, twisti

Summary: Only prune metaspace virtual spaces at safepoint so walking them is safe outside a safepoint.
Reviewed-by: mgerdin, mgronlun, hseigel, stefank

Reviewed-by: dholmes, kvn, stefank, twisti