-
Notifications
You must be signed in to change notification settings - Fork 99
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.
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
8244975: [lworld] Investigate if VM can implement Object::hashCode by calling ValueBootstrapMethods for inline types #136
Conversation
… calling ValueBootstrapMethods for inline types
👋 Welcome back dsimms! A progress list of the required criteria for merging this PR into |
@MrSimms This change now passes all automated pre-integration checks, type
Since the source branch of this PR was last updated there have been 5 commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid automatic rebasing, please merge ➡️ To integrate this PR with the above commit message to the |
Webrevs
|
src/java.base/share/classes/java/lang/invoke/ValueBootstrapMethods.java
Outdated
Show resolved
Hide resolved
src/java.base/share/classes/java/lang/invoke/ValueBootstrapMethods.java
Outdated
Show resolved
Hide resolved
src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/Gen.java
Outdated
Show resolved
Hide resolved
src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TypeEnter.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 for javac changes, assuming tests are all green
There will be follow on work taken up independently for JDK-8244560 - There is still some discussion that needs to happen for the right behavior when an inline type also happens to be a record, but that can be tackled separately as a javac work item |
Looks like I did break something: rather than .out file:
Not sure I completely understand the test...ahh, yeah I do, no longer generate hashCode, but toString was |
This looks good to me. I made a small clean up on javac |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved
/integrate |
@MrSimms The following commits have been pushed to lworld since your change was applied:
Your commit was automatically rebased without conflicts. Pushed as commit f891f43. |
Use ValueBootstrapMethods for Object::hashCode()
Progress
Issue
Reviewers
Download
$ git fetch https://git.openjdk.java.net/valhalla pull/136/head:pull/136
$ git checkout pull/136