changeset 48665:39a84de6afd6

8192971: LockCompilationTest fails intermittently Summary: Remove all unnecessary compilations Reviewed-by: kvn, thartmann
author neliasso
date Wed, 13 Dec 2017 10:21:21 +0100
parents 993b004ab38f
children 919780ab7acc
files test/hotspot/jtreg/compiler/whitebox/LockCompilationTest.java
diffstat 1 files changed, 2 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/test/hotspot/jtreg/compiler/whitebox/LockCompilationTest.java	Tue Dec 12 21:46:06 2017 -0500
+++ b/test/hotspot/jtreg/compiler/whitebox/LockCompilationTest.java	Wed Dec 13 10:21:21 2017 +0100
@@ -33,6 +33,7 @@
  *                                sun.hotspot.WhiteBox$WhiteBoxPermission
  * @run main/othervm -Xbootclasspath/a:. -Xmixed -XX:+UnlockDiagnosticVMOptions
  *                   -XX:+WhiteBoxAPI -XX:-UseCounterDecay
+ *                   -XX:CompileCommand=compileonly,*SimpleTestCaseHelper::method
  *                   compiler.whitebox.LockCompilationTest
  */
 
@@ -46,20 +47,7 @@
         // This case waits for 5 seconds and verifies that the method hasn't been
         // compiled during that time. Only do that for one of the test cases.
 
-        // Only compile SimpleTestCaseHelper.method and exclude all other to ensure no
-        // contention on the compile queue causes problems.
-        String directive =
-                "[{ match:\"*SimpleTestCaseHelper.method\", Exclude:false}, " +
-                " { match:\"*.*\", Exclude:true}]";
-        if (WHITE_BOX.addCompilerDirective(directive) != 2) {
-            throw new RuntimeException("Could not add directive");
-        }
-        try {
-            CompilerWhiteBoxTest.main(LockCompilationTest::new, new String[] {"METHOD_TEST"});
-        } finally {
-            WHITE_BOX.removeCompilerDirective(2);
-        }
-
+        CompilerWhiteBoxTest.main(LockCompilationTest::new, new String[] {"METHOD_TEST"});
     }
 
     private LockCompilationTest(TestCase testCase) {