Skip to content
Permalink
Browse files
8277824: Remove empty RefProcSubPhasesWorkerTimeTracker destructor
Co-authored-by: Kim Barrett <kbarrett@openjdk.org>
Reviewed-by: tschatzl, kbarrett
  • Loading branch information
albertnetymk and Kim Barrett committed Nov 30, 2021
1 parent 27299ea commit fde6fe7997a889b17af89b3a8fd25456568a1fa6
Showing 2 changed files with 5 additions and 8 deletions.
@@ -101,10 +101,7 @@ RefProcWorkerTimeTracker::~RefProcWorkerTimeTracker() {
RefProcSubPhasesWorkerTimeTracker::RefProcSubPhasesWorkerTimeTracker(ReferenceProcessor::RefProcSubPhases phase,
ReferenceProcessorPhaseTimes* phase_times,
uint worker_id) :
RefProcWorkerTimeTracker(phase_times->sub_phase_worker_time_sec(phase), worker_id) {
}

RefProcSubPhasesWorkerTimeTracker::~RefProcSubPhasesWorkerTimeTracker() {
_tracker(phase_times->sub_phase_worker_time_sec(phase), worker_id) {
}

RefProcPhaseTimeBaseTracker::RefProcPhaseTimeBaseTracker(const char* title,
@@ -98,23 +98,23 @@ class ReferenceProcessorPhaseTimes : public CHeapObj<mtGC> {
void print_all_references(uint base_indent = 0, bool print_total = true) const;
};

class RefProcWorkerTimeTracker : public CHeapObj<mtGC> {
class RefProcWorkerTimeTracker : public StackObj {
protected:
WorkerDataArray<double>* _worker_time;
double _start_time;
uint _worker_id;
public:
RefProcWorkerTimeTracker(WorkerDataArray<double>* worker_time, uint worker_id);
virtual ~RefProcWorkerTimeTracker();
~RefProcWorkerTimeTracker();
};

// Updates working time of each worker thread for a given sub phase.
class RefProcSubPhasesWorkerTimeTracker : public RefProcWorkerTimeTracker {
class RefProcSubPhasesWorkerTimeTracker : public StackObj {
RefProcWorkerTimeTracker _tracker;
public:
RefProcSubPhasesWorkerTimeTracker(ReferenceProcessor::RefProcSubPhases phase,
ReferenceProcessorPhaseTimes* phase_times,
uint worker_id);
~RefProcSubPhasesWorkerTimeTracker();
};

class RefProcPhaseTimeBaseTracker : public StackObj {

1 comment on commit fde6fe7

@openjdk-notifier
Copy link

@openjdk-notifier openjdk-notifier bot commented on fde6fe7 Nov 30, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.