<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 4, 2016 at 5:53 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="">On Sat, Jun 4, 2016 at 5:33 PM, Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br>
> On Sat, Jun 4, 2016 at 5:18 PM, Sean Silva via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>><br>
>> On Sat, Jun 4, 2016 at 4:52 PM, Rafael Espíndola<br>
>> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>>><br>
>>> I think this test needs to create a temporary directory and cd into it:<br>
>>><br>
>>> llvm-nm a.out | FileCheck %s<br>
>>><br>
>>> this could be using a a.out from some other test if the scheduling is<br>
>>> just right.<br>
>><br>
>><br>
>> Good catch. I was staring at my original test case in this patch which had<br>
>> explicit `-o`..<br>
>><br>
>> It looks like this issue was likely introduced in<br>
>> <a href="http://llvm.org/klaus/lld/commit/e60c3351266e56d91bfd60f34d36edbbb3a1c5e1/" rel="noreferrer" target="_blank">http://llvm.org/klaus/lld/commit/e60c3351266e56d91bfd60f34d36edbbb3a1c5e1/</a><br>
>><br>
<br>
</span>Index: test/ELF/lto/save-temps.ll<br>
===================================================================<br>
--- test/ELF/lto/save-temps.ll (revision 271819)<br>
+++ test/ELF/lto/save-temps.ll (working copy)<br>
@@ -1,7 +1,9 @@<br>
 ; REQUIRES: x86<br>
-; RUN: rm -f a.out a.out.lto.bc a.out.lto.o<br>
+; RUN: cd %T<br>
+; RUN: rm -f %t.ll a.out a.out.lto.bc a.out.lto.o<br>
+; RUN: cp %p/Inputs/save-temps.ll %t.ll<br>
<span class=""> ; RUN: llvm-as %s -o %t.o<br>
</span>-; RUN: llvm-as %p/Inputs/save-temps.ll -o %t2.o<br>
+; RUN: llvm-as %t.ll -o %t2.o<br>
 ; RUN: ld.lld -shared -m elf_x86_64 %t.o %t2.o -save-temps<br>
 ; RUN: llvm-nm a.out | FileCheck %s<br>
 ; RUN: llvm-nm a.out.lto.bc | FileCheck %s<br>
<br>
Something like this?<br></blockquote><div><br></div><div>Makes sense. Do you need to copy %p/Inputs/save-temps.ll`? I think that should expand to an absolute path and still be correct even after you change directory.</div><div><br></div><div>-- Sean Silva </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<div class=""><div class="h5"><br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</div></div></blockquote></div><br></div></div>