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

Relocate the CodeGenerator::createOrFindClonedNode function from OMR #5790

Merged
merged 1 commit into from
May 22, 2019

Conversation

wbh123456
Copy link
Contributor

@wbh123456 wbh123456 commented May 16, 2019

TR::Node *createOrFindClonedNode function in OMR::CodeGenerator has only relevance to the OpenJ9 project. Therefore, remove it from OMR and relocate it to the J9::CodeGenerator instead. In addition, fix the spelling mistake on _uncommonedNodes and relocate it from OMR to J9.

Issue: eclipse-omr/omr#2068
Signed-off-by: Bohao(Aaron) Wang [email protected]

@0xdaryl
Copy link
Contributor

0xdaryl commented May 16, 2019

Jenkins test sanity all jdk11

@0xdaryl 0xdaryl self-assigned this May 16, 2019
@0xdaryl
Copy link
Contributor

0xdaryl commented May 17, 2019

I suggest you move the _uncommmonedNodes field to OpenJ9 as part of this PR and fix the spelling mistake in the process. You're essentially introducing a new field in OpenJ9 with this corrected spelling. Once this is merged, you can delete the field (and the function) from OMR.

@0xdaryl
Copy link
Contributor

0xdaryl commented May 17, 2019

A couple of administrative nodes:

  1. This PR doesn't depend on OMR 3857 does it? Remove the "depends" tag if so
  2. This PR doesn't close OMR 2068 (OMR 3857 will close it). Rather than saying "Closes", say "Issue" to refer to it.

TR::Node *createOrFindClonedNode function in OMR::CodeGenerator has only relevance to the
OpenJ9 project. Therefore, remove it from OMR and relocate it to the
J9::CodeGenerator instead. In addition, fix the spelling mistake on
_uncommonedNodes and relocate it from OMR to J9.

Signed-off-by: Bohao(Aaron) Wang <[email protected]>
@wbh123456 wbh123456 force-pushed the relocatecreateorfind branch from 5347aef to deab173 Compare May 21, 2019 14:01
@wbh123456 wbh123456 changed the title Relocate the CodeGenerator::createOrClonedNode function from OMR Relocate the CodeGenerator::createOrFindClonedNode function from OMR May 21, 2019
@0xdaryl
Copy link
Contributor

0xdaryl commented May 21, 2019

Jenkins test sanity all jdk11

@0xdaryl
Copy link
Contributor

0xdaryl commented May 22, 2019

AIX builds are stalled due to infra issues. Everything else has passed. I'm satisfied with the testing. Merging.

@0xdaryl 0xdaryl merged commit e2da323 into eclipse-openj9:master May 22, 2019
@wbh123456 wbh123456 deleted the relocatecreateorfind branch May 22, 2019 18:51
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.

2 participants