Skip to content

8345826: Do not automatically resolve jdk.internal.vm.ci when libgraal is used #25240

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 11 commits into from
Closed

8345826: Do not automatically resolve jdk.internal.vm.ci when libgraal is used #25240

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
171703a
--add-modules=jdk.internal.vm.ci implies -XX:+EnableJVMCI
dougxc May 13, 2025
0e8773e
added comment in check_vm_args_consistency
dougxc May 15, 2025
3436033
resolve jdk.internal.vm.ci if +EnableJVMCI and -UseJVMCINativeLibrary
dougxc May 16, 2025
cc49ed0
fix TestHotSpotJVMCIRuntime
dougxc May 16, 2025
7b30238
removed use of jdk.internal.vm.ci.enabled property
dougxc May 16, 2025
3cdef58
fixed comment
dougxc May 16, 2025
1fe56b4
improved error message
dougxc May 16, 2025
cd615ef
JAVA_NOT_ENABLED_ERROR_MESSAGE -> JVMCI_NOT_ENABLED_ERROR_MESSAGE
dougxc May 16, 2025
d9223af
load the JVMCI module if +EnableJVMCI is set on the command line
dougxc May 16, 2025
196425f
load the JVMCI module if +EnableJVMCI is set in the jimage
dougxc May 18, 2025
b74077f
swapped order of recommended options in error message
dougxc May 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions src/hotspot/share/jvmci/jvmci_globals.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,13 @@ class fileStream;
"Enable JVMCI support in the VM. " \
"Defaults to true if UseJVMCICompiler is true or " \
"--add-modules=jdk.internal.vm.ci was specified. " \
"If explicity set to true on the command line OR defaulted to " \
"true by UseJVMCICompiler and libjvmci is not enabled, the " \
"behavior --add-modules=jdk.internal.vm.ci is triggered.") \
"The behavior of --add-modules=jdk.internal.vm.ci is triggered " \
"if any of the following is true: " \
"1. -XX:+EnableJVMCI is set to true on the command line. " \
"2. -XX:+EnableJVMCI is set to true by jdk/internal/vm/options " \
" in the java.base module. " \
"3. EnableJVMCI is defaulted to true by UseJVMCICompiler and " \
" libjvmci is not enabled") \
\
product(bool, UseGraalJIT, false, EXPERIMENTAL, \
"Select the Graal JVMCI compiler. This is an alias for: " \
Expand Down
4 changes: 2 additions & 2 deletions src/hotspot/share/runtime/arguments.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1802,8 +1802,8 @@ bool Arguments::check_vm_args_consistency() {
#if INCLUDE_JVMCI
if (status && EnableJVMCI) {
// Add the JVMCI module if not using libjvmci or EnableJVMCI
// was explicitly set on the command line.
if ((!UseJVMCINativeLibrary || FLAG_IS_CMDLINE(EnableJVMCI)) && ClassLoader::is_module_observable("jdk.internal.vm.ci") && !_jvmci_module_added) {
// was explicitly set on the command line or in the jimage.
if ((!UseJVMCINativeLibrary || FLAG_IS_CMDLINE(EnableJVMCI) || FLAG_IS_JIMAGE_RESOURCE(EnableJVMCI)) && ClassLoader::is_module_observable("jdk.internal.vm.ci") && !_jvmci_module_added) {
if (!create_numbered_module_property("jdk.module.addmods", "jdk.internal.vm.ci", _addmods_count++)) {
return false;
}
Expand Down