--- old/src/share/vm/gc/g1/g1ConcurrentMark.inline.hpp 2017-07-20 16:25:59.178428932 +0200 +++ new/src/share/vm/gc/g1/g1ConcurrentMark.inline.hpp 2017-07-20 16:25:59.081425923 +0200 @@ -27,6 +27,7 @@ #include "gc/g1/g1CollectedHeap.inline.hpp" #include "gc/g1/g1ConcurrentMark.hpp" +#include "gc/g1/g1ConcurrentMarkBitMap.inline.hpp" #include "gc/g1/g1ConcurrentMarkObjArrayProcessor.inline.hpp" #include "gc/g1/suspendibleThreadSet.hpp" #include "gc/shared/taskqueue.inline.hpp" @@ -36,57 +37,6 @@ return _nextMarkBitMap->par_mark((HeapWord*)obj); } -inline bool G1CMBitMap::iterate(G1CMBitMapClosure* cl, MemRegion mr) { - assert(!mr.is_empty(), "Does not support empty memregion to iterate over"); - assert(_covered.contains(mr), "Given MemRegion from " PTR_FORMAT " to " PTR_FORMAT " not contained in heap area", p2i(mr.start()), p2i(mr.end())); - - BitMap::idx_t const end_offset = addr_to_offset(mr.end()); - BitMap::idx_t offset = _bm.get_next_one_offset(addr_to_offset(mr.start()), end_offset); - - while (offset < end_offset) { - HeapWord* const addr = offset_to_addr(offset); - if (!cl->do_addr(addr)) { - return false; - } - size_t const obj_size = (size_t)((oop)addr)->size(); - offset = _bm.get_next_one_offset(offset + (obj_size >> _shifter), end_offset); - } - return true; -} - -inline HeapWord* G1CMBitMap::get_next_marked_addr(const HeapWord* addr, - const HeapWord* limit) const { - assert(limit != NULL, "limit must not be NULL"); - // Round addr up to a possible object boundary to be safe. - size_t const addr_offset = addr_to_offset(align_up(addr, HeapWordSize << _shifter)); - size_t const limit_offset = addr_to_offset(limit); - size_t const nextOffset = _bm.get_next_one_offset(addr_offset, limit_offset); - return offset_to_addr(nextOffset); -} - -#ifdef ASSERT -inline void G1CMBitMap::check_mark(HeapWord* addr) { - assert(G1CollectedHeap::heap()->is_in_exact(addr), - "Trying to access bitmap " PTR_FORMAT " for address " PTR_FORMAT " not in the heap.", - p2i(this), p2i(addr)); -} -#endif - -inline void G1CMBitMap::mark(HeapWord* addr) { - check_mark(addr); - _bm.set_bit(addr_to_offset(addr)); -} - -inline void G1CMBitMap::clear(HeapWord* addr) { - check_mark(addr); - _bm.clear_bit(addr_to_offset(addr)); -} - -inline bool G1CMBitMap::par_mark(HeapWord* addr) { - check_mark(addr); - return _bm.par_set_bit(addr_to_offset(addr)); -} - #ifndef PRODUCT template inline void G1CMMarkStack::iterate(Fn fn) const {