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

Armnn 21.08 fails to build with glibc 2.34 #581

Closed
ggardet opened this issue Sep 21, 2021 · 6 comments
Closed

Armnn 21.08 fails to build with glibc 2.34 #581

ggardet opened this issue Sep 21, 2021 · 6 comments
Labels
Build issue This problem was about building ArmNN or one of its dependencies.

Comments

@ggardet
Copy link
Contributor

ggardet commented Sep 21, 2021

Armnn 21.08 fails to build with glibc 2.34:

[ 1125s] /usr/bin/c++ -DARMNNREF_ENABLED -DARMNN_SERIALIZER -DARMNN_SERIALIZER_SCHEMA_PATH=\"/home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnnSerializer/ArmnnSchema.fbs\" -DARMNN_TF_LITE_PARSER -DDYNAMIC_BACKEND_BUILD_DIR=\"/home/abuild/rpmbuild/BUILD/armnn-21.08/build\" -DWITH_VALGRIND=1 -I/home/abuild/rpmbuild/BUILD/armnn-21.08/include -I/home/abuild/rpmbuild/BUILD/armnn-21.08/profiling -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnn -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnnUtils -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/profiling -I/home/abuild/rpmbuild/BUILD/armnn-21.08/profiling/server/src/basePipeServer -I/home/abuild/rpmbuild/BUILD/armnn-21.08/profiling/common/include -I/home/abuild/rpmbuild/BUILD/armnn-21.08/profiling/server/include/timelineDecoder -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/timelineDecoder -I/home/abuild/rpmbuild/BUILD/armnn-21.08/tests/profiling/gatordmock -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends/backendsCommon/test -isystem /home/abuild/rpmbuild/BUILD/armnn-21.08/third-party -isystem /usr/include/tensorflow/lite/schema -isystem /home/abuild/rpmbuild/BUILD/armnn-21.08/build/src/armnnSerializer -O2 -Wall -D_FORTIFY_SOURCE=2 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -Werror=return-type  -g -pthread  -Wno-error=stringop-overread -Wno-error=uninitialized -Wno-error=array-bounds -Wno-error=deprecated-copy -Wno-error=deprecated-declarations -Wno-error=implicit-fallthrough -Wno-error=unused-parameter -std=c++14 -Wall -Wextra -Werror -Wold-style-cast -Wno-missing-braces -Wconversion -Wsign-conversion -O2 -g  -fPIE -MD -MT CMakeFiles/UnitTests.dir/src/armnn/test/UnitTests.cpp.o -MF CMakeFiles/UnitTests.dir/src/armnn/test/UnitTests.cpp.o.d -o CMakeFiles/UnitTests.dir/src/armnn/test/UnitTests.cpp.o -c /home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnn/test/UnitTests.cpp
[ 1125s] In file included from /home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnn/test/UnitTests.cpp:9:
[ 1125s] /home/abuild/rpmbuild/BUILD/armnn-21.08/third-party/doctest/doctest.h:4032:47: error: size of array 'altStackMem' is not an integral constant-expression
[ 1125s]  4032 |         static char             altStackMem[4 * SIGSTKSZ];
[ 1125s]       |                                               ^
[ 1125s] make[2]: *** [CMakeFiles/UnitTests.dir/build.make:714: CMakeFiles/UnitTests.dir/src/armnn/test/UnitTests.cpp.o] Error 1

This has been fixed in upstream doctest:
doctest/doctest#473
doctest/doctest@099d541

So, likely just a matter of updating the local copy.

@ggardet
Copy link
Contributor Author

ggardet commented Sep 23, 2021

If I update doctest/doctest.h to version 2.4.6, I get the following error:

[  141s] [ 16%] Building CXX object src/backends/backendsCommon/test/CMakeFiles/armnnBackendsCommonUnitTests.dir/DynamicBackendTests.cpp.o
[  141s] cd /home/abuild/rpmbuild/BUILD/armnn-21.08/build/src/backends/backendsCommon/test && /usr/bin/c++ -DARMCOMPUTECL_ENABLED -DARMNNREF_ENABLED -DARMNN_SERIALIZER -DARMNN_SERIALIZER_SCHEMA_PATH=\"/home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnnSerializer/ArmnnSchema.fbs\" -DARMNN_TF_LITE_PARSER -DDYNAMIC_BACKEND_BUILD_DIR=\"/home/abuild/rpmbuild/BUILD/armnn-21.08/build\" -I/home/abuild/rpmbuild/BUILD/armnn-21.08/include -I/home/abuild/rpmbuild/BUILD/armnn-21.08/profiling -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnn -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/armnnUtils -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends -I/home/abuild/rpmbuild/BUILD/armnn-21.08/src/profiling -I/home/abuild/rpmbuild/BUILD/armnn-21.08/profiling/common/include -isystem /home/abuild/rpmbuild/BUILD/armnn-21.08/third-party -isystem /usr/include/half -O2 -Wall -D_FORTIFY_SOURCE=2 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -Werror=return-type -flto=auto -g -pthread  -Wno-error=stringop-overread -Wno-error=uninitialized -Wno-error=array-bounds -Wno-error=deprecated-copy -Wno-error=deprecated-declarations -Wno-error=implicit-fallthrough -Wno-error=unused-parameter -std=c++14 -Wall -Wextra -Werror -Wold-style-cast -Wno-missing-braces -Wconversion -Wsign-conversion -O2 -g  -fPIC -MD -MT src/backends/backendsCommon/test/CMakeFiles/armnnBackendsCommonUnitTests.dir/DynamicBackendTests.cpp.o -MF CMakeFiles/armnnBackendsCommonUnitTests.dir/DynamicBackendTests.cpp.o.d -o CMakeFiles/armnnBackendsCommonUnitTests.dir/DynamicBackendTests.cpp.o -c /home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends/backendsCommon/test/DynamicBackendTests.cpp
[  141s] In file included from /home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends/backendsCommon/test/DynamicBackendTests.hpp:21,
[  141s]                  from /home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends/backendsCommon/test/DynamicBackendTests.cpp:6:
[  141s] /home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends/backendsCommon/test/DynamicBackendTests.hpp: In lambda function:
[  141s] /home/abuild/rpmbuild/BUILD/armnn-21.08/src/backends/backendsCommon/test/DynamicBackendTests.hpp:149:69: error: no match for 'operator+' (operand types are 'doctest::detail::MessageBuilder' and 'std::string' {aka 'std::__cxx11::basic_string<char>'})
[  141s]   149 |                     "Base path for shared objects does not exist: " + sharedObjectPath.string());
[  141s]       |                                                                     ^ ~~~~~~~~~~~~~~~~~~~~~~~~~
[  141s]       |                                                                                              |
[  141s]       |                                                                                              std::string {aka std::__cxx11::basic_string<char>}

@Colm-in-Arm
Copy link
Collaborator

Hi @ggardet

The update to 2.4.6 is in progress:
https://review.mlplatform.org/c/ml/armnn/+/6331

Colm.

@Colm-in-Arm Colm-in-Arm added the Build issue This problem was about building ArmNN or one of its dependencies. label Oct 1, 2021
@ggardet
Copy link
Contributor Author

ggardet commented Oct 1, 2021

The update to 2.4.6 is in progress: https://review.mlplatform.org/c/ml/armnn/+/6331

I will give it a try, thanks!

@ggardet
Copy link
Contributor Author

ggardet commented Oct 1, 2021

It applies properly, but I still get the error mentioned above #581 (comment)

@janeil01
Copy link
Contributor

Hey @ggardet
the patch is now merged into master if you would like to give it another go

@ggardet
Copy link
Contributor Author

ggardet commented Oct 20, 2021

Commits 13d2e0d and 4ec6d42 rebased for 21.08 fixed the build with glibc 2.34.

@ggardet ggardet closed this as completed Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build issue This problem was about building ArmNN or one of its dependencies.
Projects
None yet
Development

No branches or pull requests

3 participants