[PATCH] D19517: ELF: Treat IFunc definitions in DSOs as functions.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 09:28:48 PDT 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL267566: ELF: Treat IFunc definitions in DSOs as functions. (authored by pcc).

Changed prior to commit:
  http://reviews.llvm.org/D19517?vs=54953&id=55020#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D19517

Files:
  lld/trunk/ELF/Symbols.h
  lld/trunk/test/ELF/Inputs/gnu-ifunc-gotpcrel.s
  lld/trunk/test/ELF/gnu-ifunc-gotpcrel.s

Index: lld/trunk/test/ELF/gnu-ifunc-gotpcrel.s
===================================================================
--- lld/trunk/test/ELF/gnu-ifunc-gotpcrel.s
+++ lld/trunk/test/ELF/gnu-ifunc-gotpcrel.s
@@ -0,0 +1,14 @@
+# REQUIRES: x86
+# RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %S/Inputs/gnu-ifunc-gotpcrel.s -o %t2.o
+# RUN: ld.lld -shared %t2.o -o %t2.so
+# RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %s -o %t.o
+# RUN: ld.lld %t.o %t2.so -o %t
+# RUN: llvm-readobj -dyn-relocations %t | FileCheck %s
+
+# CHECK:      Dynamic Relocations {
+# CHECK-NEXT:   0x120B0 R_X86_64_GLOB_DAT foo 0x0
+# CHECK-NEXT: }
+
+.globl _start
+_start:
+mov foo at gotpcrel(%rip), %rax
Index: lld/trunk/test/ELF/Inputs/gnu-ifunc-gotpcrel.s
===================================================================
--- lld/trunk/test/ELF/Inputs/gnu-ifunc-gotpcrel.s
+++ lld/trunk/test/ELF/Inputs/gnu-ifunc-gotpcrel.s
@@ -0,0 +1,4 @@
+.type foo STT_GNU_IFUNC
+.globl foo
+foo:
+ret
Index: lld/trunk/ELF/Symbols.h
===================================================================
--- lld/trunk/ELF/Symbols.h
+++ lld/trunk/ELF/Symbols.h
@@ -349,7 +349,11 @@
   SharedSymbol(SharedFile<ELFT> *F, StringRef Name, const Elf_Sym &Sym)
       : Defined(SymbolBody::SharedKind, Name, Sym.getBinding(), Sym.st_other,
                 Sym.getType()),
-        File(F), Sym(Sym) {}
+        File(F), Sym(Sym) {
+    // IFuncs defined in DSOs are treated as functions by the static linker.
+    if (isGnuIFunc())
+      Type = llvm::ELF::STT_FUNC;
+  }
 
   SharedFile<ELFT> *File;
   const Elf_Sym &Sym;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19517.55020.patch
Type: text/x-patch
Size: 1588 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160426/f9b96598/attachment.bin>


More information about the llvm-commits mailing list