> On 21 Aug 2021, at 23:18, Eric Gallager <eg...@gwmail.gwu.edu> wrote:
>
> On Fri, Aug 20, 2021 at 3:53 AM Tobias Burnus <tob...@codesourcery.com> wrote:
>>
>> On 20.08.21 09:34, Richard Biener via Fortran wrote:
>>
>>> On Thu, Aug 19, 2021 at 10:10 PM Iain Sandoe <i...@sandoe.co.uk> wrote:
>>>> libm is not needed on Darwin, and should not be added unconditionally
>>>> even if that is (mostly) harmless since it is a symlink to libc.
>>>>
>>>> tested on x86_64, i686-darwin, x86_64-linux,
>>>> OK for master?
>>> OK.
>>> Richard.
>>
>> Looks also good to me – but for completeness and as background, I also
>> want to remark the following.
>>
>> (At least as I understand it, I did not dig deeper.)
>>
>>> --- a/libgfortran/configure.ac
>>> +++ b/libgfortran/configure.ac
>>> ...
>>> +AC_CHECK_LIBM
>>
>> This CHECK_LIBM has a hard-coded list of targets and for some (like
>> Darwin) it simply does not set $LIBM.
>
> I thought the proper macro to use was LT_LIB_M ?
you could well be right, libtool.m4 contains:
# Old name:
AU_ALIAS([AC_CHECK_LIBM], [LT_LIB_M])
I guess the patch can be changed and then do another round of testing …
… will add this to the TODO, and withdraw the current patch.
Iain
