RFR: Fix TestCommonGCLoads test

Aleksey Shipilev shade at redhat.com
Tue Jun 12 15:17:44 UTC 2018


Due to odd overlook, we don't actually execute that test. The test does not even compile :)


diff -r ad56f8186d93 test/hotspot/jtreg/gc/shenandoah/compiler/TestCommonGCLoads.java
--- a/test/hotspot/jtreg/gc/shenandoah/compiler/TestCommonGCLoads.java	Tue Jun 12 15:38:41 2018 +0200
+++ b/test/hotspot/jtreg/gc/shenandoah/compiler/TestCommonGCLoads.java	Tue Jun 12 17:16:44 2018 +0200
@@ -27,21 +27,21 @@
  * @requires vm.flavor == "server"
  * @run main/othervm -XX:-BackgroundCompilation -XX:-UseOnStackReplacement -XX:-TieredCompilation
-XX:+UseShenandoahGC
  *                   -XX:+UnlockExperimentalVMOptions -XX:-ShenandoahCommonGCStateLoads
- *                   TestExpandedWBLostNullCheckDep
+ *                   TestCommonGCLoads
  * @run main/othervm -XX:-BackgroundCompilation -XX:-UseOnStackReplacement -XX:-TieredCompilation
-XX:+UseShenandoahGC
  *                   -XX:+UnlockExperimentalVMOptions -XX:+ShenandoahCommonGCStateLoads
- *                   TestExpandedWBLostNullCheckDep
+ *                   TestCommonGCLoads
  */

 public class TestCommonGCLoads {

-    Object d = new Object();
+    static Object d = new Object();

-    Target t1 = new Target();
-    Target t2 = new Target();
-    Target t3 = new Target();
-    Target t4 = new Target();
-    Target t5 = new Target();
+    static Target t1 = new Target();
+    static Target t2 = new Target();
+    static Target t3 = new Target();
+    static Target t4 = new Target();
+    static Target t5 = new Target();

     static void test() {
         t1.field = d;
@@ -56,4 +56,8 @@
             test();
         }
     }
+
+    static class Target {
+        Object field;
+    }
 }

Testing: TestCommonGCLoads x86_64

Thanks,
-Aleksey



More information about the shenandoah-dev mailing list