[PATCH] D50289: [ELF] Don't copy STT_TLS in copy relocation

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 6 12:10:23 PDT 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rLLD339042: [ELF] Don't copy STT_TLS in copy relocation (authored by MaskRay, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D50289?vs=159350&id=159351#toc

Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D50289

Files:
  ELF/Relocations.cpp
  test/ELF/Inputs/copy-rel-tls.s
  test/ELF/copy-rel-tls.s


Index: test/ELF/Inputs/copy-rel-tls.s
===================================================================
--- test/ELF/Inputs/copy-rel-tls.s
+++ test/ELF/Inputs/copy-rel-tls.s
@@ -0,0 +1,12 @@
+.bss
+.global foo
+.type foo, @object
+.size foo, 4
+foo:
+
+.section .tbss,"awT", at nobits
+.global tfoo
+.skip 0x2000
+.type tfoo, at object
+.size tfoo, 4
+tfoo:
Index: test/ELF/copy-rel-tls.s
===================================================================
--- test/ELF/copy-rel-tls.s
+++ test/ELF/copy-rel-tls.s
@@ -0,0 +1,15 @@
+// REQUIRES: x86
+// RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %s -o %t.o
+// RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %p/Inputs/copy-rel-tls.s -o %t1.o
+// RUN: ld.lld %t1.o -shared -soname t1.so -o %t1.so
+// RUN: ld.lld %t.o %t1.so -o %t
+// RUN: llvm-nm %t1.so | FileCheck %s
+// RUN: llvm-nm %t | FileCheck --check-prefix=TLS %s
+// foo and tfoo have the same st_value but we should not copy tfoo.
+// CHECK: 2000 B foo
+// CHECK: 2000 B tfoo
+// TLS-NOT: tfoo
+
+.global _start
+_start:
+  leaq foo, %rax
Index: ELF/Relocations.cpp
===================================================================
--- ELF/Relocations.cpp
+++ ELF/Relocations.cpp
@@ -466,7 +466,7 @@
   SmallSet<SharedSymbol *, 4> Ret;
   for (const Elf_Sym &S : File.getGlobalELFSyms()) {
     if (S.st_shndx == SHN_UNDEF || S.st_shndx == SHN_ABS ||
-        S.st_value != SS.Value)
+        S.getType() == STT_TLS || S.st_value != SS.Value)
       continue;
     StringRef Name = check(S.getName(File.getStringTable()));
     Symbol *Sym = Symtab->find(Name);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50289.159351.patch
Type: text/x-patch
Size: 1582 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180806/6aad80c2/attachment.bin>


More information about the llvm-commits mailing list