Hi, On 24/08/17 08:59, Martin Quinson wrote: > Hello, > > thanks for the patch. I came to something very similar, only > protecting it to be used only by GCC. I was waiting to find the time > to test it before upload, but if you tested, I'll upload it right away.
I have tested the patch I attached on a mips machine. If you want I could test your alternative patch as well. Thanks, James > On Wed, Aug 23, 2017 at 02:41:15PM +0100, James Cowgill wrote: >> Hi, >> >> The attached patch seems to work (ie replace atomic-library with it). >> Using target_link_libraries should put any linker arguments at the end >> instead of the beginning. >> >> Thanks, >> James > >> --- a/tools/cmake/MakeLib.cmake >> +++ b/tools/cmake/MakeLib.cmake >> @@ -114,6 +114,7 @@ if(enable_smpi AND APPLE) >> set(SIMGRID_DEP "${SIMGRID_DEP} -Wl,-U -Wl,_smpi_simulated_main") >> endif() >> >> +set(SIMGRID_DEP "${SIMGRID_DEP} -Wl,--as-needed -latomic >> -Wl,--no-as-needed") >> target_link_libraries(simgrid ${SIMGRID_DEP}) >> >> # Dependencies from maintainer mode
signature.asc
Description: OpenPGP digital signature