[clang] c149ff3 - [Driver] Link Flang runtime on FreeBSD, NetBSD, OpenBSD, DragonFly and Haiku (#69817)

Joerg Sonnenberger via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 27 04:22:36 PDT 2023


On Friday, October 27, 2023 10:43:23 AM CEST via cfe-commits wrote:
> Author: Brad Smith
> Date: 2023-10-27T04:43:19-04:00
> New Revision: c149ff3d372a70e7d9de838a1ff0357394b8d017
> 
> URL:
> https://github.com/llvm/llvm-project/commit/c149ff3d372a70e7d9de838a1ff0357
> 394b8d017 DIFF:
> https://github.com/llvm/llvm-project/commit/c149ff3d372a70e7d9de838a1ff0357
> 394b8d017.diff
> 
> LOG: [Driver] Link Flang runtime on FreeBSD, NetBSD, OpenBSD, DragonFly and
> Haiku (#69817)
> 
> Added:
> 
> 
> Modified:
>     clang/lib/Driver/ToolChains/DragonFly.cpp
>     clang/lib/Driver/ToolChains/FreeBSD.cpp
>     clang/lib/Driver/ToolChains/Haiku.cpp
>     clang/lib/Driver/ToolChains/NetBSD.cpp
>     clang/lib/Driver/ToolChains/OpenBSD.cpp
>     flang/test/Driver/linker-flags.f90
> 
> Removed:
> 
> 
> 
> ############################################################################
> #### diff  --git a/clang/lib/Driver/ToolChains/DragonFly.cpp
> b/clang/lib/Driver/ToolChains/DragonFly.cpp index
> 9dc8d9d4363bd71..500dd98665075b1 100644
> --- a/clang/lib/Driver/ToolChains/DragonFly.cpp
> +++ b/clang/lib/Driver/ToolChains/DragonFly.cpp
> @@ -144,6 +144,16 @@ void dragonfly::Linker::ConstructJob(Compilation &C,
> const JobAction &JA, CmdArgs.push_back("-lm");
>      }
> 
> +    // Additional linker set-up and flags for Fortran. This is required in
> order +    // to generate executables. As Fortran runtime depends on the C
> runtime, +    // these dependencies need to be listed before the C runtime
> below (i.e. +    // AddRuntTimeLibs).
> +    if (D.IsFlangMode()) {
> +      addFortranRuntimeLibraryPath(ToolChain, Args, CmdArgs);
> +      addFortranRuntimeLibs(ToolChain, CmdArgs);
> +      CmdArgs.push_back("-lm");
> +    }
> +
>      if (Args.hasArg(options::OPT_pthread))
>        CmdArgs.push_back("-lpthread");
> 
> 

Shouldn't that be "AddRunTimeLibs" without the extra t?

Joerg




More information about the cfe-commits mailing list