Skip to content
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

Fix cast accessing ramConstantPool #14061

Closed
wants to merge 1 commit into from

Conversation

keithc-ca
Copy link
Contributor

Addresses this comment.

Be careful to avoid sign-extending the address.

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
@keithc-ca
Copy link
Contributor Author

Marked as draft - may not be needed since #13836 was reverted and its replacement should incorporate this.

@EricYangIBM
Copy link
Contributor

@EricYangIBM
Copy link
Contributor

EricYangIBM commented Dec 2, 2021

Seems like all tests passed. New PR: #14063

Actually seems like one failure: job_output.php?id=21708291
Is this a known issue?

@keithc-ca
Copy link
Contributor Author

This fix was included in #14063: closing.

@keithc-ca keithc-ca closed this Dec 7, 2021
@keithc-ca keithc-ca deleted the cast branch January 21, 2022 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants