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.

Sign up for GitHub

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

Jump to bottom

Build tests without -O0 also in the autoconf build #95

Open

AdrianBunk wants to merge 1 commit into edrosten:main from AdrianBunk:main

AdrianBunk commented Aug 3, 2024

C++ code compiled without optimizations can be inefficient in many ways, and on some more exotic architectures (MIPS, Alpha) this caused a build failure:
https://buildd.debian.org/status/fetch.php?pkg=libcvd&arch=mips64el&ver=0.0%7Egit20221020150751.30e8cfc%2Bds1-1&stamp=1695631656&raw=0
https://buildd.debian.org/status/fetch.php?pkg=libcvd&arch=alpha&ver=0.0%7Egit20221020150751.30e8cfc%2Bds1-1&stamp=1722259419&raw=0

/usr/bin/ld: /tmp/cc5JDH6b.o: .got subsegment exceeds 64K (size 86336)
/usr/bin/ld: failed to set dynamic section sizes: No such file or directory
collect2: error: ld returned 1 exit status
make[1]: *** [Makefile:298: tests/load_and_save.test] Error 1

CMake is already using the normal optimization flags also for building the tests.


          Build tests without -O0 also in the autoconf build

ab86fe7

edrosten approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet