summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRui Ueyama <ruiu@google.com>2018-05-31 13:00:25 +0000
committerRui Ueyama <ruiu@google.com>2018-05-31 13:00:25 +0000
commitf75ea0b99580f3ffc7473f6703890181ad231b83 (patch)
treee5c94ca2a9b30806e211e4dbd1011eb64d7f6545
parentd9a453832ddd110d7896e0ffcff1c610aec31b60 (diff)
downloadbcm5719-llvm-f75ea0b99580f3ffc7473f6703890181ad231b83.tar.gz
bcm5719-llvm-f75ea0b99580f3ffc7473f6703890181ad231b83.zip
Implement --{push,pop}-state.
--push-state implemented in this patch saves the states of --as-needed, --whole-archive and --static. It saves less number of flags than GNU linkers. Since even GNU linkers save different flags, no one seems to care about the details. In this patch, I tried to save the minimal number of flags to not complicate the implementation and the siutation. I'm not personally happy about adding the --{push,pop}-state flags though. That options seem too hacky to me. However, gcc started using the options since GCC 8 when GNU ld is available at the build time. Therefore, lld is no longer a drop-in replacmenet for GNU linker for that machine without supporting the flags. Fixes https://bugs.llvm.org/show_bug.cgi?id=34567 Differential Revision: https://reviews.llvm.org/D47542 llvm-svn: 333646
-rw-r--r--lld/ELF/Driver.cpp15
-rw-r--r--lld/ELF/Options.td6
-rw-r--r--lld/docs/ld.lld.19
-rw-r--r--lld/test/ELF/push-state.s36
4 files changed, 66 insertions, 0 deletions
diff --git a/lld/ELF/Driver.cpp b/lld/ELF/Driver.cpp
index d196bb167ec..e6eba274e02 100644
--- a/lld/ELF/Driver.cpp
+++ b/lld/ELF/Driver.cpp
@@ -940,6 +940,10 @@ static bool getBinaryOption(StringRef S) {
}
void LinkerDriver::createFiles(opt::InputArgList &Args) {
+ // For --{push,pop}-state.
+ std::vector<std::tuple<bool, bool, bool>> Stack;
+
+ // Iterate over argv to process input files and positional arguments.
for (auto *Arg : Args) {
switch (Arg->getOption().getUnaliasedOption().getID()) {
case OPT_library:
@@ -1016,6 +1020,17 @@ void LinkerDriver::createFiles(opt::InputArgList &Args) {
InputFile::IsInGroup = false;
++InputFile::NextGroupId;
break;
+ case OPT_push_state:
+ Stack.push_back({Config->AsNeeded, Config->Static, InWholeArchive});
+ break;
+ case OPT_pop_state:
+ if (Stack.empty()) {
+ error("unbalanced --push-state/--pop-state");
+ break;
+ }
+ std::tie(Config->AsNeeded, Config->Static, InWholeArchive) = Stack.back();
+ Stack.pop_back();
+ break;
}
}
diff --git a/lld/ELF/Options.td b/lld/ELF/Options.td
index 422784d87d2..c8a1260cfb0 100644
--- a/lld/ELF/Options.td
+++ b/lld/ELF/Options.td
@@ -244,6 +244,12 @@ defm print_icf_sections: B<"print-icf-sections",
"List identical folded sections",
"Do not list identical folded sections (default)">;
+def pop_state: F<"pop-state">,
+ HelpText<"Undo the effect of -push-state">;
+
+def push_state: F<"push-state">,
+ HelpText<"Save the current state of -as-needed, -static and -whole-archive">;
+
def print_map: F<"print-map">,
HelpText<"Print a link map to the standard output">;
diff --git a/lld/docs/ld.lld.1 b/lld/docs/ld.lld.1
index 95d9750cca6..7f3a747bbbd 100644
--- a/lld/docs/ld.lld.1
+++ b/lld/docs/ld.lld.1
@@ -289,6 +289,15 @@ Create a position independent executable.
List removed unused sections.
.It Fl -print-map
Print a link map to the standard output.
+.It Fl -push-state
+Save the current state of
+.Fl -as-needed ,
+.Fl -static ,
+and
+.Fl -while-archive.
+.It Fl -pop-state
+Undo the effect of
+.Fl -push-state.
.It Fl -relocatable
Create relocatable object file.
.It Fl -reproduce Ns = Ns Ar value
diff --git a/lld/test/ELF/push-state.s b/lld/test/ELF/push-state.s
new file mode 100644
index 00000000000..5a01cd2eeed
--- /dev/null
+++ b/lld/test/ELF/push-state.s
@@ -0,0 +1,36 @@
+// REQUIRES: x86
+
+// RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t1.o
+// RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux \
+// RUN: %p/Inputs/whole-archive.s -o %t2.o
+// RUN: rm -f %t.a
+// RUN: llvm-ar rcs %t.a %t2.o
+
+// RUN: ld.lld -o %t.exe -push-state -whole-archive %t.a %t1.o -M | \
+// RUN: FileCheck -check-prefix=WHOLE %s
+// WHOLE: _bar
+
+// RUN: ld.lld -o %t.exe -push-state -whole-archive -pop-state %t.a %t1.o -M | \
+// RUN: FileCheck -check-prefix=NO-WHOLE %s
+// NO-WHOLE-NOT: _bar
+
+
+// RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %p/Inputs/shared.s -o %t3.o
+// RUN: ld.lld -shared %t3.o -soname libfoo -o %t.so
+
+// RUN: ld.lld -o %t.exe -push-state -as-needed %t.so %t1.o
+// RUN: llvm-readobj -dynamic-table %t.exe | FileCheck -check-prefix=AS-NEEDED %s
+// AS-NEEDED-NOT: NEEDED Shared library: [libfoo]
+
+// RUN: ld.lld -o %t.exe -push-state -as-needed -pop-state %t.so %t1.o
+// RUN: llvm-readobj -dynamic-table %t.exe | FileCheck -check-prefix=NO-AS-NEEDED %s
+// NO-AS-NEEDED: NEEDED Shared library: [libfoo]
+
+
+// RUN: mkdir -p %t.dir
+// RUN: cp %t.so %t.dir/libfoo.so
+// RUN: ld.lld -o %t.exe -L%t.dir -push-state -static -pop-state %t1.o -lfoo
+// RUN: not ld.lld -o %t.exe -L%t.dir -push-state -static %t1.o -lfoo
+
+.globl _start
+_start:
OpenPOWER on IntegriCloud