# HG changeset patch
# User coleenp
# Date 1564569433 14400
# Node ID dbe471d2f8f8247dbacdbe54067aa617d1b687be
# Parent  36ac7dfc2141128780f4fd8a7c73864c6d926425
8228673: Remove develop flag StrictSafepointChecks
Reviewed-by: shade, dholmes

diff -r 36ac7dfc2141 -r dbe471d2f8f8 src/hotspot/share/gc/shared/memAllocator.cpp
--- a/src/hotspot/share/gc/shared/memAllocator.cpp	Wed Jul 31 10:03:44 2019 +0200
+++ b/src/hotspot/share/gc/shared/memAllocator.cpp	Wed Jul 31 06:37:13 2019 -0400
@@ -172,11 +172,9 @@
   // This is a VM policy failure, so how do we exhaustively test it?
   assert(!_thread->has_pending_exception(),
          "shouldn't be allocating with pending exception");
-  if (StrictSafepointChecks) {
-    // Allocation of an oop can always invoke a safepoint,
-    // hence, the true argument.
-    _thread->check_for_valid_safepoint_state(true);
-  }
+  // Allocation of an oop can always invoke a safepoint,
+  // hence, the true argument.
+  _thread->check_for_valid_safepoint_state(true);
 }
 #endif
 
diff -r 36ac7dfc2141 -r dbe471d2f8f8 src/hotspot/share/runtime/globals.hpp
--- a/src/hotspot/share/runtime/globals.hpp	Wed Jul 31 10:03:44 2019 +0200
+++ b/src/hotspot/share/runtime/globals.hpp	Wed Jul 31 06:37:13 2019 -0400
@@ -398,10 +398,6 @@
           "Set when executing debug methods in debug.cpp "                  \
           "(to prevent triggering assertions)")                             \
                                                                             \
-  notproduct(bool, StrictSafepointChecks, trueInDebug,                      \
-          "Enable strict checks that safepoints cannot happen for threads " \
-          "that use NoSafepointVerifier")                                   \
-                                                                            \
   notproduct(bool, VerifyLastFrame, false,                                  \
           "Verify oops on last frame on entry to VM")                       \
                                                                             \
diff -r 36ac7dfc2141 -r dbe471d2f8f8 src/hotspot/share/runtime/mutex.cpp
--- a/src/hotspot/share/runtime/mutex.cpp	Wed Jul 31 10:03:44 2019 +0200
+++ b/src/hotspot/share/runtime/mutex.cpp	Wed Jul 31 06:37:13 2019 -0400
@@ -53,7 +53,7 @@
   }
 #endif // CHECK_UNHANDLED_OOPS
 
-  DEBUG_ONLY(check_prelock_state(self, StrictSafepointChecks));
+  DEBUG_ONLY(check_prelock_state(self, true));
   assert(_owner != self, "invariant");
 
   Monitor* in_flight_monitor = NULL;
diff -r 36ac7dfc2141 -r dbe471d2f8f8 src/hotspot/share/runtime/synchronizer.cpp
--- a/src/hotspot/share/runtime/synchronizer.cpp	Wed Jul 31 10:03:44 2019 +0200
+++ b/src/hotspot/share/runtime/synchronizer.cpp	Wed Jul 31 06:37:13 2019 -0400
@@ -442,7 +442,7 @@
 ObjectLocker::ObjectLocker(Handle obj, Thread* thread, bool doLock) {
   _dolock = doLock;
   _thread = thread;
-  debug_only(if (StrictSafepointChecks) _thread->check_for_valid_safepoint_state(false);)
+  _thread->check_for_valid_safepoint_state(false);
   _obj = obj;
 
   if (_dolock) {