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 implicit conversion causing truncation warnings #18633

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

0xdaryl
Copy link
Contributor

@0xdaryl 0xdaryl commented Dec 15, 2023

Fix AIX/pLinux, Arm, and AArch64 warnings concerning constant values being implicitly converted from an unsigned to a signed type or from a larger to a smaller type which had the potential to cause truncation.

@0xdaryl 0xdaryl requested a review from dsouzai as a code owner December 15, 2023 12:26
@0xdaryl
Copy link
Contributor Author

0xdaryl commented Dec 15, 2023

@hzongaro @dylanjtuttle : successfully completed internal testing of OMR Acceptance tests for JDK21 on AIX, pLinux, M-series macOS, and AArch64 Linux.

@hzongaro hzongaro self-assigned this Dec 18, 2023
Fix AIX/pLinux, Arm, and AArch64 warnings concerning constant values
being implicitly converted from an unsigned to a signed type or from
a larger to a smaller type which had the potential to cause truncation.

Fixes eclipse-openj9#18626

Co-authored-by: Dylan Tuttle <[email protected]>
Co-authored-by: Daryl Maier <[email protected]>
Copy link
Member

@hzongaro hzongaro left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks!

@hzongaro
Copy link
Member

Jenkins test sanity aix,plinux,alinux64,amac jdk8,jdk11,jdk17,jdk21

@0xdaryl
Copy link
Contributor Author

0xdaryl commented Dec 20, 2023

All the JDK 21 AArch64 CRIU failures appear to be known issues on ub22-aarch64-1. I will re-launch the testing

@0xdaryl
Copy link
Contributor Author

0xdaryl commented Dec 20, 2023

The re-run of the tests was successful -> https://openj9-jenkins.osuosl.org/job/Test_openjdk21_j9_sanity.functional_aarch64_linux_Personal_testList_1/88/

There are no outstanding issues with this PR testing.

@hzongaro hzongaro merged commit 0001f68 into eclipse-openj9:master Dec 20, 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.

3 participants