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

(0.21.0) Implement ARM64Trg1Src2ZeroInstruction #62

Merged
merged 1 commit into from
Jun 9, 2020

Conversation

knn-k
Copy link
Contributor

@knn-k knn-k commented Jun 9, 2020

This commit implements ARM64Trg1Src2ZeroInstruction class so that we can
stop setting register dependencies on the zero register in generating
instructions for integer multiply.

This commit also contains the following changes:

Original PRs for master: eclipse-omr/omr#5289, eclipse-omr/omr#5290

Signed-off-by: KONNO Kazuhiro konno@jp.ibm.com

This commit implements ARM64Trg1Src2ZeroInstruction class so that we can
stop setting register dependencies on the zero register in generating
instructions for integer multiply.

This commit also contains the following changes:

- Implement and use insertZeroRegister() functions in classes introduced by eclipse-openj9#61
- Remove the code for printing "cmp" from ARM64Trg1Src2Instruction
(It was moved to ARM64ZeroSrc2Instruction)

Original PRs for master: eclipse-omr/omr#5289, eclipse-omr/omr#5290

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@0xdaryl
Copy link
Contributor

0xdaryl commented Jun 9, 2020

This is an AArch64-specific change. It was committed to OMR prior to the OpenJ9 split for 0.21.0, but because the OMR Acceptance build was broken it did not promote and wasn't part of the split.

@0xdaryl 0xdaryl merged commit 55b8417 into eclipse-openj9:v0.21.0-release Jun 9, 2020
@knn-k knn-k deleted the aarch64multInstr branch June 9, 2020 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants