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

Add an entry to support newvalue into CodeGenGPU #6569

Merged
merged 1 commit into from
Oct 4, 2019

Conversation

Charlmzz
Copy link
Contributor

@Charlmzz Charlmzz commented Jul 24, 2019

Add an entry to support newvalue into CodeGenGPU.

Issue: #4028

Signed-off-by: Yiling Han Yiling.Han@ibm.com

The change supports the newvalue opcode.

Issue: eclipse-openj9#4028

Signed-off-by: Yiling Han <Yiling.Han@ibm.com>
@andrewcraik
Copy link
Contributor

This must depend on an OMR change? If so can you please identify which one?

@andrewcraik
Copy link
Contributor

Could you also please make the PR title and message a bit more meaningful so that one doesn't have to read the commit?

@Charlmzz
Copy link
Contributor Author

The change depends on eclipse-omr/omr#4156.

@Charlmzz Charlmzz changed the title To support the newvalue opcode Add an entry to support newvalue into CodeGenGPU Jul 24, 2019
@andrewcraik andrewcraik added comp:jit depends:omr Pull request is dependent on a corresponding change in OMR enhancement labels Jul 24, 2019
@Charlmzz
Copy link
Contributor Author

Updated for review.

@andrewcraik
Copy link
Contributor

This is a simple add and the sanity won't work with the OMR now out of whack so I'm going to merge this now to get an acceptance build going. I have checked the change it is a single line addition at the correct point in the opcode table to make the opcodes in OpenJ9 line up with OMR once again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:jit depends:omr Pull request is dependent on a corresponding change in OMR enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants