[PATCH] libgfortran : Use the libtool macro to determine libm availability.

Iain Sandoe iain@sandoe.co.uk
Sun Aug 22 08:44:06 GMT 2021



> On 21 Aug 2021, at 23:18, Eric Gallager <egall@gwmail.gwu.edu> wrote:
> 
> On Fri, Aug 20, 2021 at 3:53 AM Tobias Burnus <tobias@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 <iain@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



More information about the Gcc-patches mailing list