-
Notifications
You must be signed in to change notification settings - Fork 738
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
base: master
Are you sure you want to change the base?
Conversation
while there you can use your macro at line 118 and 132, and remove the declarations of those methods, so the solution is consistent |
@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) |
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]>
Updated. The declarations needed to stay because GCC complains if |
but it is successfully applied at the definition of j9javaArray_##elemType##_EA |
In draft mode until IBM Java 8 builds work also. |
#21153 has been reverted; I'll see if I can rework this to achieve the goal of that change while keeping all compilers happy. |
Compilers on platforms that don't use GCC (e.g. Windows) were failing.