8333554: Parallel: Remove unused PSParallelCompact::is_in

Reviewed-by: tschatzl
robehn · Jun 5, 2024 · 3cbdf8d · 3cbdf8d
1 parent 3944e67
commit 3cbdf8d
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 18 deletions.
diff --git a/src/hotspot/share/gc/parallel/parMarkBitMap.hpp b/src/hotspot/share/gc/parallel/parMarkBitMap.hpp
@@ -32,9 +32,9 @@
 class PSVirtualSpace;
 
 class ParMarkBitMap: public CHeapObj<mtGC> {
-public:
   typedef BitMap::idx_t idx_t;
 
+public:
   inline ParMarkBitMap();
   bool initialize(MemRegion covered_region);
 

diff --git a/src/hotspot/share/gc/parallel/psParallelCompact.hpp b/src/hotspot/share/gc/parallel/psParallelCompact.hpp
@@ -44,7 +44,6 @@ class PSOldGen;
 class ParCompactionManager;
 class PSParallelCompact;
 class MoveAndUpdateClosure;
-class RefProcTaskExecutor;
 class ParallelOldTracer;
 class STWGCTimer;
 
@@ -681,7 +680,6 @@ ParallelCompactData::is_region_aligned(HeapWord* addr) const
 class PSParallelCompact : AllStatic {
 public:
   // Convenient access to type names.
-  typedef ParMarkBitMap::idx_t idx_t;
   typedef ParallelCompactData::RegionData RegionData;
 
   typedef enum {
@@ -800,11 +798,6 @@ class PSParallelCompact : AllStatic {
 
   template <class T> static inline void adjust_pointer(T* p);
 
-  // Compaction support.
-  // Return true if p is in the range [beg_addr, end_addr).
-  static inline bool is_in(HeapWord* p, HeapWord* beg_addr, HeapWord* end_addr);
-  static inline bool is_in(oop* p, HeapWord* beg_addr, HeapWord* end_addr);
-
   // Convenience wrappers for per-space data kept in _space_info.
   static inline MutableSpace*     space(SpaceId space_id);
   static inline HeapWord*         new_top(SpaceId space_id);
@@ -899,8 +892,6 @@ class MoveAndUpdateClosure: public StackObj {
   inline void update_state(size_t words);
 
 public:
-  typedef ParMarkBitMap::idx_t idx_t;
-
   ParMarkBitMap*        bitmap() const { return _bitmap; }
 
   size_t    words_remaining()    const { return _words_remaining; }

diff --git a/src/hotspot/share/gc/parallel/psParallelCompact.inline.hpp b/src/hotspot/share/gc/parallel/psParallelCompact.inline.hpp
@@ -40,14 +40,6 @@ inline bool PSParallelCompact::is_marked(oop obj) {
   return mark_bitmap()->is_marked(obj);
 }
 
-inline bool PSParallelCompact::is_in(HeapWord* p, HeapWord* beg_addr, HeapWord* end_addr) {
-  return p >= beg_addr && p < end_addr;
-}
-
-inline bool PSParallelCompact::is_in(oop* p, HeapWord* beg_addr, HeapWord* end_addr) {
-  return is_in((HeapWord*)p, beg_addr, end_addr);
-}
-
 inline MutableSpace* PSParallelCompact::space(SpaceId id) {
   assert(id < last_space_id, "id out of range");
   return _space_info[id].space();