archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-26 06:14:49 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8233065: PSParallelCompact::move_and_update is unused and should be removed

Browse source 
Reviewed-by: lkorinth, tschatzl
This commit is contained in:
Stefan Johansson 2019-11-11 13:20:26 +01:00
parent b2a9673829
commit 5e113f37e8
2 changed files with 0 additions and 43 deletions
Download patch file Download diff file Expand all files Collapse all files

40
src/hotspot/share/gc/parallel/psParallelCompact.cpp
View file

@ -3201,46 +3201,6 @@ void PSParallelCompact::fill_blocks(size_t region_idx)
}
}
void
PSParallelCompact::move_and_update(ParCompactionManager* cm, SpaceId space_id) {
const MutableSpace* sp = space(space_id);
if (sp->is_empty()) {
return;
}
ParallelCompactData& sd = PSParallelCompact::summary_data();
ParMarkBitMap* const bitmap = mark_bitmap();
HeapWord* const dp_addr = dense_prefix(space_id);
HeapWord* beg_addr = sp->bottom();
HeapWord* end_addr = sp->top();
assert(beg_addr <= dp_addr && dp_addr <= end_addr, "bad dense prefix");
const size_t beg_region = sd.addr_to_region_idx(beg_addr);
const size_t dp_region = sd.addr_to_region_idx(dp_addr);
if (beg_region < dp_region) {
update_and_deadwood_in_dense_prefix(cm, space_id, beg_region, dp_region);
}
// The destination of the first live object that starts in the region is one
// past the end of the partial object entering the region (if any).
HeapWord* const dest_addr = sd.partial_obj_end(dp_region);
HeapWord* const new_top = _space_info[space_id].new_top();
assert(new_top >= dest_addr, "bad new_top value");
const size_t words = pointer_delta(new_top, dest_addr);
if (words > 0) {
ObjectStartArray* start_array = _space_info[space_id].start_array();
MoveAndUpdateClosure closure(bitmap, cm, start_array, dest_addr, words);
ParMarkBitMap::IterationStatus status;
status = bitmap->iterate(&closure, dest_addr, end_addr);
assert(status == ParMarkBitMap::full, "iteration not complete");
assert(bitmap->find_obj_beg(closure.source(), end_addr) == end_addr,
"live objects skipped because closure is full");
}
}
jlong PSParallelCompact::millis_since_last_gc() {
// We need a monotonically non-decreasing time in ms but
// os::javaTimeMillis() does not guarantee monotonicity.