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

Fully distinguish body info address relocations #17227

Merged
merged 1 commit into from Apr 19, 2023

Conversation

cjjdespres
Copy link
Contributor

Since merging the TR_BodyInfoAddress and TR_BodyInfoAddressLoad relocations is not worth the effort at the moment, the documentation should at least point out exactly why both exist.

Since merging the TR_BodyInfoAddress and TR_BodyInfoAddressLoad
relocations is not worth the effort at the moment, the documentation
should at least point out exactly why both exist.

Signed-off-by: Christian Despres <despresc@ibm.com>
@cjjdespres
Copy link
Contributor Author

Attn @mpirvu. In #17153 we discussed exactly why both relocations need to exist. The documentation should now reflect that.

Copy link
Contributor

@mpirvu mpirvu left a comment

Choose a reason for hiding this comment

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

LGTM

@mpirvu mpirvu self-assigned this Apr 19, 2023
@mpirvu mpirvu merged commit ea2117e into eclipse-openj9:master Apr 19, 2023
2 checks passed
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.

None yet

2 participants