Revert "ELF: Add /lib and /usr/lib as default search paths."
authorRafael Espindola <rafael.espindola@gmail.com>
Tue, 8 Mar 2016 17:13:12 +0000 (17:13 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Tue, 8 Mar 2016 17:13:12 +0000 (17:13 +0000)
It was causing errors like

/lib/libc.so.6 is incompatible with elf_x86_64

when linking on Fedora.

Every system has different default paths. It seems better to just trust
the driver to pass the correct -L options.

This reverts commit 262910.

llvm-svn: 262941

lld/ELF/Config.h
lld/ELF/Driver.cpp
lld/ELF/LinkerScript.cpp
lld/ELF/Options.td
lld/test/ELF/Inputs/nostdlib.s [deleted file]
lld/test/ELF/nostdlib.s [deleted file]

index 7b58008..47f21fa 100644 (file)
@@ -67,7 +67,6 @@ struct Configuration {
   bool Mips64EL = false;
   bool NoUndefined;
   bool NoinhibitExec;
-  bool Nostdlib;
   bool PrintGcSections;
   bool Relocatable;
   bool Shared;
index d49d201..c4773d8 100644 (file)
@@ -207,6 +207,9 @@ void LinkerDriver::main(ArrayRef<const char *> ArgsArr) {
 
 // Initializes Config members by the command line options.
 void LinkerDriver::readConfigs(opt::InputArgList &Args) {
+  for (auto *Arg : Args.filtered(OPT_L))
+    Config->SearchPaths.push_back(Arg->getValue());
+
   std::vector<StringRef> RPaths;
   for (auto *Arg : Args.filtered(OPT_rpath))
     RPaths.push_back(Arg->getValue());
@@ -234,7 +237,6 @@ void LinkerDriver::readConfigs(opt::InputArgList &Args) {
   Config->ICF = Args.hasArg(OPT_icf);
   Config->NoUndefined = Args.hasArg(OPT_no_undefined);
   Config->NoinhibitExec = Args.hasArg(OPT_noinhibit_exec);
-  Config->Nostdlib = Args.hasArg(OPT_nostdlib);
   Config->PrintGcSections = Args.hasArg(OPT_print_gc_sections);
   Config->Relocatable = Args.hasArg(OPT_relocatable);
   Config->Shared = Args.hasArg(OPT_shared);
@@ -277,14 +279,6 @@ void LinkerDriver::readConfigs(opt::InputArgList &Args) {
 
   for (auto *Arg : Args.filtered(OPT_undefined))
     Config->Undefined.push_back(Arg->getValue());
-
-  if (!Config->Nostdlib) {
-    Config->SearchPaths.push_back("=/lib");
-    Config->SearchPaths.push_back("=/usr/lib");
-  }
-
-  for (auto *Arg : Args.filtered(OPT_L))
-    Config->SearchPaths.push_back(Arg->getValue());
 }
 
 void LinkerDriver::createFiles(opt::InputArgList &Args) {
index f9ab4db..b4580b5 100644 (file)
@@ -396,9 +396,7 @@ void ScriptParser::readOutputFormat() {
 
 void ScriptParser::readSearchDir() {
   expect("(");
-  StringRef Path = next();
-  if (!Config->Nostdlib)
-    Config->SearchPaths.push_back(Path);
+  Config->SearchPaths.push_back(next());
   expect(")");
 }
 
index 46a81f0..99a2080 100644 (file)
@@ -91,9 +91,6 @@ def noinhibit_exec : Flag<["--"], "noinhibit-exec">,
 def no_undefined : Flag<["--"], "no-undefined">,
   HelpText<"Report unresolved symbols even if the linker is creating a shared library">;
 
-def nostdlib: Flag<["-", "--"], "nostdlib">,
-  HelpText<"Only search directories specified on the command line">;
-
 def o : JoinedOrSeparate<["-"], "o">, MetaVarName<"<path>">,
   HelpText<"Path to file to write output">;
 
diff --git a/lld/test/ELF/Inputs/nostdlib.s b/lld/test/ELF/Inputs/nostdlib.s
deleted file mode 100644 (file)
index db41c9c..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
-.globl foo
-foo:
-  ret
diff --git a/lld/test/ELF/nostdlib.s b/lld/test/ELF/nostdlib.s
deleted file mode 100644 (file)
index f2c45da..0000000
+++ /dev/null
@@ -1,16 +0,0 @@
-# REQUIRES: x86
-
-# RUN: mkdir -p %t.dir/lib
-# RUN: mkdir -p %t.dir/usr/lib
-# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t1.o
-# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %p/Inputs/nostdlib.s -o %t2.o
-# RUN: ld.lld -shared -o %t.dir/lib/libfoo.so %t2.o
-# RUN: ld.lld -shared -o %t.dir/usr/lib/libbar.so %t2.o
-# RUN: ld.lld --sysroot=%t.dir -o %t %t1.o -lfoo
-# RUN: ld.lld --sysroot=%t.dir -o %t %t1.o -lbar
-# RUN: not ld.lld --sysroot=%t.dir -nostdlib -o %t %t1.o -lfoo
-# RUN: not ld.lld --sysroot=%t.dir -nostdlib -o %t %t1.o -lbar
-
-.globl _start
-_start:
-  ret