Skip to content

8367689: Revert removal of several compilation-related vmStructs fields #27318

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

8367689: Revert removal of several compilation-related vmStructs fields #27318

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
b2a76ef
bring back fields
fandreuz Sep 16, 2025
5400dd0
remove indent
fandreuz Sep 16, 2025
5bc13c6
ops
fandreuz Sep 16, 2025
3a263b3
nn
fandreuz Sep 16, 2025
e15cbda
nn
fandreuz Sep 16, 2025
f9cb18b
cc
fandreuz Sep 16, 2025
8603beb
nn
fandreuz Sep 16, 2025
3165277
nn
fandreuz Sep 16, 2025
e8b7849
note
fandreuz Sep 16, 2025
194a9d6
comment
fandreuz Sep 17, 2025
d7c4db0
missing close
Sep 18, 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
1 change: 1 addition & 0 deletions src/hotspot/share/ci/ciClassList.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ friend class ciObjectFactory; \
// Any more access must be given explicitly.
#define CI_PACKAGE_ACCESS_TO \
friend class ciObjectFactory; \
friend class VMStructs; \
friend class ciCallSite; \
friend class ciConstantPoolCache; \
friend class ciField; \
Expand Down
14 changes: 14 additions & 0 deletions src/hotspot/share/runtime/vmStructs.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -666,6 +666,14 @@
static_field(VMRegImpl, regName[0], const char*) \
static_field(VMRegImpl, stack0, VMReg) \
\
/******************************************************************************************/ \
/* CI (NOTE: these CI fields are retained in VMStructs for the benefit of external tools, */ \
/* to ease their migration to a future alternative.) */ \
/******************************************************************************************/ \
\
Copy link
Contributor

@kevinjwalls kevinjwalls Sep 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops just a missing close ), but yes other than that I think we're done 8-)

Copy link
Contributor Author

@fandreuz fandreuz Sep 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, fixed in d7c4db0

nonstatic_field(CompilerThread, _env, ciEnv*) \
nonstatic_field(ciEnv, _task, CompileTask*) \
\
/************/ \
/* Monitors */ \
/************/ \
Expand Down Expand Up @@ -1148,6 +1156,12 @@
declare_toplevel_type(BasicLock) \
declare_toplevel_type(BasicObjectLock) \
\
/*********************/ \
/* CI */ \
/*********************/ \
\
declare_toplevel_type(ciEnv) \
\
/********************/ \
/* -XX flags */ \
/********************/ \
Expand Down