Skip to content

8339113: AccessFlags can be u2 in metadata #22246

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 18 commits into from
Closed

8339113: AccessFlags can be u2 in metadata #22246

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
5e8fee6
8339113: AccessFlags can be u2 in metadata
coleenp Nov 19, 2024
3106bdd
Remove JVM_ACC_WRITTEN_FLAGS because they are all written in the clas…
coleenp Dec 17, 2024
3ec96d5
Update src/hotspot/share/opto/library_call.cpp
coleenp Dec 19, 2024
cc69a3f
Update src/hotspot/cpu/ppc/templateInterpreterGenerator_ppc.cpp
coleenp Dec 19, 2024
fbe8331
Renamed AccessFlags.as_int() to as_unsigned_short(), moved masks for …
coleenp Dec 19, 2024
428a043
Merge branch 'master' into access-flags
coleenp Dec 19, 2024
aa2887d
Fix merge and compilation errors.
coleenp Dec 19, 2024
522ade8
Fixed bug.
coleenp Dec 19, 2024
828e483
Use ldrh rather than load_unsigned_short for aarch64, use testbitdi f…
coleenp Dec 19, 2024
4faf19b
Restore ACC in comment.
coleenp Dec 20, 2024
df95d44
Make vmIntrinsics::find_id() take u2 not jshort.
coleenp Jan 2, 2025
5a84b13
Fix lhz in ppc
coleenp Jan 2, 2025
8cafd45
Import @offamitkumar patch for s390.
coleenp Jan 2, 2025
c60e96b
Happy New Year
coleenp Jan 2, 2025
8682a77
Adopt @sspitsyn suggestion and strengthen field/method/class flag ret…
coleenp Jan 3, 2025
82bd1a2
Fix more copyrights.
coleenp Jan 3, 2025
d43b2fa
Remove unused SA function.
coleenp Jan 6, 2025
35784c7
Make more AccessFlags fetches more specific and remove an assert and …
coleenp Jan 7, 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
2 changes: 1 addition & 1 deletion src/hotspot/cpu/aarch64/interp_masm_aarch64.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -503,7 +503,7 @@ void InterpreterMacroAssembler::remove_activation(

// get method access flags
ldr(r1, Address(rfp, frame::interpreter_frame_method_offset * wordSize));
load_unsigned_short(r2, Address(r1, Method::access_flags_offset()));
ldrh(r2, Address(r1, Method::access_flags_offset()));
tbz(r2, exact_log2(JVM_ACC_SYNCHRONIZED), unlocked);

// Don't unlock anything if the _do_not_unlock_if_synchronized flag
Expand Down
2 changes: 1 addition & 1 deletion src/hotspot/cpu/aarch64/sharedRuntime_aarch64.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -760,7 +760,7 @@ AdapterHandlerEntry* SharedRuntime::generate_i2c2i_adapters(MacroAssembler *masm
Label L_skip_barrier;

{ // Bypass the barrier for non-static methods
__ load_unsigned_short(rscratch1, Address(rmethod, Method::access_flags_offset()));
__ ldrh(rscratch1, Address(rmethod, Method::access_flags_offset()));
__ andsw(zr, rscratch1, JVM_ACC_STATIC);
__ br(Assembler::EQ, L_skip_barrier); // non-static
}
Expand Down
16 changes: 8 additions & 8 deletions src/hotspot/cpu/aarch64/templateInterpreterGenerator_aarch64.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -809,7 +809,7 @@ void TemplateInterpreterGenerator::lock_method() {
#ifdef ASSERT
{
Label L;
__ load_unsigned_short(r0, access_flags);
__ ldrh(r0, access_flags);
__ tst(r0, JVM_ACC_SYNCHRONIZED);
__ br(Assembler::NE, L);
__ stop("method doesn't need synchronization");
Expand All @@ -820,7 +820,7 @@ void TemplateInterpreterGenerator::lock_method() {
// get synchronization object
{
Label done;
__ load_unsigned_short(r0, access_flags);
__ ldrh(r0, access_flags);
__ tst(r0, JVM_ACC_STATIC);
// get receiver (assume this is frequent case)
__ ldr(r0, Address(rlocals, Interpreter::local_offset_in_bytes(0)));
Expand Down Expand Up @@ -1225,7 +1225,7 @@ address TemplateInterpreterGenerator::generate_native_entry(bool synchronized) {

// make sure method is native & not abstract
#ifdef ASSERT
__ load_unsigned_short(r0, access_flags);
__ ldrh(r0, access_flags);
{
Label L;
__ tst(r0, JVM_ACC_NATIVE);
Expand Down Expand Up @@ -1277,7 +1277,7 @@ address TemplateInterpreterGenerator::generate_native_entry(bool synchronized) {
#ifdef ASSERT
{
Label L;
__ load_unsigned_short(r0, access_flags);
__ ldrh(r0, access_flags);
__ tst(r0, JVM_ACC_SYNCHRONIZED);
__ br(Assembler::EQ, L);
__ stop("method needs synchronization");
Expand Down Expand Up @@ -1354,7 +1354,7 @@ address TemplateInterpreterGenerator::generate_native_entry(bool synchronized) {
// pass mirror handle if static call
{
Label L;
__ load_unsigned_short(t, Address(rmethod, Method::access_flags_offset()));
__ ldrh(t, Address(rmethod, Method::access_flags_offset()));
__ tbz(t, exact_log2(JVM_ACC_STATIC), L);
// get mirror
__ load_mirror(t, rmethod, r10, rscratch2);
Expand Down Expand Up @@ -1564,7 +1564,7 @@ address TemplateInterpreterGenerator::generate_native_entry(bool synchronized) {
// do unlocking if necessary
{
Label L;
__ load_unsigned_short(t, Address(rmethod, Method::access_flags_offset()));
__ ldrh(t, Address(rmethod, Method::access_flags_offset()));
__ tbz(t, exact_log2(JVM_ACC_SYNCHRONIZED), L);
// the code below should be shared with interpreter macro
// assembler implementation
Expand Down Expand Up @@ -1695,7 +1695,7 @@ address TemplateInterpreterGenerator::generate_normal_entry(bool synchronized) {

// make sure method is not native & not abstract
#ifdef ASSERT
__ load_unsigned_short(r0, access_flags);
__ ldrh(r0, access_flags);
{
Label L;
__ tst(r0, JVM_ACC_NATIVE);
Expand Down Expand Up @@ -1751,7 +1751,7 @@ address TemplateInterpreterGenerator::generate_normal_entry(bool synchronized) {
#ifdef ASSERT
{
Label L;
__ load_unsigned_short(r0, access_flags);
__ ldrh(r0, access_flags);
__ tst(r0, JVM_ACC_SYNCHRONIZED);
__ br(Assembler::EQ, L);
__ stop("method needs synchronization");
Expand Down
4 changes: 2 additions & 2 deletions src/hotspot/cpu/ppc/interp_masm_ppc_64.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -687,8 +687,8 @@ void InterpreterMacroAssembler::unlock_if_synchronized_method(TosState state,

push(state);

// Skip if we don't have to unlock. (???is this right???)
rldicl_(R0, Raccess_flags, 64-JVM_ACC_SYNCHRONIZED_BIT, 63); // Extract bit and compare to 0.
// Skip if we don't have to unlock.
testbitdi(CCR0, R0, Raccess_flags, JVM_ACC_SYNCHRONIZED_BIT);
beq(CCR0, Lunlocked);

cmpwi(CCR0, Rdo_not_unlock_flag, 0);
Expand Down