Skip to content

Commit

Permalink
8325134: Serial: Remove Generation::used_region
Browse files Browse the repository at this point in the history
Reviewed-by: tschatzl, iwalulya
  • Loading branch information
albertnetymk committed Feb 5, 2024
1 parent 4da28b4 commit cdf918b
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 9 deletions.
4 changes: 0 additions & 4 deletions src/hotspot/share/gc/serial/generation.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -113,10 +113,6 @@ class Generation: public CHeapObj<mtGC> {

MemRegion reserved() const { return _reserved; }

// Returns a region guaranteed to contain all the objects in the
// generation.
virtual MemRegion used_region() const { return _reserved; }

/* Returns "TRUE" iff "p" points into the reserved area of the generation. */
bool is_in_reserved(const void* p) const {
return _reserved.contains(p);
Expand Down
3 changes: 2 additions & 1 deletion src/hotspot/share/gc/serial/tenuredGeneration.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
#include "gc/serial/generation.hpp"
#include "gc/shared/gcStats.hpp"
#include "gc/shared/generationCounters.hpp"
#include "gc/shared/space.hpp"
#include "utilities/macros.hpp"

class SerialBlockOffsetSharedArray;
Expand Down Expand Up @@ -92,8 +93,8 @@ class TenuredGeneration: public Generation {
size_t capacity() const;
size_t used() const;
size_t free() const;
MemRegion used_region() const;

MemRegion used_region() const { return space()->used_region(); }
MemRegion prev_used_region() const { return _prev_used_region; }
void save_used_region() { _prev_used_region = used_region(); }

Expand Down
4 changes: 0 additions & 4 deletions src/hotspot/share/gc/serial/tenuredGeneration.inline.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -41,10 +41,6 @@ inline size_t TenuredGeneration::free() const {
return space()->free();
}

inline MemRegion TenuredGeneration::used_region() const {
return space()->used_region();
}

inline bool TenuredGeneration::is_in(const void* p) const {
return space()->is_in(p);
}
Expand Down

1 comment on commit cdf918b

@openjdk-notifier
Copy link

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.