src/share/vm/gc/g1/g1CollectedHeap.hpp

rev 9422 : 8143251: HeapRetentionTest.java Test is failing on jdk9/dev
Reviewed-by:
rev 9423 : [mq]: 8143251-user-requested-cm-rev1

@@ -243,12 +243,10 @@
   // It decides whether an explicit GC should start a concurrent cycle
   // instead of doing a STW GC. Currently, a concurrent cycle is
   // explicitly started if:
   // (a) cause == _gc_locker and +GCLockerInvokesConcurrent, or
   // (b) cause == _g1_humongous_allocation
-  // These are defined in user_requested_concurrent_full_gc() because
-  // these sometimes need to be treated differently:
   // (c) cause == _java_lang_system_gc and +ExplicitGCInvokesConcurrent.
   // (d) cause == _dcmd_gc_run and +ExplicitGCInvokesConcurrent.
   // (e) cause == _update_allocation_context_stats_inc
   // (f) cause == _wb_conc_mark
   bool should_do_concurrent_full_gc(GCCause::Cause cause);

@@ -578,14 +576,11 @@
 
   void clear_cset_fast_test() {
     _in_cset_fast_test.clear();
   }
 
-  // A complement to should_do_concurrent_full_gc. GCs caused
-  // by a user sometimes needs to be treated differently
-  // from those caused by the VM.
-  bool user_requested_concurrent_full_gc(GCCause::Cause cause);
+  bool is_user_requested_concurrent_full_gc(GCCause::Cause cause);
 
   // This is called at the start of either a concurrent cycle or a Full
   // GC to update the number of old marking cycles started.
   void increment_old_marking_cycles_started();