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

Implement proper RX-b instruction support on Z #3900

Open
fjeremic opened this issue May 24, 2019 · 2 comments
Open

Implement proper RX-b instruction support on Z #3900

fjeremic opened this issue May 24, 2019 · 2 comments

Comments

@fjeremic
Copy link
Contributor

As part of #3693 we've put in a fix in a6c0e45 to change the format of the BC instruction which whose first usage was introduced in #3779 to an RX-a instruction. The reason this was done is to work around the issue that we currently do not have a proper instruction class to support RX-b instructions.

The task here is to effectively revert a6c0e45 and implement proper support for RX-b instructions in [1] then make use of the new overload within the linkage class where the (currently) only use of the BC instruction is.

[1] https://github.com/eclipse/omr/blob/051896be071812db8a3b71485b014be26c80e7f4/compiler/z/codegen/S390Instruction.hpp#L4422-L4489

@github-actions
Copy link

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment on the issue or it will be closed in 60 days.

@github-actions github-actions bot added the stale label May 25, 2020
@fjeremic
Copy link
Contributor Author

Placing this on the backlog as it is something we want to address still.

@fjeremic fjeremic added backlog and removed stale labels May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant