8301340: Make DirtyCardToOopClosure stack-allocated

Reviewed-by: stefank, tschatzl

Author: albertnetymk

src/hotspot/share/gc/shared/cardTableRS.cpp

@@ -449,8 +449,8 @@ void CardTableRS::non_clean_card_iterate(TenuredSpace* sp,
   }
   // clear_cl finds contiguous dirty ranges of cards to process and clear.
 
-  DirtyCardToOopClosure* dcto_cl = sp->new_dcto_cl(cl);
-  ClearNoncleanCardWrapper clear_cl(dcto_cl, ct);
+  DirtyCardToOopClosure dcto_cl{sp, cl};
+  ClearNoncleanCardWrapper clear_cl(&dcto_cl, ct);
 
   clear_cl.do_MemRegion(mr);
 }

src/hotspot/share/gc/shared/space.cpp

@@ -150,11 +150,6 @@ void DirtyCardToOopClosure::walk_mem_region_with_cl(MemRegion mr,
   }
 }
 
-DirtyCardToOopClosure*
-ContiguousSpace::new_dcto_cl(OopIterateClosure* cl) {
-  return new DirtyCardToOopClosure(this, cl);
-}
-
 void Space::initialize(MemRegion mr,
                        bool clear_space,
                        bool mangle_space) {

src/hotspot/share/gc/shared/space.hpp

@@ -239,7 +239,7 @@ class Space: public CHeapObj<mtGC> {
 // 2. That the space is really made up of objects and not just
 //    blocks.
 
-class DirtyCardToOopClosure: public MemRegionClosureRO {
+class DirtyCardToOopClosure: public MemRegionClosure {
 protected:
   OopIterateClosure* _cl;
   Space* _sp;
@@ -472,8 +472,6 @@ class ContiguousSpace: public CompactibleSpace {
     set_top(compaction_top());
   }
 
-  DirtyCardToOopClosure* new_dcto_cl(OopIterateClosure* cl);
-
   // Apply "blk->do_oop" to the addresses of all reference fields in objects
   // starting with the _saved_mark_word, which was noted during a generation's
   // save_marks and is required to denote the head of an object.

src/hotspot/share/memory/memRegion.hpp

@@ -105,15 +105,4 @@ class MemRegionClosure : public StackObj {
   virtual void do_MemRegion(MemRegion mr) = 0;
 };
 
-// A ResourceObj version of MemRegionClosure
-
-class MemRegionClosureRO: public MemRegionClosure {
-public:
-  void* operator new(size_t size) throw() {
-    return resource_allocate_bytes(size);
-  }
-
-  void  operator delete(void* p) {} // nothing to do
-};
-
 #endif // SHARE_MEMORY_MEMREGION_HPP