]> granicus.if.org Git - clang/commitdiff
AMDGPU: Use lld as the linker again
authorTom Stellard <thomas.stellard@amd.com>
Thu, 5 May 2016 17:03:41 +0000 (17:03 +0000)
committerTom Stellard <thomas.stellard@amd.com>
Thu, 5 May 2016 17:03:41 +0000 (17:03 +0000)
Summary:
Now that LLVM is emitting version 2 of the AMD code object, we can
start using lld again for linking instead of our custom tool.

Reviewers: arsenm, kzhuravl

Subscribers: rafael, cfe-commits

Differential Revision: http://reviews.llvm.org/D19952

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@268648 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Driver/Tools.cpp
lib/Driver/Tools.h
test/Driver/amdgpu-toolchain.c

index c688763e69df04a162127827dbb430d9a2d08970..3b639bc975ad2061b7f72c99dc087d7f5aa98073 100644 (file)
@@ -6786,6 +6786,8 @@ void amdgpu::Linker::ConstructJob(Compilation &C, const JobAction &JA,
   std::string Linker = getToolChain().GetProgramPath(getShortName());
   ArgStringList CmdArgs;
   AddLinkerInputs(getToolChain(), Inputs, Args, CmdArgs);
+  CmdArgs.push_back("-shared");
+  CmdArgs.push_back("-o");
   CmdArgs.push_back(Output.getFilename());
   C.addCommand(llvm::make_unique<Command>(JA, *this, Args.MakeArgString(Linker),
                                           CmdArgs, Inputs));
index d25d91d87f34eb311960f6fecedd8a502b9c2154..c9d5c2238d6b7616702e5c5fb6560bf3b1001983 100644 (file)
@@ -242,7 +242,7 @@ namespace amdgpu {
 
 class LLVM_LIBRARY_VISIBILITY Linker : public GnuTool {
 public:
-  Linker(const ToolChain &TC) : GnuTool("amdgpu::Linker", "amdphdrs", 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,
index 5a67bae246f1c478c6c774743c1a680eae768ef3..833b49df5d4255a0397ed53a0d94d3fa2694d66e 100644 (file)
@@ -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: amdphdrs{{.*}}
+// AS_LINK: ld.lld{{.*}} "-shared"