RFR (XS): 8209689: Compiler.isGraalEnabled should not check jvmci.Compiler property
Christian Thalinger
cthalinger at twitter.com
Mon Aug 20 12:50:31 UTC 2018
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>
More information about the hotspot-compiler-dev
mailing list