RFR (XS): 8209689: Compiler.isGraalEnabled should not check jvmci.Compiler property

[Christian Thalinger]

https://bugs.openjdk.java.net/browse/JDK-8209689 <https://bugs.openjdk.java.net/browse/JDK-8209689>

I noticed that some tests are run in our Graal nightlies although the @requires explicitly states: 

!vm.graal.enabled

Turns out the code in Compiler.isGraalEnabled still checks for the jvmci.Compiler property but this property is not set and is null by default.

(A bit more information in the ticket.)

diff --git a/test/lib/sun/hotspot/code/Compiler.java b/test/lib/sun/hotspot/code/Compiler.java
--- a/test/lib/sun/hotspot/code/Compiler.java
+++ b/test/lib/sun/hotspot/code/Compiler.java
@@ -55,11 +55,6 @@
         if (useJvmciComp == null || !useJvmciComp) {
             return false;
         }
-        // This check might be redundant but let's keep it for now.
-        String jvmciCompiler = System.getProperty("jvmci.Compiler");
-        if (jvmciCompiler == null || !jvmciCompiler.equals("graal")) {
-            return false;
-        }
 
         Boolean tieredCompilation = WB.getBooleanVMFlag("TieredCompilation");
         Long compLevel = WB.getIntxVMFlag("TieredStopAtLevel");

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20180820/72e63bde/attachment.html>