On Sat, Sep 05, 2015 at 12:28:43PM -0700, Saleem Abdulrasool wrote: > > Ping? > > > > Testcase? Looks fine otherwise.
Basically check that -lc is present when clang is called in a certain way I guess? Or something more sophisticated? Xan > > > > > From 8e81d6b095542c0ff1e28cf1f09d675f8afe1a25 Mon Sep 17 00:00:00 2001 > > > From: =?UTF-8?q?Xan=20L=C3=B3pez?= <x...@igalia.com> > > > Date: Mon, 17 Aug 2015 16:04:26 +0200 > > > Subject: [PATCH 4/4] [Solaris] Add -lc also when linking shared libraries > > > > > > This is actually needed, otherwise libc won't be added at all. For > > > instance when building libclang.so all the libc symbols won't be > > > found, with ld warning about libc being an "implicit dependency". > > > --- > > > lib/Driver/Tools.cpp | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp > > > index cf01d7b..2034c0f 100644 > > > --- a/lib/Driver/Tools.cpp > > > +++ b/lib/Driver/Tools.cpp > > > @@ -6899,9 +6899,9 @@ void solaris::Linker::ConstructJob(Compilation &C, > > const JobAction &JA, > > > if (getToolChain().getDriver().CCCIsCXX()) > > > getToolChain().AddCXXStdlibLibArgs(Args, CmdArgs); > > > CmdArgs.push_back("-lgcc_s"); > > > + CmdArgs.push_back("-lc"); > > > if (!Args.hasArg(options::OPT_shared)) { > > > CmdArgs.push_back("-lgcc"); > > > - CmdArgs.push_back("-lc"); > > > CmdArgs.push_back("-lm"); > > > } > > > } > > > -- > > > 2.4.3 > > > > > > > > _______________________________________________ > > > cfe-commits mailing list > > > cfe-commits@lists.llvm.org > > > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > > > > _______________________________________________ > > cfe-commits mailing list > > cfe-commits@lists.llvm.org > > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > > > > > > -- > Saleem Abdulrasool > compnerd (at) compnerd (dot) org _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits