Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8277825: Remove unused ReferenceProcessorPhaseTimes::_sub_phases_total_time_ms #6557

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
@@ -200,7 +200,6 @@ void ReferenceProcessorPhaseTimes::set_phase_time_ms(ReferenceProcessor::RefProc
void ReferenceProcessorPhaseTimes::reset() {
for (int i = 0; i < ReferenceProcessor::RefSubPhaseMax; i++) {
_sub_phases_worker_time_sec[i]->reset();
_sub_phases_total_time_ms[i] = uninitialized();
}

for (int i = 0; i < ReferenceProcessor::RefPhaseMax; i++) {
@@ -227,17 +226,6 @@ ReferenceProcessorPhaseTimes::~ReferenceProcessorPhaseTimes() {
delete _soft_weak_final_refs_phase_worker_time_sec;
}

double ReferenceProcessorPhaseTimes::sub_phase_total_time_ms(ReferenceProcessor::RefProcSubPhases sub_phase) const {
ASSERT_SUB_PHASE(sub_phase);
return _sub_phases_total_time_ms[sub_phase];
}

void ReferenceProcessorPhaseTimes::set_sub_phase_total_phase_time_ms(ReferenceProcessor::RefProcSubPhases sub_phase,
double time_ms) {
ASSERT_SUB_PHASE(sub_phase);
_sub_phases_total_time_ms[sub_phase] = time_ms;
}

void ReferenceProcessorPhaseTimes::add_ref_cleared(ReferenceType ref_type, size_t count) {
ASSERT_REF_TYPE(ref_type);
Atomic::add(&_ref_cleared[ref_type_2_index(ref_type)], count, memory_order_relaxed);
@@ -41,8 +41,6 @@ class ReferenceProcessorPhaseTimes : public CHeapObj<mtGC> {

// Records per thread time information of each sub phase.
WorkerDataArray<double>* _sub_phases_worker_time_sec[ReferenceProcessor::RefSubPhaseMax];
// Total time of each sub phase.
double _sub_phases_total_time_ms[ReferenceProcessor::RefSubPhaseMax];

// Records total elapsed time for each phase.
double _phases_time_ms[ReferenceProcessor::RefPhaseMax];
@@ -62,7 +60,6 @@ class ReferenceProcessorPhaseTimes : public CHeapObj<mtGC> {
GCTimer* _gc_timer;

double phase_time_ms(ReferenceProcessor::RefProcPhases phase) const;
double sub_phase_total_time_ms(ReferenceProcessor::RefProcSubPhases sub_phase) const;

double total_time_ms() const { return _total_time_ms; }

@@ -84,8 +81,6 @@ class ReferenceProcessorPhaseTimes : public CHeapObj<mtGC> {
WorkerDataArray<double>* sub_phase_worker_time_sec(ReferenceProcessor::RefProcSubPhases phase) const;
void set_phase_time_ms(ReferenceProcessor::RefProcPhases phase, double par_phase_time_ms);

void set_sub_phase_total_phase_time_ms(ReferenceProcessor::RefProcSubPhases sub_phase, double ref_proc_time_ms);

void set_total_time_ms(double total_time_ms) { _total_time_ms = total_time_ms; }

void add_ref_cleared(ReferenceType ref_type, size_t count);