Takumi - can you confirm/deny/describe why the rm was necessary? Do you know off-hand if %t<foo> was replaced by %T/foo if this would be a non-issue? <br><br>(Lang any quick examples of %t<foo> without deletions you were using as reference? (in case we need to fix a bunch?))<br>
<div>On Fri Jan 10 2014 at 12:48:43 PM, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I had assumed that %t<foo> is also nuked, since that pattern is used<br>
in several places. In that case the 'rm' line would be redundant. I<br>
haven't confirmed that though.<br>
<br>
- Lang.<br>
<br>
On Fri, Jan 10, 2014 at 10:27 AM, <a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a> <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
> On Fri Jan 10 2014 at 2:44:36 AM, NAKAMURA Takumi <<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Author: chapuni<br>
>> Date: Fri Jan 10 04:38:23 2014<br>
>> New Revision: 198926<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=198926&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=198926&view=rev</a><br>
>> Log:<br>
>> llvm/test/ExecutionEngine/<u></u>MCJIT/load-object-a.ll: Fix not to use<br>
>> %t.cachedir/%p.<br>
>><br>
>> %p is like X:\foo\bar.<br>
>><br>
>> Modified:<br>
>>     llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/load-<u></u>object-a.ll<br>
>><br>
>> Modified: llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/load-<u></u>object-a.ll<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/ExecutionEngine/MCJIT/load-object-a.ll?rev=198926&r1=198925&r2=198926&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/load-<u></u>object-a.ll?rev=198926&r1=<u></u>198925&r2=198926&view=diff</a><br>

>><br>
>> ==============================<u></u>==============================<u></u>==================<br>
>> --- llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/load-<u></u>object-a.ll (original)<br>
>> +++ llvm/trunk/test/<u></u>ExecutionEngine/MCJIT/load-<u></u>object-a.ll Fri Jan 10<br>
>> 04:38:23 2014<br>
>> @@ -1,18 +1,27 @@<br>
>>  ; REQUIRES: shell<br>
>><br>
>>  ; This first line will generate the .o files for the next run line<br>
>> +; RUN: rm -rf %t.cachedir<br>
><br>
><br>
> Lang/Takumi:<br>
><br>
> Do we need to do this? Most other test cases seem to just use %T to get a<br>
> temporary directory which I assume is nuked by the test framework<br>
> appropriately without the test having to handle it explicitly. (& I assume<br>
> %t, a single temporary file, is nuked by the test framework too)<br>
><br>
> So perhaps it should just be %T rather than %t.cachedir? Or maybe<br>
> %T/cachedir?<br>
><br>
>><br>
>>  ; RUN: mkdir -p %t.cachedir<br>
>>  ; RUN: %lli_mcjit -extra-module=%p/Inputs/multi-<u></u>module-b.ll<br>
>> -extra-module=%p/Inputs/multi-<u></u>module-c.ll -enable-cache-manager<br>
>> -object-cache-dir=%t.cachedir %s<br>
>><br>
>> +; Collect generated objects.<br>
>> +; RUN: rm -rf %t.cachedir2<br>
>> +; RUN: mkdir -p %t.cachedir2<br>
>> +; RUN: find %t.cachedir -type f -name 'multi-module-?.o' -exec mv -v '{}'<br>
>> %t.cachedir2 ';'<br>
>> +<br>
>>  ; This line tests MCJIT object loading<br>
>> -; RUN: %lli_mcjit -extra-object=%t.cachedir/%p/<u></u>Inputs/multi-module-b.o<br>
>> -extra-object=%t.cachedir/%p/<u></u>Inputs/multi-module-c.o %s<br>
>> +; RUN: %lli_mcjit -extra-object=%t.cachedir2/<u></u>multi-module-b.o<br>
>> -extra-object=%t.cachedir2/<u></u>multi-module-c.o %s<br>
>> +<br>
>> +; RUN: rm -rf %t.cachedir3<br>
>> +; RUN: mkdir -p %t.cachedir3<br>
>><br>
>>  ; These lines put the object files into an archive<br>
>> -; RUN: llvm-ar r %t.cachedir/%p/Inputs/load-<u></u>object.a<br>
>> %t.cachedir/%p/Inputs/multi-<u></u>module-b.o<br>
>> -; RUN: llvm-ar r %t.cachedir/%p/Inputs/load-<u></u>object.a<br>
>> %t.cachedir/%p/Inputs/multi-<u></u>module-c.o<br>
>> +; RUN: llvm-ar r %t.cachedir3/load-object.a %t.cachedir2/multi-module-b.o<br>
>> +; RUN: llvm-ar r %t.cachedir3/load-object.a %t.cachedir2/multi-module-c.o<br>
>><br>
>>  ; This line test MCJIT archive loading<br>
>> -; RUN: %lli_mcjit -extra-archive=%t.cachedir/%p/<u></u>Inputs/load-object.a %s<br>
>> +; RUN: %lli_mcjit -extra-archive=%t.cachedir3/<u></u>load-object.a %s<br>
>><br>
>>  declare i32 @FB()<br>
>><br>
>><br>
>><br>
>> ______________________________<u></u>_________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>