<div dir="ltr">Thanks!!</div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jun 4, 2016 at 6:12 PM Davide Italiano via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">r271820.<br>
<br>
On Sat, Jun 4, 2016 at 6:07 PM, Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> wrote:<br>
> Thanks for your review, Sean. I'll commit without the cp, and take a<br>
> look at the bots to ensure the non-determinism is gone.<br>
> Feel free to shout at me in case this doesn't fix and I'll take another look.<br>
><br>
> On Sat, Jun 4, 2016 at 6:05 PM, Sean Silva <<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>> wrote:<br>
>><br>
>><br>
>> On Sat, Jun 4, 2016 at 6:03 PM, Sean Silva <<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>> wrote:<br>
>>><br>
>>><br>
>>><br>
>>> On Sat, Jun 4, 2016 at 5:53 PM, Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>><br>
>>> wrote:<br>
>>>><br>
>>>> On Sat, Jun 4, 2016 at 5:33 PM, Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>><br>
>>>> 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" target="_blank">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" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br>
>>>> >>><br>
>>>> >>> I think this test needs to create a temporary directory and cd into<br>
>>>> >>> 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<br>
>>>> >> had<br>
>>>> >> explicit `-o`..<br>
>>>> >><br>
>>>> >> It looks like this issue was likely introduced in<br>
>>>> >><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>
>>>> 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>
>>>>  ; RUN: llvm-as %s -o %t.o<br>
>>>> -; 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>
>>><br>
>>><br>
>>> Makes sense. Do you need to copy `%p/Inputs/save-temps.ll`? I think that<br>
>>> should expand to an absolute path and still be correct even after you change<br>
>>> directory.<br>
>><br>
>><br>
>> Sorry, that may have been a bit ambiguous. Basically "LGTM, but I don't<br>
>> think you should need to copy `%p/Inputs/save-temps.ll`".<br>
>><br>
>> -- Sean Silva<br>
>><br>
>>><br>
>>> -- Sean Silva<br>
>>><br>
>>>><br>
>>>> --<br>
>>>> Davide<br>
>>>><br>
>>>> "There are no solved problems; there are only problems that are more<br>
>>>> or less solved" -- Henri Poincare<br>
>>><br>
>>><br>
>><br>
><br>
><br>
><br>
> --<br>
> Davide<br>
><br>
> "There are no solved problems; there are only problems that are more<br>
> or less solved" -- Henri Poincare<br>
<br>
<br>
<br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>