8224599: Remove globals_ext.hpp

Reviewed-by: coleenp, kvn, gziemski, ehelin

Author: stefank

src/hotspot/share/gc/cms/jvmFlagConstraintsCMS.hpp

@@ -25,7 +25,7 @@
 #ifndef SHARE_GC_CMS_JVMFLAGCONSTRAINTSCMS_HPP
 #define SHARE_GC_CMS_JVMFLAGCONSTRAINTSCMS_HPP
 
-#include "runtime/globals.hpp"
+#include "runtime/flags/jvmFlag.hpp"
 #include "utilities/globalDefinitions.hpp"
 
 // CMS Flag Constraints

src/hotspot/share/gc/g1/jvmFlagConstraintsG1.hpp

@@ -25,7 +25,7 @@
 #ifndef SHARE_GC_G1_JVMFLAGCONSTRAINTSG1_HPP
 #define SHARE_GC_G1_JVMFLAGCONSTRAINTSG1_HPP
 
-#include "runtime/globals.hpp"
+#include "runtime/flags/jvmFlag.hpp"
 #include "utilities/globalDefinitions.hpp"
 
 // G1 Flag Constraints

src/hotspot/share/gc/parallel/jvmFlagConstraintsParallel.hpp

@@ -25,7 +25,7 @@
 #ifndef SHARE_GC_PARALLEL_JVMFLAGCONSTRAINTSPARALLEL_HPP
 #define SHARE_GC_PARALLEL_JVMFLAGCONSTRAINTSPARALLEL_HPP
 
-#include "runtime/globals.hpp"
+#include "runtime/flags/jvmFlag.hpp"
 #include "utilities/globalDefinitions.hpp"
 
 // Parallel Subconstraints

src/hotspot/share/gc/shared/jvmFlagConstraintsGC.hpp

@@ -25,6 +25,7 @@
 #ifndef SHARE_GC_SHARED_JVMFLAGCONSTRAINTSGC_HPP
 #define SHARE_GC_SHARED_JVMFLAGCONSTRAINTSGC_HPP
 
+#include "runtime/flags/jvmFlag.hpp"
 #include "utilities/globalDefinitions.hpp"
 #include "utilities/macros.hpp"
 #if INCLUDE_CMSGC

src/hotspot/share/runtime/flags/jvmFlag.cpp

@@ -339,9 +339,8 @@ bool JVMFlag::is_constant_in_binary() const {
 }
 
 bool JVMFlag::is_unlocker() const {
-  return strcmp(_name, "UnlockDiagnosticVMOptions") == 0     ||
-  strcmp(_name, "UnlockExperimentalVMOptions") == 0   ||
-  is_unlocker_ext();
+  return strcmp(_name, "UnlockDiagnosticVMOptions") == 0 ||
+         strcmp(_name, "UnlockExperimentalVMOptions") == 0;
 }
 
 bool JVMFlag::is_unlocked() const {
@@ -351,7 +350,7 @@ bool JVMFlag::is_unlocked() const {
   if (is_experimental()) {
     return UnlockExperimentalVMOptions;
   }
-  return is_unlocked_ext();
+  return true;
 }
 
 void JVMFlag::clear_diagnostic() {
@@ -388,18 +387,18 @@ JVMFlag::MsgType JVMFlag::get_locked_message(char* buf, int buflen) const {
                  _name);
     return JVMFlag::NOTPRODUCT_FLAG_BUT_PRODUCT_BUILD;
   }
-  return get_locked_message_ext(buf, buflen);
+  return JVMFlag::NONE;
 }
 
 bool JVMFlag::is_writeable() const {
-  return is_manageable() || (is_product() && is_read_write()) || is_writeable_ext();
+  return is_manageable() || (is_product() && is_read_write());
 }
 
 // All flags except "manageable" are assumed to be internal flags.
 // Long term, we need to define a mechanism to specify which flags
 // are external/stable and change this function accordingly.
 bool JVMFlag::is_external() const {
-  return is_manageable() || is_external_ext();
+  return is_manageable();
 }
 
 // Helper function for JVMFlag::print_on().
@@ -881,7 +880,6 @@ static JVMFlag flagTable[] = {
              IGNORE_RANGE, \
              IGNORE_CONSTRAINT, \
              IGNORE_WRITEABLE)
-  FLAGTABLE_EXT
   {0, NULL, NULL}
 };
 

src/hotspot/share/runtime/flags/jvmFlag.hpp

@@ -206,11 +206,6 @@ struct JVMFlag {
   bool is_writeable() const;
   bool is_external() const;
 
-  bool is_unlocker_ext() const;
-  bool is_unlocked_ext() const;
-  bool is_writeable_ext() const;
-  bool is_external_ext() const;
-
   void clear_diagnostic();
 
   JVMFlag::MsgType get_locked_message(char*, int) const;

src/hotspot/share/runtime/flags/jvmFlagConstraintList.cpp

@@ -269,8 +269,6 @@ void JVMFlagConstraintList::init(void) {
             EMIT_CONSTRAINT_CHECK,
             IGNORE_WRITEABLE)
 
-  EMIT_CONSTRAINTS_FOR_GLOBALS_EXT
-
   EMIT_CONSTRAINT_END
 }
 

src/hotspot/share/runtime/flags/jvmFlagRangeList.cpp

@@ -334,8 +334,6 @@ void JVMFlagRangeList::init(void) {
             IGNORE_CONSTRAINT,
             IGNORE_WRITEABLE)
 
-  EMIT_RANGES_FOR_GLOBALS_EXT
-
   EMIT_RANGE_END
 }
 

src/hotspot/share/runtime/flags/jvmFlagWriteableList.cpp

@@ -126,8 +126,6 @@ void JVMFlagWriteableList::init(void) {
             IGNORE_CONSTRAINT,
             EMIT_WRITEABLE_CHECK)
 
-  EMIT_WRITEABLES_FOR_GLOBALS_EXT
-
   EMIT_WRITEABLE_END
 }
 

src/hotspot/share/runtime/globals.cpp

@@ -78,5 +78,3 @@ ALL_FLAGS(MATERIALIZE_DEVELOPER_FLAG,     \
           IGNORE_RANGE,                   \
           IGNORE_CONSTRAINT,              \
           IGNORE_WRITEABLE)
-
-MATERIALIZE_FLAGS_EXT