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.
Many thanks, Mt 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 -- If the facts don't fit the theory, change the facts. -- Albert Einstein (?)
signature.asc
Description: PGP signature