[PATCH] D30441: Fix wrong TLS symbol values.

[Rui Ueyama via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL296505: Fix wrong TLS symbol values. (authored by ruiu).

Changed prior to commit:
  https://reviews.llvm.org/D30441?vs=89964&id=90062#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D30441

Files:
  lld/trunk/ELF/SyntheticSections.cpp
  lld/trunk/test/ELF/gc-debuginfo-tls.s
  lld/trunk/test/ELF/mips-tls-64.s
  lld/trunk/test/ELF/mips-tls-hilo.s
  lld/trunk/test/ELF/mips-tls.s


Index: lld/trunk/test/ELF/gc-debuginfo-tls.s
===================================================================
--- lld/trunk/test/ELF/gc-debuginfo-tls.s
+++ lld/trunk/test/ELF/gc-debuginfo-tls.s
@@ -6,7 +6,7 @@
 
 # NOGC:      Symbol {
 # NOGC:        Name: patatino
-# NOGC-NEXT:   Value: 0x1000
+# NOGC-NEXT:   Value: 0x0
 # NOGC-NEXT:   Size: 0
 # NOGC-NEXT:   Binding: Local
 # NOGC-NEXT:   Type: TLS
Index: lld/trunk/test/ELF/mips-tls-hilo.s
===================================================================
--- lld/trunk/test/ELF/mips-tls-hilo.s
+++ lld/trunk/test/ELF/mips-tls-hilo.s
@@ -21,7 +21,7 @@
 # DIS-NEXT:    2000c:   24 62 90 00   addiu   $2, $3, -28672
 #                       %lo(loc0 - .tdata - 0x7000) --^
 
-# DIS: 00040000 l       .tdata          00000000 loc0
+# DIS: 00000000 l       .tdata          00000000 loc0
 
 # CHECK:      Relocations [
 # CHECK-NEXT: ]
Index: lld/trunk/test/ELF/mips-tls.s
===================================================================
--- lld/trunk/test/ELF/mips-tls.s
+++ lld/trunk/test/ELF/mips-tls.s
@@ -27,7 +27,7 @@
 # DIS-NEXT:  30020 00000000 00000001 00000000 00000001
 # DIS-NEXT:  30030 ffff8004 ffff9004
 
-# DIS: 00040000 l       .tdata          00000000 loc
+# DIS: 00000000 l       .tdata          00000000 loc
 # DIS: 00000004 g       .tdata          00000000 bar
 # DIS: 00000000 g       *UND*           00000000 foo
 
Index: lld/trunk/test/ELF/mips-tls-64.s
===================================================================
--- lld/trunk/test/ELF/mips-tls-64.s
+++ lld/trunk/test/ELF/mips-tls-64.s
@@ -29,7 +29,7 @@
 # DIS-NEXT:  30040 00000000 00000000 00000000 00000001
 # DIS-NEXT:  30050 ffffffff ffff8004 ffffffff ffff9004
 
-# DIS: 0000000000040000 l       .tdata          00000000 loc
+# DIS: 0000000000000000 l       .tdata          00000000 loc
 # DIS: 0000000000000004 g       .tdata          00000000 bar
 # DIS: 0000000000000000 g       *UND*           00000000 foo
 
Index: lld/trunk/ELF/SyntheticSections.cpp
===================================================================
--- lld/trunk/ELF/SyntheticSections.cpp
+++ lld/trunk/ELF/SyntheticSections.cpp
@@ -1376,14 +1376,6 @@
 
     if (const OutputSection *OutSec = Body->getOutputSection<ELFT>()) {
       ESym->st_shndx = OutSec->SectionIndex;
-
-      // This piece of code should go away as it doesn't make sense,
-      // but we want to keep it tentatively because some tests for TLS
-      // variable depends on this. We should fix the test and remove
-      // this code.
-      if (Body->isLocal())
-        if (auto *DS = dyn_cast<DefinedRegular<ELFT>>(Body))
-          ESym->st_value = OutSec->Addr + DS->Section->getOffset(*DS);
     } else if (isa<DefinedRegular<ELFT>>(Body)) {
       ESym->st_shndx = SHN_ABS;
     } else if (isa<DefinedCommon>(Body)) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30441.90062.patch
Type: text/x-patch
Size: 2821 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170228/8d1f73df/attachment.bin>