Skip to content

Commit

Permalink
8341992: GenShen: Fix formatting, remove unreachable code, unused imp…
Browse files Browse the repository at this point in the history
…orts and unnecessary comments

Backport-of: bb7938446b3b6d6729df627cd580957f4a3a2116
  • Loading branch information
William Kemper committed Dec 6, 2024
1 parent 9738579 commit c69a964
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 7 deletions.
2 changes: 0 additions & 2 deletions src/hotspot/share/gc/shenandoah/shenandoahAffiliation.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ inline const char* shenandoah_affiliation_code(ShenandoahAffiliation type) {
return "O";
default:
ShouldNotReachHere();
return "?";
}
}

Expand All @@ -55,7 +54,6 @@ inline const char* shenandoah_affiliation_name(ShenandoahAffiliation type) {
return "OLD";
default:
ShouldNotReachHere();
return "?";
}
}

Expand Down
4 changes: 2 additions & 2 deletions src/hotspot/share/gc/shenandoah/shenandoahHeap.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -532,9 +532,9 @@ class ShenandoahHeap : public CollectedHeap {
ShenandoahFreeSet* free_set() const { return _free_set; }
ShenandoahPacer* pacer() const { return _pacer; }

ShenandoahPhaseTimings* phase_timings() const { return _phase_timings; }
ShenandoahPhaseTimings* phase_timings() const { return _phase_timings; }

ShenandoahEvacOOMHandler* oom_evac_handler() { return &_oom_evac_handler; }
ShenandoahEvacOOMHandler* oom_evac_handler() { return &_oom_evac_handler; }

void on_cycle_start(GCCause::Cause cause, ShenandoahGeneration* generation);
void on_cycle_end(ShenandoahGeneration* generation);
Expand Down
2 changes: 0 additions & 2 deletions src/hotspot/share/gc/shenandoah/shenandoahHeap.inline.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -331,8 +331,6 @@ void ShenandoahHeap::increase_object_age(oop obj, uint additional_age) {
// necessarily be determined because of concurrent locking by the
// mutator
uint ShenandoahHeap::get_object_age(oop obj) {
// This is impossible to do unless we "freeze" ABA-type oscillations
// With Lilliput, we can do this more easily.
markWord w = obj->mark();
assert(!w.is_marked(), "must not be forwarded");
if (w.has_monitor()) {
Expand Down
1 change: 0 additions & 1 deletion src/hotspot/share/gc/shenandoah/shenandoahMmuTracker.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@
#ifndef SHARE_GC_SHENANDOAH_SHENANDOAHMMUTRACKER_HPP
#define SHARE_GC_SHENANDOAH_SHENANDOAHMMUTRACKER_HPP

#include "runtime/mutex.hpp"
#include "utilities/numberSeq.hpp"

class ShenandoahGeneration;
Expand Down

0 comments on commit c69a964

Please sign in to comment.