<div dir="ltr">Sure. That's a good test.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 12, 2016 at 1:37 PM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Oct 12, 2016 at 1:07 PM, Rui Ueyama via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> ruiu added a comment.<br>
><br>
> Does this work for --whole-archive?<br>
><br>
<br>
</span>I think so.<br>
<br>
$ git diff<br>
diff --git a/test/ELF/lto/thin-<wbr>archivecollision.ll<br>
b/test/ELF/lto/thin-<wbr>archivecollision.ll<br>
index bd754d8..4e07187 100644<br>
--- a/test/ELF/lto/thin-<wbr>archivecollision.ll<br>
+++ b/test/ELF/lto/thin-<wbr>archivecollision.ll<br>
@@ -7,6 +7,10 @@<br>
<span class=""> ; RUN: ld.lld %t.o %t1.a %t2.a -o %t<br>
</span><span class=""> ; RUN: llvm-nm %t | FileCheck %s<br>
<br>
</span>+; Check we handle this case correctly even in presence of --whole-archive.<br>
+; RUN: ld.lld %t.o --whole-archive %t1.a %t2.a -o %t<br>
<span class="">+; RUN: llvm-nm %t | FileCheck %s<br>
+<br>
</span> ; CHECK: T _start<br>
 ; CHECK: T blah<br>
 ; CHECK: T foo<br>
<br>
$ ninja check-lld<br>
[1/1] Running lld test suite<br>
Testing Time: 4.79s<br>
  Expected Passes    : 895<br>
  Unsupported Tests  : 10<br>
[davide@localhost build]$<br>
<br>
<br>
Do you think it's OK to add this test (just to be sure we don't regress?)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</font></span></blockquote></div><br></div>