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

Use assertion with message to simplify debugging #16507

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

dmitripivkine
Copy link
Contributor

@dmitripivkine dmitripivkine commented Jan 3, 2023

Currently regular assertion is used in
MM_GCExtensions::releaseNativesForContinuationObject(). Replace it to assertion with message to simplify debugging process: print Continuation object address as well as address of J9VMContinuation structure.

Signed-off-by: Dmitri Pivkine Dmitri_Pivkine@ca.ibm.com

@dmitripivkine
Copy link
Contributor Author

Jenkins signed off by check

@amicic
Copy link
Contributor

amicic commented Jan 3, 2023

Jenkins compile win,aix jdk19

@dmitripivkine
Copy link
Contributor Author

lost test build results in attempts to re-run ip-validation, all tests have passed

Currently regular assertion is used in
MM_GCExtensions::releaseNativesForContinuationObject(). Replace it to
assertion with message to simplify debugging process: print Continuation
object address as well as address of J9VMContinuation structure.

Signed-off-by: Dmitri Pivkine <Dmitri_Pivkine@ca.ibm.com>
@amicic
Copy link
Contributor

amicic commented Jan 4, 2023

lost test build results in attempts to re-run ip-validation, all tests have passed

Specifically:
https://openj9-jenkins.osuosl.org/job/Build_JDK19_ppc64_aix_Personal/58/
https://openj9-jenkins.osuosl.org/job/Build_JDK19_x86-64_windows_Personal/231/

@amicic amicic merged commit c9907a0 into eclipse-openj9:master Jan 4, 2023
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