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

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
Merge remote-tracking branch 'upstream/lworld' into new_acmp
  • Loading branch information
@fparain
fparain committed Oct 22, 2025
commit 893afdc6d65864bf6069aa1bf6aa421c51848ff6
3 changes: 0 additions & 3 deletions src/hotspot/share/runtime/globals.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2050,9 +2050,6 @@ const int ObjectAlignmentInBytes = 8;
"Minimal number of elements in a sorted collection to prefer" \
"binary search over simple linear search." ) \
\
product(bool, UseClassMetaspaceForAllClasses, false, DIAGNOSTIC, \
"Use the class metaspace for all classes including " \
"abstract and interface classes.") \
product(bool, UseAltSubstitutabilityMethod, false, \
Copy link
Member

@liach liach Oct 22, 2025

Choose a reason for hiding this comment

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

We might need this flag on/off for core-libs tests in addition to runtime tests.

"Use alternate version of the isSubstitutable method to " \
"compare value class instances") \
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.