[lld] Allow rel iplt symbols with dynamic symbol table

[smeenai]

Summary

Emit these symbols as long as we're building in a static configuration,
even if we're emitting a dynamic symbol table. This is consistent with
both bfd and gold.

Ordinarily, the combination of -static and -export-dynamic wouldn't make
much sense. Unfortunately, cmake versions prior to 3.4 forcefully
injected -rdynamic [1], so it seems worthwhile to support.

[1] https://cmake.org/cmake/help/v3.4/policy/CMP0065.html

[smeenai]

Instead of duplicating the test, I can either generalize the existing test to handle both the regular and --export-dynamic cases by replacing hardcoded values with regular expressions where necessary, or just add multiple check prefixes to the existing test and handle the value differences that way. Let me know what's preferable.

[ruiu]

I'd just check for the existence of __rela_iplt_{start,end} symbols. We don't need to verify its disassembly as that should already been tested by other tests.

[smeenai]

Simplify test

[ruiu]

test/ELF/gnu-ifunc-dynsym.s

7 ↗

(On Diff #109855)

Remove -DAG?

[smeenai]

Address comment

[smeenai]

test/ELF/gnu-ifunc-dynsym.s

7 ↗

(On Diff #109855)

I had the -DAG because I don't care about the order in which the symbols appear, only that they're both present. I'm happy without it as well though.

[smeenai]

test/ELF/gnu-ifunc-dynsym.s

7 ↗

(On Diff #109855)

To clarify, I meant their order in the llvm-nm output. Of course the end symbol should be after the start symbol in terms of addresses, but llvm-nm appears to print out symbols in reverse order of addresses, at least in this instance.

[ruiu]

LGTM

test/ELF/gnu-ifunc-dynsym.s

7 ↗

(On Diff #109855)

Ah, I didn't actually know that FileCheck had such feature.

lld/trunk/ELF/Writer.cpp

 
 // The beginning and the ending of .rel[a].plt section are marked
 // with __rel[a]_iplt_{start,end} symbols if it is a statically linked
 // executable. The runtime needs these symbols in order to resolve
 // all IRELATIVE relocs on startup. For dynamic executables, we don't
 // need these symbols, since IRELATIVE relocs are resolved through GOT
 // and PLT. For details, see http://www.airs.com/blog/archives/403.
 template <class ELFT> void Writer<ELFT>::addRelIpltSymbols() {
-  if (InX::DynSymTab)
+  if (!Config->Static)
     return;
   StringRef S = Config->IsRela ? "__rela_iplt_start" : "__rel_iplt_start";
   addOptionalRegular<ELFT>(S, In<ELFT>::RelaIplt, 0, STV_HIDDEN, STB_WEAK);
 
   S = Config->IsRela ? "__rela_iplt_end" : "__rel_iplt_end";
   addOptionalRegular<ELFT>(S, In<ELFT>::RelaIplt, -1, STV_HIDDEN, STB_WEAK);
 }
 

lld/trunk/test/ELF/gnu-ifunc-dynsym.s

+// RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %s -o %t.o
+// RUN: ld.lld -static -export-dynamic %t.o -o %tout
+// RUN: llvm-nm -U %tout | FileCheck %s
+// REQUIRES: x86
+
+// CHECK: __rela_iplt_end
+// CHECK: __rela_iplt_start
+
+.text
+.type foo STT_GNU_IFUNC
+.globl foo
+foo:
+ ret
+
+.globl _start
+_start:
+ call foo
+ movl $__rela_iplt_start,%edx
+ movl $__rela_iplt_end,%edx