-
Notifications
You must be signed in to change notification settings - Fork 721
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
x86: Preserve xmm4 in fpRem helper #17157
Conversation
@hzongaro Would you mind reviewing? |
Brad @BradleyWood, I haven't ever needed to look in detail at the various linkage conventions. Can you point me to the description of the linkage convention that would apply for this helper? I just wanted to understand why the helper should be responsible for preserving |
In an off-line discussion, Daryl @0xdaryl indicated that the callee should preserve that register, so I'll proceed with the review. |
Fixes eclipse-openj9#17129 Signed-off-by: BradleyWood <[email protected]>
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.
Looks good. Thanks!
Jenkins test sanity xlinux,win,osx jdk11,jdk17 |
Jenkins test sanity xlinux jdk17 |
Testing has completed successfully. Merging. |
Fixes #17129