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 compile error introduced in #21153 #21159

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

keithc-ca
Copy link
Contributor

Compilers on platforms that don't use GCC (e.g. Windows) were failing.

@keithc-ca keithc-ca requested a review from amicic February 20, 2025 19:56
@amicic
Copy link
Contributor

amicic commented Feb 20, 2025

while there you can use your macro at line 118 and 132, and remove the declarations of those methods, so the solution is consistent

@amicic
Copy link
Contributor

amicic commented Feb 20, 2025

@TobiAjila you did not have a problem with unused j9javaArray_BA ?

edit: I guess, it is used by j9javaArray_##elemType##_EA (although effectively either both are used or unused)

@keithc-ca
Copy link
Contributor Author

use your macro at line 118 and 132, and remove the declarations of those methods, so the solution is consistent

Good idea. I'll update the change accordingly.

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]>
@keithc-ca
Copy link
Contributor Author

Updated. The declarations needed to stay because GCC complains if __attribute__ ((__unused__)) is applied to a function definition.

@amicic
Copy link
Contributor

amicic commented Feb 20, 2025

Updated. The declarations needed to stay because GCC complains if __attribute__ ((__unused__)) is applied to a function definition.

but it is successfully applied at the definition of j9javaArray_##elemType##_EA

@keithc-ca keithc-ca marked this pull request as draft February 20, 2025 20:50
@keithc-ca
Copy link
Contributor Author

In draft mode until IBM Java 8 builds work also.

@keithc-ca
Copy link
Contributor Author

#21153 has been reverted; I'll see if I can rework this to achieve the goal of that change while keeping all compilers happy.

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