8333954: Parallel: Remove unused arguments of type ParCompactionManager*

Reviewed-by: iwalulya
This commit is contained in:
Albert Mingkun Yang 2024-06-11 08:37:47 +00:00
parent 788b876ebd
commit 93f3918ee1
2 changed files with 9 additions and 17 deletions
src/hotspot/share/gc/parallel

@ -2228,7 +2228,7 @@ void PSParallelCompact::fill_region(ParCompactionManager* cm, MoveAndUpdateClosu
if (closure.is_full()) {
decrement_destination_counts(cm, src_space_id, src_region_idx,
closure.source());
closure.complete_region(cm, dest_addr, region_ptr);
closure.complete_region(dest_addr, region_ptr);
return;
}
@ -2271,7 +2271,7 @@ void PSParallelCompact::fill_region(ParCompactionManager* cm, MoveAndUpdateClosu
if (closure.is_full()) {
decrement_destination_counts(cm, src_space_id, src_region_idx,
closure.source());
closure.complete_region(cm, dest_addr, region_ptr);
closure.complete_region(dest_addr, region_ptr);
return;
}
@ -2304,7 +2304,7 @@ void PSParallelCompact::fill_and_update_shadow_region(ParCompactionManager* cm,
region_ptr->shadow_to_normal();
return fill_region(cm, cl, region_idx);
} else {
MoveAndUpdateShadowClosure cl(mark_bitmap(), cm, region_idx, shadow_region);
MoveAndUpdateShadowClosure cl(mark_bitmap(), region_idx, shadow_region);
return fill_region(cm, cl, region_idx);
}
}
@ -2381,8 +2381,7 @@ void MoveAndUpdateClosure::copy_partial_obj(size_t partial_obj_size)
update_state(words);
}
void MoveAndUpdateClosure::complete_region(ParCompactionManager *cm, HeapWord *dest_addr,
PSParallelCompact::RegionData *region_ptr) {
void MoveAndUpdateClosure::complete_region(HeapWord* dest_addr, PSParallelCompact::RegionData* region_ptr) {
assert(region_ptr->shadow_state() == ParallelCompactData::RegionData::NormalRegion, "Region should be finished");
region_ptr->set_completed();
}
@ -2412,8 +2411,7 @@ void MoveAndUpdateClosure::do_addr(HeapWord* addr, size_t words) {
update_state(words);
}
void MoveAndUpdateShadowClosure::complete_region(ParCompactionManager *cm, HeapWord *dest_addr,
PSParallelCompact::RegionData *region_ptr) {
void MoveAndUpdateShadowClosure::complete_region(HeapWord* dest_addr, PSParallelCompact::RegionData* region_ptr) {
assert(region_ptr->shadow_state() == ParallelCompactData::RegionData::ShadowRegion, "Region should be shadow");
// Record the shadow region index
region_ptr->set_shadow_region(_shadow);

@ -895,8 +895,7 @@ public:
// updated.
void copy_partial_obj(size_t partial_obj_size);
virtual void complete_region(ParCompactionManager* cm, HeapWord* dest_addr,
PSParallelCompact::RegionData* region_ptr);
virtual void complete_region(HeapWord* dest_addr, PSParallelCompact::RegionData* region_ptr);
};
inline void MoveAndUpdateClosure::decrement_words_remaining(size_t words) {
@ -932,11 +931,9 @@ inline void MoveAndUpdateClosure::update_state(size_t words)
class MoveAndUpdateShadowClosure: public MoveAndUpdateClosure {
inline size_t calculate_shadow_offset(size_t region_idx, size_t shadow_idx);
public:
inline MoveAndUpdateShadowClosure(ParMarkBitMap* bitmap, ParCompactionManager* cm,
size_t region, size_t shadow);
inline MoveAndUpdateShadowClosure(ParMarkBitMap* bitmap, size_t region, size_t shadow);
virtual void complete_region(ParCompactionManager* cm, HeapWord* dest_addr,
PSParallelCompact::RegionData* region_ptr);
virtual void complete_region(HeapWord* dest_addr, PSParallelCompact::RegionData* region_ptr);
private:
size_t _shadow;
@ -950,10 +947,7 @@ inline size_t MoveAndUpdateShadowClosure::calculate_shadow_offset(size_t region_
}
inline
MoveAndUpdateShadowClosure::MoveAndUpdateShadowClosure(ParMarkBitMap *bitmap,
ParCompactionManager *cm,
size_t region,
size_t shadow) :
MoveAndUpdateShadowClosure::MoveAndUpdateShadowClosure(ParMarkBitMap* bitmap, size_t region, size_t shadow) :
MoveAndUpdateClosure(bitmap, region),
_shadow(shadow) {
_offset = calculate_shadow_offset(region, shadow);