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
