<div dir="ltr"><br><br><div class="gmail_quote">On Wed, Apr 29, 2015 at 9:36 AM Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Teresa Johnson <<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>> writes:<br>
> On Wed, Apr 29, 2015 at 6:27 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br>
>> On 29 April 2015 at 14:16, Teresa Johnson <<a href="mailto:tejohnson@google.com" target="_blank">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>
>     compiler-rt/trunk/test/profile/instrprof-override-filename-then-reset-default.c<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></blockquote><div><br></div><div>That's what we've done in the past and seems a reasonable idea.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> 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" target="_blank">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>
</blockquote></div></div>