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 unused macro to access barrier helpers #21153

Merged
merged 1 commit into from
Feb 20, 2025
Merged

Conversation

tajila
Copy link
Contributor

@tajila tajila commented Feb 20, 2025

When compiling on GCC with O0 (no optimization) it fails with a warning that 9javaArray_##elemType##_EA functions are defined but unused.

This change enables compilation with no optimization to succeed.

When compiling on GCC with O0 (no optimization) it fails with a warning
that `9javaArray_##elemType##_EA` functions are defined but unused.

This change enables compilation with no optimization to succeed.

Signed-off-by: Tobi Ajila <[email protected]>
@tajila tajila requested a review from amicic February 20, 2025 14:32
@tajila
Copy link
Contributor Author

tajila commented Feb 20, 2025

jenkins test sanity xlinux jdk17

1 similar comment
@tajila
Copy link
Contributor Author

tajila commented Feb 20, 2025

jenkins test sanity xlinux jdk17

@tajila
Copy link
Contributor Author

tajila commented Feb 20, 2025

jenkins compile xlinux jdk17

@tajila tajila marked this pull request as ready for review February 20, 2025 16:52
@tajila
Copy link
Contributor Author

tajila commented Feb 20, 2025

@amicic Please take a look

@amicic amicic merged commit acb2124 into eclipse-openj9:master Feb 20, 2025
4 checks passed
@keithc-ca
Copy link
Contributor

This seems to break windows builds; see ibmruntimes/openj9-openjdk-jdk#943.
I expect the additional __attribute__ ((__unused__)) needs to be conditional like earlier in the header file.

keithc-ca added a commit to keithc-ca/openj9 that referenced this pull request Feb 20, 2025
Compilers on platforms that don't use GCC (e.g. Windows) were failing.

Signed-off-by: Keith W. Campbell <[email protected]>
keithc-ca added a commit to keithc-ca/openj9 that referenced this pull request Feb 20, 2025
Compilers on platforms that don't use GCC (e.g. Windows) were failing.

Reuse GNU_UNUSED where appropriate.

Signed-off-by: Keith W. Campbell <[email protected]>
@pshipton
Copy link
Member

Reverted via #21163, see the description for the reason.

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.

4 participants