<p dir="ltr">Lgtm.</p>
<p dir="ltr">Thanks<br>
Rafael</p>
<div class="gmail_quote">On Mar 28, 2016 6:44 PM, "Davide Italiano" <<a href="mailto:dccitaliano@gmail.com">dccitaliano@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davide updated this revision to Diff 51847.<br>
davide added a comment.<br>
<br>
Rafael's comments.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D18357" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18357</a><br>
<br>
Files:<br>
  ELF/SymbolTable.cpp<br>
  ELF/Symbols.h<br>
  test/ELF/lto/Inputs/shared.s<br>
  test/ELF/lto/undefined-puts.ll<br>
<br>
Index: test/ELF/lto/undefined-puts.ll<br>
===================================================================<br>
--- /dev/null<br>
+++ test/ELF/lto/undefined-puts.ll<br>
@@ -0,0 +1,28 @@<br>
+; REQUIRES: x86<br>
+; RUN: llvm-mc %p/Inputs/shared.s -o %t1.o -filetype=obj -triple=x86_64-unknown-linux<br>
+; RUN: ld.lld %t1.o -o %t1.so -shared<br>
+; RUN: llvm-as %s -o %t2.o<br>
+; RUN: ld.lld %t1.so %t2.o -m elf_x86_64 -o %t<br>
+; RUN: llvm-readobj -dyn-symbols -dyn-relocations %t | FileCheck %s<br>
+<br>
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"<br>
+target triple = "x86_64-unknown-linux-gnu"<br>
+<br>
+@.str = private unnamed_addr constant [6 x i8] c"blah\0A\00", align 1<br>
+<br>
+define i32 @_start() {<br>
+  %str = call i32 (i8*, ...) @printf(i8* getelementptr inbounds ([6 x i8], [6 x i8]* @.str, i32 0, i32 0))<br>
+  ret i32 0<br>
+}<br>
+<br>
+declare i32 @printf(i8*, ...)<br>
+<br>
+; Check that puts symbol is present in the dynamic symbol table and<br>
+; there's a relocation for it.<br>
+; CHECK: Dynamic Relocations {<br>
+; CHECK-NEXT:  0x11007 R_X86_64_PC32 puts 0xFFFFFFFFFFFFFFFC<br>
+; CHECK-NEXT: }<br>
+<br>
+; CHECK: DynamicSymbols [<br>
+; CHECK: Symbol {<br>
+; CHECK:    Name: puts@<br>
Index: test/ELF/lto/Inputs/shared.s<br>
===================================================================<br>
--- /dev/null<br>
+++ test/ELF/lto/Inputs/shared.s<br>
@@ -0,0 +1,5 @@<br>
+.globl  printf<br>
+printf:<br>
+<br>
+.globl  puts<br>
+puts:<br>
Index: ELF/Symbols.h<br>
===================================================================<br>
--- ELF/Symbols.h<br>
+++ ELF/Symbols.h<br>
@@ -89,6 +89,8 @@<br>
   bool isInGot() const { return GotIndex != -1U; }<br>
   bool isInPlt() const { return PltIndex != -1U; }<br>
<br>
+  void setUsedInRegularObj() { IsUsedInRegularObj = true; }<br>
+<br>
   template <class ELFT><br>
   typename ELFT::uint getVA(typename ELFT::uint Addend = 0) const;<br>
<br>
@@ -319,7 +321,6 @@<br>
   std::unique_ptr<InputFile> getMember();<br>
<br>
   void setWeak() { IsWeak = true; }<br>
-  void setUsedInRegularObj() { IsUsedInRegularObj = true; }<br>
<br>
 private:<br>
   ArchiveFile *File;<br>
Index: ELF/SymbolTable.cpp<br>
===================================================================<br>
--- ELF/SymbolTable.cpp<br>
+++ ELF/SymbolTable.cpp<br>
@@ -108,6 +108,7 @@<br>
   Obj->parse(DummyGroups);<br>
   for (SymbolBody *Body : Obj->getNonLocalSymbols()) {<br>
     Symbol *Sym = insert(Body);<br>
+    Sym->Body->setUsedInRegularObj();<br>
     if (!Sym->Body->isUndefined() && Body->isUndefined())<br>
       continue;<br>
     Sym->Body = Body;<br>
<br>
<br>
</blockquote></div>