Skip to content

8370450: [lworld] Alternate implementation of the substitutability test method #1695

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

8370450: [lworld] Alternate implementation of the substitutability test method #1695

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
23c2ccd
Add alternate version of the substitutability method
fparain Oct 22, 2025
893afdc
Merge remote-tracking branch 'upstream/lworld' into new_acmp
fparain Oct 22, 2025
c4d5cf4
Apply changes suggested in review
fparain Oct 23, 2025
22b7404
Merge remote-tracking branch 'upstream/lworld' into new_acmp
fparain Oct 23, 2025
661ffa7
More fixes uggested in reviews
fparain Oct 24, 2025
cf456a9
Remove useless check
fparain Oct 24, 2025
f5f7448
Add comment
fparain Oct 27, 2025
e9e79fe
Add comment
fparain Oct 27, 2025
52959d0
Field to field map must be final
fparain Oct 31, 2025
99f2114
Merge remote-tracking branch 'upstream/lworld' into new_acmp
fparain Oct 31, 2025
899f480
Fix behavior when alt method is not used
fparain Nov 4, 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
Prev Previous commit
Next Next commit
Apply changes suggested in review
  • Loading branch information
@fparain
fparain committed Oct 23, 2025
commit c4d5cf453886a5acecb613c074331badb559601e
2 changes: 1 addition & 1 deletion src/hotspot/share/classfile/fieldLayoutBuilder.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -286,7 +286,7 @@ class FieldLayoutBuilder : public ResourceObj {
FieldGroup* _static_fields;
FieldLayout* _layout;
FieldLayout* _static_layout;
GrowableArray<Pair<int,int>>* _nonoop_acmp_map ;
GrowableArray<Pair<int,int>>* _nonoop_acmp_map;
GrowableArray<int>* _oop_acmp_map;
int _nonstatic_oopmap_count;
int _payload_alignment;
Expand Down
5 changes: 4 additions & 1 deletion src/hotspot/share/oops/instanceKlass.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -647,7 +647,10 @@ class InstanceKlass: public Klass {
// reference type
ReferenceType reference_type() const { return (ReferenceType)_reference_type; }

int acmp_maps_offset() const { return _acmp_maps_offset; }
int acmp_maps_offset() const {
assert(_acmp_maps_offset != 0, "Not initialized");
return _acmp_maps_offset;
}
void set_acmp_maps_offset(int offset) { _acmp_maps_offset = offset; }

// this class cp index
Expand Down