[lld] 93331a1 - [ELF] Support archive:file syntax in input section descriptions

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 25 07:57:55 PST 2020


Author: Fangrui Song
Date: 2020-02-25T07:57:43-08:00
New Revision: 93331a17e8b3d6205efc8f1d4e7a74523f3b7035

URL: https://github.com/llvm/llvm-project/commit/93331a17e8b3d6205efc8f1d4e7a74523f3b7035
DIFF: https://github.com/llvm/llvm-project/commit/93331a17e8b3d6205efc8f1d4e7a74523f3b7035.diff

LOG: [ELF] Support archive:file syntax in input section descriptions

Fixes https://bugs.llvm.org/show_bug.cgi?id=44450

https://sourceware.org/binutils/docs/ld/Input-Section-Basics.html#Input-Section-Basics
The following two rules are not implemented.

* `archive:` matches every file in the archive.
* `:file` matches a file not in an archive.

Reviewed By: grimar, ruiu

Differential Revision: https://reviews.llvm.org/D75100

Added: 
    lld/test/ELF/linkerscript/input-archive.s

Modified: 
    lld/ELF/LinkerScript.cpp

Removed: 
    


################################################################################
diff  --git a/lld/ELF/LinkerScript.cpp b/lld/ELF/LinkerScript.cpp
index 9c9aacc7eceb..d78b4907a5be 100644
--- a/lld/ELF/LinkerScript.cpp
+++ b/lld/ELF/LinkerScript.cpp
@@ -323,7 +323,7 @@ static std::string getFilename(InputFile *file) {
     return "";
   if (file->archiveName.empty())
     return std::string(file->getName());
-  return (file->archiveName + "(" + file->getName() + ")").str();
+  return (file->archiveName + ':' + file->getName()).str();
 }
 
 bool LinkerScript::shouldKeep(InputSectionBase *s) {

diff  --git a/lld/test/ELF/linkerscript/input-archive.s b/lld/test/ELF/linkerscript/input-archive.s
new file mode 100644
index 000000000000..f70f3c6fa411
--- /dev/null
+++ b/lld/test/ELF/linkerscript/input-archive.s
@@ -0,0 +1,32 @@
+# REQUIRES: x86
+# UNSUPPORTED: system-windows
+## Test that archive:file is supported in an input section description.
+
+# RUN: mkdir -p %t.dir
+# RUN: echo '.data; .byte 1' | llvm-mc -filetype=obj -triple=x86_64 - -o %t.dir/a.o
+# RUN: llvm-mc -filetype=obj -triple=x86_64 %s -o %t.dir/b.o
+# RUN: rm -f %t.a
+# RUN: llvm-ar rc %t.a %t.dir/a.o %t.dir/b.o
+
+## *.a:b.o matches /path/to/input-archive.s.tmp.a:b.o
+## *b.o matches /path/to/input-archive.s.tmp.a:b.o
+# RUN: echo 'SECTIONS { \
+# RUN:   .foo : { %t.a:a.o(.data) } \
+# RUN:   .bar : { *.a:b.o(.data) } \
+# RUN:   .qux : { *b.o(.data1) } \
+# RUN:   }' > %t.script
+# RUN: ld.lld -T %t.script --whole-archive %t.a -o %t
+# RUN: llvm-readelf -x .foo -x .bar -x .qux %t | FileCheck %s
+
+# CHECK:      Hex dump of section '.foo':
+# CHECK-NEXT: 0x00000000 01
+# CHECK:      Hex dump of section '.bar':
+# CHECK-NEXT: 0x00000001 02
+# CHECK:      Hex dump of section '.qux':
+# CHECK-NEXT: 0x00000002 03
+
+.data
+.byte 2
+
+.section .data1,"aw", at progbits
+.byte 3


        


More information about the llvm-commits mailing list