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

Fixing errors present when creating a build with PROD_WITH_ASSUMES enabled #15389

Merged
merged 2 commits into from
Jun 22, 2022

Conversation

manashaVetrivelu
Copy link
Contributor

@manashaVetrivelu manashaVetrivelu commented Jun 21, 2022

Changed 'TR::ILOpCode::NumAllIlOps' to 'TR::NumAllIlOps'.
Casted the argument so the type is void*.

Signed-off-by: Manasha Vetrivelu manasha.vetrivelu@ibm.com

Changed 'TR::ILOpCode::NumAllIlOps' to 'TR::NumAllIlOps'

Signed-off-by: Manasha Vetrivelu <manasha.vetrivelu@ibm.com>
Casted the argument so the type is void*

Signed-off-by: Manasha Vetrivelu <manasha.vetrivelu@ibm.com>
@manashaVetrivelu manashaVetrivelu changed the title Format '%p' expects argument of type 'void*' Fixing errors present when creating a build with PROD_WITH_ASSUMES enabled Jun 21, 2022
@mpirvu mpirvu self-assigned this Jun 21, 2022
Copy link
Contributor

@mpirvu mpirvu left a comment

Choose a reason for hiding this comment

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

LGTM

@mpirvu
Copy link
Contributor

mpirvu commented Jun 21, 2022

jenkins compile all jdk17

@mpirvu mpirvu merged commit e8bf271 into eclipse-openj9:master Jun 22, 2022
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.

None yet

2 participants