8168038: Some methods in G1RemSet are virtual unnecessarily
authorjprovino
Mon, 27 Feb 2017 12:41:41 -0500
changeset 46291 a24addecca3a
parent 46273 898ec28d19d7
child 46292 6c4a987f7cd8
8168038: Some methods in G1RemSet are virtual unnecessarily Summary: G1RemSet has a number of virtual functions, yet is derived from CHeapObj and has no subclasses. Reviewed-by: kbarrett, tschatzl
hotspot/src/share/vm/gc/g1/g1RemSet.hpp
--- a/hotspot/src/share/vm/gc/g1/g1RemSet.hpp	Fri Feb 17 02:31:12 2017 +0000
+++ b/hotspot/src/share/vm/gc/g1/g1RemSet.hpp	Mon Feb 27 12:41:41 2017 -0500
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2001, 2016, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2001, 2017, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -149,18 +149,18 @@
   // If oops_in_heap_closure is not NULL, a true result is returned
   // if the given card contains oops that have references into the
   // current collection set.
-  virtual bool refine_card(jbyte* card_ptr,
-                           uint worker_i,
-                           G1ParPushHeapRSClosure* oops_in_heap_closure);
+  bool refine_card(jbyte* card_ptr,
+                   uint worker_i,
+                   G1ParPushHeapRSClosure* oops_in_heap_closure);
 
   // Print accumulated summary info from the start of the VM.
-  virtual void print_summary_info();
+  void print_summary_info();
 
   // Print accumulated summary info from the last time called.
-  virtual void print_periodic_summary_info(const char* header, uint period_count);
+  void print_periodic_summary_info(const char* header, uint period_count);
 
   // Prepare remembered set for verification.
-  virtual void prepare_for_verify();
+  void prepare_for_verify();
 
   size_t conc_refine_cards() const {
     return _conc_refine_cards;