diff options
| author | Rainer Orth <ro@gcc.gnu.org> | 2019-07-16 11:06:43 +0000 |
|---|---|---|
| committer | Rainer Orth <ro@gcc.gnu.org> | 2019-07-16 11:06:43 +0000 |
| commit | a5dc9c98352c396c0114403486fbd47e092d084a (patch) | |
| tree | a0e8cfc421386e4b36208bfab8ed94037181a411 /clang/lib/Driver/ToolChains/Solaris.cpp | |
| parent | 971ac4ca2cb8172a1c8f59c7703680b41aa7a5f5 (diff) | |
| download | bcm5719-llvm-a5dc9c98352c396c0114403486fbd47e092d084a.tar.gz bcm5719-llvm-a5dc9c98352c396c0114403486fbd47e092d084a.zip | |
[Driver] Don't pass --dynamic-linker to ld on Solaris
I noticed that clang currently passes --dynamic-linker to ld. This has been the case
since Solaris 11 support was added initially back in 2012 by David Chisnall (r150580).
I couldn't find any patch submission, let alone a justification, for this, and it seems
completely useless: --dynamic-linker is a gld compatibility form of the option, the
native option being -I. First of all, however, the dynamic linker passed is simply the
default, so there's no reason at all to specify it in the first place.
This patch removes passing the option and adjusts the affected testcase accordingly.
Tested on x86_64-pc-solaris2.11 and sparcv9-sun-solaris2.11.
Differential Revision: https://reviews.llvm.org/D64493
llvm-svn: 366202
Diffstat (limited to 'clang/lib/Driver/ToolChains/Solaris.cpp')
| -rw-r--r-- | clang/lib/Driver/ToolChains/Solaris.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/clang/lib/Driver/ToolChains/Solaris.cpp b/clang/lib/Driver/ToolChains/Solaris.cpp index c65b7837395..38f24d4cf7e 100644 --- a/clang/lib/Driver/ToolChains/Solaris.cpp +++ b/clang/lib/Driver/ToolChains/Solaris.cpp @@ -65,10 +65,6 @@ void solaris::Linker::ConstructJob(Compilation &C, const JobAction &JA, CmdArgs.push_back("-Bdynamic"); if (Args.hasArg(options::OPT_shared)) { CmdArgs.push_back("-shared"); - } else { - CmdArgs.push_back("--dynamic-linker"); - CmdArgs.push_back( - Args.MakeArgString(getToolChain().GetFilePath("ld.so.1"))); } // libpthread has been folded into libc since Solaris 10, no need to do |
