-
Notifications
You must be signed in to change notification settings - Fork 733
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
Set aladd/aiadd as internalPtr in calculateElementAddress #13294
Set aladd/aiadd as internalPtr in calculateElementAddress #13294
Conversation
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 to me.
@0xdaryl Ready for review. Thanks Daryl! |
Jenkins test sanity all jdk11,jdk16 |
Test_openjdk11_j9_sanity.functional_x86-64_mac_Personal failed due to a crash in
|
Signed-off-by: Annabelle Huo <[email protected]>
6ce3054
to
cd7ef40
Compare
@0xdaryl Could you help relaunch PR build test? The above failure was due to known issues. I also rebased this PR to the latest master. |
Jenkins test sanity all jdk17 |
PR builds aren't going to work until artifactory is fixed. |
Jenkins test sanity all jdk17 |
The macOS crash in the interpreter is real and not one I've seen before. I don't believe your changes are responsible, but an issue should be opened nonetheless and the failing test re-run:
|
The crash from Test_openjdk17_j9_sanity.functional_x86-64_mac_Personal 23 is in
|
Jenkins test sanity osx jdk17 |
Set
aladd
/aiadd
as internal pointer incalculateElementAddress
Signed-off-by: Annabelle Huo [email protected]