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

arm64: error: variable 'r3' set but not used [-Werror,-Wunused-but-set-variable] #245

Closed
yurivict opened this issue Mar 20, 2022 · 7 comments

Comments

@yurivict
Copy link

yurivict commented Mar 20, 2022

Build fails on arm64:

cc -Ilibcglm.so.0.8.4.p -I. -I.. -fno-color-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -O2 -pipe -fstack-protector-strong -fno-strict-aliasing -fPIC -MD -MQ libcglm.so.0.8.4.p/src_affine2d.c.o -MF libcglm.so.0.8.4.p/src_affine2d.c.o.d -o libcglm.so.0.8.4.p/src_affine2d.c.o -c ../src/affine2d.c
FAILED: libcglm.so.0.8.4.p/src_affine2d.c.o 
cc -Ilibcglm.so.0.8.4.p -I. -I.. -fno-color-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -O2 -pipe -fstack-protector-strong -fno-strict-aliasing -fPIC -MD -MQ libcglm.so.0.8.4.p/src_affine2d.c.o -MF libcglm.so.0.8.4.p/src_affine2d.c.o.d -o libcglm.so.0.8.4.p/src_affine2d.c.o -c ../src/affine2d.c
In file included from ../src/affine2d.c:8:
In file included from ../src/../include/cglm/cglm.h:18:
In file included from ../src/../include/cglm/affine.h:41:
In file included from ../src/../include/cglm/affine-mat.h:30:
./src/../include/cglm/simd/neon/affine.h:89:29: error: variable 'r3' set but not used [-Werror,-Wunused-but-set-variable]
  glmm_128      r0, r1, r2, r3, x0;
                            ^
1 error generated.

Version: 0.8.4
clang-12
OS: FreeBSD 13

@recp
Copy link
Owner

recp commented Mar 21, 2022

Hi @yurivict

Thanks for reporting this,

THe issue must. be fixed at #244, can you pull the latest commit or version please?

@yurivict
Copy link
Author

THe issue must. be fixed at #244, can you pull the latest commit or version please?

Thanks.

Since I don't have any ARM systems (the failure was in the central build) - I'd wait for the next release.

@recp
Copy link
Owner

recp commented Mar 21, 2022

The next release of cglm was landed yesterday: https://github.com/recp/cglm/releases ☺️

@yurivict
Copy link
Author

If the fix is included - I'd see if fallout messages would keep coming.

@RobertLemmens
Copy link

I was using a slighty older version of cglm and ran into this. Update the submodule to latest master and build works again. Thanks maintainers!

@yurivict
Copy link
Author

@recp There are no more failures on arm with 0.8.5.

@recp
Copy link
Owner

recp commented Jun 15, 2022

@RobertLemmens @yurivict thanks 🚀

@recp recp closed this as completed Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants