RFR: 8264806: Remove the experimental JIT compiler [v2]

Doug Simon dnsimon at openjdk.java.net
Tue Apr 27 19:52:22 UTC 2021


On Tue, 27 Apr 2021 19:07:45 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> > I guess this should really be named `isJVMCICompilerEnabled` now and the `vm.graal.enabled` predicate renamed to `vm.jvmcicompiler.enabled` but maybe that's too big a change (or can be done later).
> 
> @dougxc, I don't think that we should (or even can) rename it to `vm.jvmcicompiler.enabled`. although the value of the property is indeed `true` whenever a jvmci compiler is enabled, it is used to filter out tests that are incompatible with a particular compiler -- graal. so what we can do is to change `requires/VMProps.java` to set `vm.graal.enabled` only if the jvmci compiler is indeed graal, but on the other hand, we haven't heard anyone complaining that the test coverage for their jvmci compiler has been reduced, so I don't see much of the benefit here.
> 
> -- Igor

Yes, we should just it as is until a second JVMCI compiler shows up.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3421


More information about the hotspot-compiler-dev mailing list