<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 29, 2015 at 9:48 AM, Teresa Johnson <span dir="ltr"><<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ok, thanks for the suggestion. I will rework the tests to do that.<br></blockquote><div><br>In case you haven't found it already, %T in the lit syntax gives you a uniquely named directory for the test<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888">Teresa<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Apr 29, 2015 at 9:47 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
><br>
><br>
> On Wed, Apr 29, 2015 at 9:36 AM Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br>
>><br>
>> Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> writes:<br>
>> > On Wed, Apr 29, 2015 at 6:27 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>><br>
>> > wrote:<br>
>> >> On 29 April 2015 at 14:16, Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> wrote:<br>
>> >>> Two of the compiler-rt/profile tests I added are checking for similar<br>
>> >>> behavior with resetting the filename to the default. I wonder if they<br>
>> >>> are running in parallel and clobbering each other since the profile<br>
>> >>> output names are not unique after the reset.<br>
>> >><br>
>> >> Hi Teresa,<br>
>> >><br>
>> >> This would explain the intermittent failures. Maybe making the names<br>
>> >> unique would fix the issues, would that be an easy change?<br>
>> ><br>
>> > After thinking about it I believe this is what is most likely<br>
>> > happening, and that these two new tests I added will need to be<br>
>> > reverted:<br>
>> ><br>
>> ><br>
>> > compiler-rt/trunk/test/profile/instrprof-override-filename-then-reset-default.c<br>
>> ><br>
>> > compiler-rt/trunk/test/profile/instrprof-set-filename-then-reset-default.c<br>
>> ><br>
>> > since they aren't writing to unique output names. These are 2 of the 4<br>
>> > tests added in r236056 (the other two are fine).<br>
>> ><br>
>> > They can't be changed to write to a unique name since those tests were<br>
>> > specifically testing that the profile output name gets reset to the<br>
>> > default when null is passed to the profile filename setting<br>
>> > interfaces. The other profile tests use unique names.<br>
>><br>
>> Could we have these tests `cd` into a uniquely named directory or<br>
>> something? Maybe that's more complicated than its worth.<br>
>><br>
><br>
> That's what we've done in the past and seems a reasonable idea.<br>
><br>
> -eric<br>
><br>
>><br>
>> > Since I don't have write access yet, can someone revert those two<br>
>> > files for me?<br>
>> ><br>
>> > Thanks,<br>
>> > Teresa<br>
>> >><br>
>> >> cheers,<br>
>> >> --renato<br>
>> _______________________________________________<br>
>> LLVM Developers mailing list<br>
>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br>
<br>
<br>
</div></div><span class="im HOEnZb">--<br>
Teresa Johnson | Software Engineer | <a href="mailto:tejohnson@google.com">tejohnson@google.com</a> | <a href="tel:408-460-2413" value="+14084602413">408-460-2413</a><br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br></div></div>