8138846: Remove unused enum ConcurrentGCThread::CGC_flag_type
authorpliden
Tue, 06 Oct 2015 08:05:11 +0200
changeset 33123 8a67e36f19d3
parent 33122 cdb3e19c1d9e
child 33124 e256f7a94c38
child 33127 0dfbcf8b7d7a
8138846: Remove unused enum ConcurrentGCThread::CGC_flag_type Reviewed-by: jwilhelm, brutisso
hotspot/src/share/vm/gc/shared/concurrentGCThread.cpp
hotspot/src/share/vm/gc/shared/concurrentGCThread.hpp
--- a/hotspot/src/share/vm/gc/shared/concurrentGCThread.cpp	Mon Oct 05 21:17:11 2015 -0400
+++ b/hotspot/src/share/vm/gc/shared/concurrentGCThread.cpp	Tue Oct 06 08:05:11 2015 +0200
@@ -33,10 +33,6 @@
 #include "runtime/javaCalls.hpp"
 #include "runtime/os.hpp"
 
-// CopyrightVersion 1.2
-
-int  ConcurrentGCThread::_CGC_flag            = CGC_nil;
-
 ConcurrentGCThread::ConcurrentGCThread() :
   _should_terminate(false), _has_terminated(false) {
 };
--- a/hotspot/src/share/vm/gc/shared/concurrentGCThread.hpp	Mon Oct 05 21:17:11 2015 -0400
+++ b/hotspot/src/share/vm/gc/shared/concurrentGCThread.hpp	Tue Oct 06 08:05:11 2015 +0200
@@ -35,19 +35,6 @@
   bool _should_terminate;
   bool _has_terminated;
 
-  enum CGC_flag_type {
-    CGC_nil           = 0x0,
-    CGC_dont_suspend  = 0x1,
-    CGC_CGC_safepoint = 0x2,
-    CGC_VM_safepoint  = 0x4
-  };
-
-  static int _CGC_flag;
-
-  static bool CGC_flag_is_set(int b)       { return (_CGC_flag & b) != 0; }
-  static int set_CGC_flag(int b)           { return _CGC_flag |= b; }
-  static int reset_CGC_flag(int b)         { return _CGC_flag &= ~b; }
-
   // Create and start the thread (setting it's priority high.)
   void create_and_start();
 
@@ -63,13 +50,10 @@
   void terminate();
 
 public:
-  // Constructor
-
   ConcurrentGCThread();
-  ~ConcurrentGCThread() {} // Exists to call NamedThread destructor.
 
   // Tester
-  bool is_ConcurrentGC_thread() const          { return true;       }
+  bool is_ConcurrentGC_thread() const { return true; }
 };
 
 // The SurrogateLockerThread is used by concurrent GC threads for