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

Fix value Type generator #4656

Merged
merged 1 commit into from
Feb 12, 2019
Merged

Fix value Type generator #4656

merged 1 commit into from
Feb 12, 2019

Conversation

tajila
Copy link
Contributor

@tajila tajila commented Feb 7, 2019

Fix value Type generator

This PR contains the changes in A. Crowthers PR in #3960 minus
the additional tests for the verifier changes.

Since the vm verifier changes are not in yet those will come at a
later time.

Signed-off-by: tajila atobia@ca.ibm.com

@tajila
Copy link
Contributor Author

tajila commented Feb 7, 2019

@llxia Please review these changes

@llxia
Copy link
Contributor

llxia commented Feb 8, 2019

Please fix the copyright issue. Thanks.

@tajila
Copy link
Contributor Author

tajila commented Feb 8, 2019

@llxia I have fixed the copywrites

@llxia
Copy link
Contributor

llxia commented Feb 8, 2019

@AdamBrousseau Is there any update on Valhalla PR build? Thanks

This PR contains the changes in A. Crowthers PR in eclipse-openj9#3960 minus 
the additional tests for the verifier changes.

Since the vm verifier changes are not in yet those wll come at a 
later time.

Signed-off-by: tajila <atobia@ca.ibm.com>
Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia
Copy link
Contributor

llxia commented Feb 12, 2019

Thanks Tobi for sending me the internal build link.
Will forgo the PR build as it will not build/execute Valhalla tests.

@llxia llxia merged commit c37238a into eclipse-openj9:master Feb 12, 2019
@AdamBrousseau
Copy link
Contributor

No update from me on Valhalla builds. I asked Tobi to open an issue so we can prioritise. It will be blocked by #2836 too.

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.

4 participants