8229158: make UseSwitchProfiling non-experimental or false by-default
authorchagedorn
Tue, 20 Aug 2019 10:46:23 +0200
changeset 57803 23e3ab980622
parent 57802 854e828d6b5b
child 57804 9b7b9f16dfd9
8229158: make UseSwitchProfiling non-experimental or false by-default Summary: Changed UseSwitchProfiling from experimental to diagnostic. Reviewed-by: dholmes, shade, thartmann
src/hotspot/share/runtime/globals.hpp
test/hotspot/jtreg/compiler/loopopts/LoopUnswitchingBadNodeBudget.java
test/hotspot/jtreg/compiler/loopopts/TestOverunrolling.java
--- a/src/hotspot/share/runtime/globals.hpp	Tue Aug 20 09:22:18 2019 +0200
+++ b/src/hotspot/share/runtime/globals.hpp	Tue Aug 20 10:46:23 2019 +0200
@@ -2437,11 +2437,11 @@
   diagnostic(bool, ShowRegistersOnAssert, true,                             \
           "On internal errors, include registers in error report.")         \
                                                                             \
-  experimental(bool, UseSwitchProfiling, true,                              \
+  diagnostic(bool, UseSwitchProfiling, true,                                \
           "leverage profiling for table/lookup switch")                     \
                                                                             \
   JFR_ONLY(product(bool, FlightRecorder, false,                             \
-          "(Deprecated) Enable Flight Recorder"))                                        \
+          "(Deprecated) Enable Flight Recorder"))                           \
                                                                             \
   JFR_ONLY(product(ccstr, FlightRecorderOptions, NULL,                      \
           "Flight Recorder options"))                                       \
--- a/test/hotspot/jtreg/compiler/loopopts/LoopUnswitchingBadNodeBudget.java	Tue Aug 20 09:22:18 2019 +0200
+++ b/test/hotspot/jtreg/compiler/loopopts/LoopUnswitchingBadNodeBudget.java	Tue Aug 20 10:46:23 2019 +0200
@@ -32,7 +32,7 @@
  * @run main/othervm -XX:-TieredCompilation -XX:-BackgroundCompilation
  *      -XX:-UseOnStackReplacement -XX:CompileOnly=LoopUnswitchingBadNodeBudget::test
  *      -XX:CompileCommand=dontinline,LoopUnswitchingBadNodeBudget::helper
- *      -XX:+UnlockExperimentalVMOptions -XX:-UseSwitchProfiling LoopUnswitchingBadNodeBudget
+ *      -XX:+UnlockDiagnosticVMOptions -XX:-UseSwitchProfiling LoopUnswitchingBadNodeBudget
  *
  */
 
--- a/test/hotspot/jtreg/compiler/loopopts/TestOverunrolling.java	Tue Aug 20 09:22:18 2019 +0200
+++ b/test/hotspot/jtreg/compiler/loopopts/TestOverunrolling.java	Tue Aug 20 10:46:23 2019 +0200
@@ -26,7 +26,7 @@
  * @bug 8159016 8202949 8203915
  * @summary Tests correct dominator information after over-unrolling a loop.
  * @requires vm.gc == "Parallel" | vm.gc == "null"
- * @run main/othervm -XX:+IgnoreUnrecognizedVMOptions -XX:+UnlockExperimentalVMOptions
+ * @run main/othervm -XX:+IgnoreUnrecognizedVMOptions -XX:+UnlockDiagnosticVMOptions
  *                   -Xcomp -XX:-TieredCompilation -XX:-UseSwitchProfiling
  *                   -XX:-UseCountedLoopSafepoints -XX:LoopUnrollLimit=250
  *                   -XX:-UseG1GC -XX:+UseParallelGC compiler.loopopts.TestOverunrolling