[crac] Integrated: Fix JVMCI after #41
Anton Kozlov
akozlov at openjdk.org
Thu Jun 29 11:22:21 UTC 2023
On Wed, 28 Jun 2023 12:13:16 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:
> The recently added CPU_MAX feature went out of sync with JVMCI code [1]. Since this is not a real feature, but auxilary value, a distinct name for the maximum value solves the issue.
>
> [1] jtreg_test_hotspot_jtreg_tier1_compiler/compiler/jvmci/JVM_GetJVMCIRuntimeTest.jtr:
>
> jdk.vm.ci.common.JVMCIError: Missing CPU feature constants: [MAX]
> at jdk.internal.vm.ci/jdk.vm.ci.hotspot.HotSpotJVMCIBackendFactory.convertFeatures(HotSpotJVMCIBackendFactory.java:79)
> at jdk.internal.vm.ci/jdk.vm.ci.hotspot.amd64.AMD64HotSpotJVMCIBackendFactory.computeFeatures(AMD64HotSpotJVMCIBackendFactory.java:53)
> at jdk.internal.vm.ci/jdk.vm.ci.hotspot.amd64.AMD64HotSpotJVMCIBackendFactory.createTarget(AMD64HotSpotJVMCIBackendFactory.java:74)
> at jdk.internal.vm.ci/jdk.vm.ci.hotspot.amd64.AMD64HotSpotJVMCIBackendFactory.createJVMCIBackend(AMD64HotSpotJVMCIBackendFactory.java:109)
> at jdk.internal.vm.ci/jdk.vm.ci.hotspot.HotSpotJVMCIRuntime.<init>(HotSpotJVMCIRuntime.java:549)
> at jdk.internal.vm.ci/jdk.vm.ci.hotspot.HotSpotJVMCIRuntime.runtime(HotSpotJVMCIRuntime.java:176)
> at jdk.internal.vm.ci/jdk.vm.ci.runtime.JVMCI.initializeRuntime(Native Method)
> at jdk.internal.vm.ci/jdk.vm.ci.runtime.JVMCI.getRuntime(JVMCI.java:65)
> at compiler.jvmci.JVM_GetJVMCIRuntimeTest.run(JVM_GetJVMCIRuntimeTest.java:77)
> at compiler.jvmci.JVM_GetJVMCIRuntimeTest.main(JVM_GetJVMCIRuntimeTest.java:70)
> at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
> at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.base/java.lang.reflect.Method.invoke(Method.java:568)
> at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
> at java.base/java.lang.Thread.run(Thread.java:833)
This pull request has now been integrated.
Changeset: a9f87061
Author: Anton Kozlov <akozlov at openjdk.org>
URL: https://git.openjdk.org/crac/commit/a9f87061492024457d65cc94ac8aa9eb102285d7
Stats: 12 lines in 2 files changed: 2 ins; 3 del; 7 mod
Fix JVMCI after #41
Reviewed-by: rmarchenko
-------------
PR: https://git.openjdk.org/crac/pull/88
More information about the crac-dev
mailing list