r257175 - Driver: Use the new ELF lld linker for AMDGPU

[Rafael Espíndola via cfe-commits]

awesome!

On 8 January 2016 at 10:14, Tom Stellard via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: tstellar
> Date: Fri Jan  8 09:14:31 2016
> New Revision: 257175
>
> URL: http://llvm.org/viewvc/llvm-project?rev=257175&view=rev
> Log:
> Driver: Use the new ELF lld linker for AMDGPU
>
> Summary: 'gnu-old' has been deprecated in favor or 'gnu'.
>
> Reviewers: arsenm, ruiu, rafael
>
> Subscribers: cfe-commits
>
> Differential Revision: http://reviews.llvm.org/D15953
>
> Modified:
>     cfe/trunk/lib/Driver/Tools.cpp
>     cfe/trunk/lib/Driver/Tools.h
>     cfe/trunk/test/Driver/amdgpu-toolchain.c
>
> Modified: cfe/trunk/lib/Driver/Tools.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=257175&r1=257174&r2=257175&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Driver/Tools.cpp (original)
> +++ cfe/trunk/lib/Driver/Tools.cpp Fri Jan  8 09:14:31 2016
> @@ -6520,10 +6520,6 @@ void amdgpu::Linker::ConstructJob(Compil
>
>    std::string Linker = getToolChain().GetProgramPath(getShortName());
>    ArgStringList CmdArgs;
> -  CmdArgs.push_back("-flavor");
> -  CmdArgs.push_back("old-gnu");
> -  CmdArgs.push_back("-target");
> -  CmdArgs.push_back(Args.MakeArgString(getToolChain().getTripleString()));
>    AddLinkerInputs(getToolChain(), Inputs, Args, CmdArgs);
>    CmdArgs.push_back("-o");
>    CmdArgs.push_back(Output.getFilename());
>
> Modified: cfe/trunk/lib/Driver/Tools.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.h?rev=257175&r1=257174&r2=257175&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Driver/Tools.h (original)
> +++ cfe/trunk/lib/Driver/Tools.h Fri Jan  8 09:14:31 2016
> @@ -240,7 +240,7 @@ namespace amdgpu {
>
>  class LLVM_LIBRARY_VISIBILITY Linker : public GnuTool {
>  public:
> -  Linker(const ToolChain &TC) : GnuTool("amdgpu::Linker", "lld", TC) {}
> +  Linker(const ToolChain &TC) : GnuTool("amdgpu::Linker", "ld.lld", TC) {}
>    bool isLinkJob() const override { return true; }
>    bool hasIntegratedCPP() const override { return false; }
>    void ConstructJob(Compilation &C, const JobAction &JA,
>
> Modified: cfe/trunk/test/Driver/amdgpu-toolchain.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/amdgpu-toolchain.c?rev=257175&r1=257174&r2=257175&view=diff
> ==============================================================================
> --- cfe/trunk/test/Driver/amdgpu-toolchain.c (original)
> +++ cfe/trunk/test/Driver/amdgpu-toolchain.c Fri Jan  8 09:14:31 2016
> @@ -1,3 +1,3 @@
>  // RUN: %clang -### -target amdgcn--amdhsa -x assembler -mcpu=kaveri %s 2>&1 | FileCheck -check-prefix=AS_LINK %s
>  // AS_LINK: clang{{.*}} "-cc1as"
> -// AS_LINK: lld{{.*}} "-flavor" "old-gnu" "-target" "amdgcn--amdhsa"
> +// AS_LINK: ld.lld{{.*}}
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits