8148951: Remove unused method Generation::performs_in_place_marking()
authorbrutisso
Wed, 03 Feb 2016 18:21:43 +0100
changeset 35930 722866f5e209
parent 35929 2783e5e132e3
child 35931 f6c656c05bfc
8148951: Remove unused method Generation::performs_in_place_marking() Reviewed-by: david, jwilhelm
hotspot/src/share/vm/gc/serial/defNewGeneration.hpp
hotspot/src/share/vm/gc/shared/genCollectedHeap.cpp
hotspot/src/share/vm/gc/shared/generation.hpp
--- a/hotspot/src/share/vm/gc/serial/defNewGeneration.hpp	Wed Feb 03 18:18:00 2016 +0100
+++ b/hotspot/src/share/vm/gc/serial/defNewGeneration.hpp	Wed Feb 03 18:21:43 2016 +0100
@@ -285,9 +285,6 @@
   // Save the tops for eden, from, and to
   virtual void record_spaces_top();
 
-  // Doesn't require additional work during GC prologue and epilogue
-  virtual bool performs_in_place_marking() const { return false; }
-
   // Accessing marks
   void save_marks();
   void reset_saved_marks();
--- a/hotspot/src/share/vm/gc/shared/genCollectedHeap.cpp	Wed Feb 03 18:18:00 2016 +0100
+++ b/hotspot/src/share/vm/gc/shared/genCollectedHeap.cpp	Wed Feb 03 18:21:43 2016 +0100
@@ -458,7 +458,6 @@
         prepared_for_verification = true;
       }
 
-      assert(!_young_gen->performs_in_place_marking(), "No young generation do in place marking");
       collect_generation(_young_gen,
                          full,
                          size,
@@ -489,8 +488,6 @@
         prepare_for_verify();
       }
 
-      assert(_old_gen->performs_in_place_marking(), "All old generations do in place marking");
-
       if (do_young_collection) {
         // We did a young GC. Need a new GC id for the old GC.
         GCIdMarkAndRestore gc_id_mark;
--- a/hotspot/src/share/vm/gc/shared/generation.hpp	Wed Feb 03 18:18:00 2016 +0100
+++ b/hotspot/src/share/vm/gc/shared/generation.hpp	Wed Feb 03 18:21:43 2016 +0100
@@ -309,13 +309,6 @@
   // do nothing.
   virtual void par_oop_since_save_marks_iterate_done(int thread_num) {}
 
-  // This generation does in-place marking, meaning that mark words
-  // are mutated during the marking phase and presumably reinitialized
-  // to a canonical value after the GC. This is currently used by the
-  // biased locking implementation to determine whether additional
-  // work is required during the GC prologue and epilogue.
-  virtual bool performs_in_place_marking() const { return true; }
-
   // Returns "true" iff collect() should subsequently be called on this
   // this generation. See comment below.
   // This is a generic implementation which can be overridden.