Need a sponsor for push 8141135 into jd9/hs-rt

Alexander Harlap alexander.harlap at oracle.com
Mon Nov 23 17:25:16 UTC 2015


I need a sponsor to push attached 8141135 - Remove G1RemSet::write_ref

Alex
-------------- next part --------------
# HG changeset patch
# User aharlap
# Date 1448296982 18000
# Node ID 9ced7c17cea19dbaf2c020486c978cbc032d28e0
# Parent  e9b6c63ca4a6c761943fc6a3f7ae3ccdf80965a4
8141135: Remove G1RemSet::write_ref
Summary: Remove unused memembers of G1RemSet
Reviewed-by: kbarrett

diff -r e9b6c63ca4a6 -r 9ced7c17cea1 src/share/vm/gc/g1/g1RemSet.hpp
--- a/src/share/vm/gc/g1/g1RemSet.hpp	Fri Nov 20 12:42:21 2015 +0100
+++ b/src/share/vm/gc/g1/g1RemSet.hpp	Mon Nov 23 11:43:02 2015 -0500
@@ -54,16 +54,6 @@
   uint n_workers();
 
 protected:
-  enum SomePrivateConstants {
-    UpdateRStoMergeSync  = 0,
-    MergeRStoDoDirtySync = 1,
-    DoDirtySync          = 2,
-    LastSync             = 3,
-
-    SeqTask              = 0,
-    NumSeqTasks          = 1
-  };
-
   CardTableModRefBS*     _ct_bs;
   G1CollectorPolicy*     _g1p;
 
@@ -123,7 +113,6 @@
 
   // Record, if necessary, the fact that *p (where "p" is in region "from",
   // which is required to be non-NULL) has changed to a new non-NULL value.
-  template <class T> void write_ref(HeapRegion* from, T* p);
   template <class T> void par_write_ref(HeapRegion* from, T* p, uint tid);
 
   // Requires "region_bm" and "card_bm" to be bitmaps with 1 bit per region
diff -r e9b6c63ca4a6 -r 9ced7c17cea1 src/share/vm/gc/g1/g1RemSet.inline.hpp
--- a/src/share/vm/gc/g1/g1RemSet.inline.hpp	Fri Nov 20 12:42:21 2015 +0100
+++ b/src/share/vm/gc/g1/g1RemSet.inline.hpp	Mon Nov 23 11:43:02 2015 -0500
@@ -35,11 +35,6 @@
 }
 
 template <class T>
-inline void G1RemSet::write_ref(HeapRegion* from, T* p) {
-  par_write_ref(from, p, 0);
-}
-
-template <class T>
 inline void G1RemSet::par_write_ref(HeapRegion* from, T* p, uint tid) {
   oop obj = oopDesc::load_decode_heap_oop(p);
   if (obj == NULL) {


More information about the hotspot-gc-dev mailing list