stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8333486: Parallel: Remove unused methods in psParallelCompact

Browse Source 
Reviewed-by: tschatzl
This commit is contained in:
Albert Mingkun Yang 2024-06-04 16:39:03 +00:00
parent 664c993c41
commit 4a1cdd5ba9
2 changed files with 10 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/hotspot/share/gc/parallel/psParallelCompact.cpp
View File

@ -280,21 +280,6 @@ PSParallelCompact::print_generic_summary_data(ParallelCompactData& summary_data,
::print_generic_summary_data(summary_data,beg_addr, end_addr); ::print_generic_summary_data(summary_data,beg_addr, end_addr);
} }
void
print_generic_summary_data(ParallelCompactData& summary_data,
SpaceInfo* space_info)
{
if (!log_develop_is_enabled(Trace, gc, compaction)) {
return;
}
for (unsigned int id = 0; id < PSParallelCompact::last_space_id; ++id) {
const MutableSpace* space = space_info[id].space();
print_generic_summary_data(summary_data, space->bottom(),
MAX2(space->top(), space_info[id].new_top()));
}
}
static void static void
print_initial_summary_data(ParallelCompactData& summary_data, print_initial_summary_data(ParallelCompactData& summary_data,
const MutableSpace* space) { const MutableSpace* space) {

22
src/hotspot/share/gc/parallel/psParallelCompact.hpp
View File

@ -166,7 +166,7 @@ inline bool SplitInfo::is_split(size_t region_idx) const
class SpaceInfo class SpaceInfo
{ {
public: public:
MutableSpace* space() const { return _space; } MutableSpace* space() const { return _space; }
// Where the free space will start after the collection. Valid only after the // Where the free space will start after the collection. Valid only after the
@ -190,9 +190,7 @@ class SpaceInfo
void set_dense_prefix(HeapWord* addr) { _dense_prefix = addr; } void set_dense_prefix(HeapWord* addr) { _dense_prefix = addr; }
void set_start_array(ObjectStartArray* s) { _start_array = s; } void set_start_array(ObjectStartArray* s) { _start_array = s; }
void publish_new_top() const { _space->set_top(_new_top); } private:
private:
MutableSpace* _space; MutableSpace* _space;
HeapWord* _new_top; HeapWord* _new_top;
HeapWord* _dense_prefix; HeapWord* _dense_prefix;
@ -681,7 +679,7 @@ ParallelCompactData::is_region_aligned(HeapWord* addr) const
// https://doi.org/10.1145/3313808.3313820 // https://doi.org/10.1145/3313808.3313820
class PSParallelCompact : AllStatic { class PSParallelCompact : AllStatic {
public: public:
// Convenient access to type names. // Convenient access to type names.
typedef ParMarkBitMap::idx_t idx_t; typedef ParMarkBitMap::idx_t idx_t;
typedef ParallelCompactData::RegionData RegionData; typedef ParallelCompactData::RegionData RegionData;
@ -701,7 +699,7 @@ public:
friend class PSParallelCompactTest; friend class PSParallelCompactTest;
private: private:
static STWGCTimer _gc_timer; static STWGCTimer _gc_timer;
static ParallelOldTracer _gc_tracer; static ParallelOldTracer _gc_tracer;
static elapsedTimer _accumulated_time; static elapsedTimer _accumulated_time;
@ -716,10 +714,10 @@ public:
static SpanSubjectToDiscoveryClosure _span_based_discoverer; static SpanSubjectToDiscoveryClosure _span_based_discoverer;
static ReferenceProcessor* _ref_processor; static ReferenceProcessor* _ref_processor;
public: public:
static ParallelOldTracer* gc_tracer() { return &_gc_tracer; } static ParallelOldTracer* gc_tracer() { return &_gc_tracer; }
private: private:
static void initialize_space_info(); static void initialize_space_info();
@ -768,7 +766,7 @@ public:
static void fill_range_in_dense_prefix(HeapWord* start, HeapWord* end); static void fill_range_in_dense_prefix(HeapWord* start, HeapWord* end);
public: public:
static void fill_dead_objs_in_dense_prefix(uint worker_id, uint num_workers); static void fill_dead_objs_in_dense_prefix(uint worker_id, uint num_workers);
static bool invoke(bool maximum_heap_compaction); static bool invoke(bool maximum_heap_compaction);
@ -885,12 +883,12 @@ public:
}; };
class MoveAndUpdateClosure: public StackObj { class MoveAndUpdateClosure: public StackObj {
private: private:
ParMarkBitMap* const _bitmap; ParMarkBitMap* const _bitmap;
size_t _words_remaining; // Words left to copy. size_t _words_remaining; // Words left to copy.
static inline size_t calculate_words_remaining(size_t region); static inline size_t calculate_words_remaining(size_t region);
protected: protected:
HeapWord* _source; // Next addr that would be read. HeapWord* _source; // Next addr that would be read.
HeapWord* _destination; // Next addr to be written. HeapWord* _destination; // Next addr to be written.
ObjectStartArray* const _start_array; ObjectStartArray* const _start_array;
@ -900,7 +898,7 @@ class MoveAndUpdateClosure: public StackObj {
// Update variables to indicate that word_count words were processed. // Update variables to indicate that word_count words were processed.
inline void update_state(size_t words); inline void update_state(size_t words);
public: public:
typedef ParMarkBitMap::idx_t idx_t; typedef ParMarkBitMap::idx_t idx_t;
ParMarkBitMap* bitmap() const { return _bitmap; } ParMarkBitMap* bitmap() const { return _bitmap; }