Linker issue on mipsel and armel architectures

[pgrt]

Hello,

In Debian, the build is failing on armel and mipsel with messages like
/usr/bin/ld: /usr/include/c++/11/bits/atomic_base.h:488: undefined reference to `__atomic_load_8'

I could solve this with the attached patch. Yet the issue is not showing up on other architectures: maybe you won't want to take it into account.

Best,
Pierre

linker_flags_for_armel_mipsel.txt

[bluescarni]

@pgrt thanks for the report and apologies for the delay.

This seems the kind of things that would be better done on a case-by-case basis, rather than hard-coding it in the build system. Could you try to see if you can manage to achieve the same effect by using environment variables instead?

[bluescarni]

I mean something like CFLAGS, CXXFLAGS, LDFLAGS and similar.

[pgrt]

Hello,

No worry about the delay, really!
Thanks for the suggestions, I will give it a try very soon and have you informed!

Best,
Pierre

[pgrt]

Hello again,
As you suggested, adding
-Wl,--push-state,--no-as-needed,-latomic,--pop-state
to LDFLAGS allows to build successfully on mipsel, without patching the CMakeLists.txt!

Best,
Pierre

[bluescarni]

Hello again, As you suggested, adding -Wl,--push-state,--no-as-needed,-latomic,--pop-state to LDFLAGS allows to build successfully on mipsel, without patching the CMakeLists.txt!

Best, Pierre

Great to hear, I'll close the issue report then.

[josch]

Hi @pgrt -- I recently got to the bottom of the same undefined __atomic_load_8 linking error in vcmi which, like pagmo, also uses libtbb. Maybe pagmo, which also is a CMake project experiences the same issue? I reported it here: uxlfoundation/oneTBB#1454

[pgrt]

HI @josch and thanks for having me informed! I saw your report and the discussions that held afterwards. let's see if a solution can come from libtbb then;

Best wishes,
Pierre