<div dir="ltr">Note that as far as places to put temporary files, the right place to put them is alongside the other test outputs in the test output "sandbox" directory.<div><br></div><div style>Somewhat orthogonal, but we should also fix up lit to purge those sandboxes before it starts a new test run.</div>
<div style><br></div><div style> - Daniel</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 16, 2013 at 3:31 PM, Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Jan 17, 2013 at 1:23 AM, Evgeniy Stepanov<br>
<<a href="mailto:eugeni.stepanov@gmail.com">eugeni.stepanov@gmail.com</a>> wrote:<br>
> On Thu, Jan 17, 2013 at 1:19 AM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br>
>> On Wed, Jan 16, 2013 at 9:33 PM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br>
>>> On Wed, Jan 16, 2013 at 9:24 PM, Chris Lattner <<a href="mailto:clattner@apple.com">clattner@apple.com</a>> wrote:<br>
>>>><br>
>>>> On Jan 16, 2013, at 10:32 AM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:<br>
>>>><br>
>>>>> Hello,<br>
>>>>><br>
>>>>> When someone breaks a FileCheck-based test on some buildbot, sometimes<br>
>>>>> it may not be obvious *why* did it fail.  If the failure can not be<br>
>>>>> reproduced locally, it can be very hard to fix.<br>
>>>>><br>
>>>>> I propose adding a "very verbose" mode to FileCheck.  In this mode<br>
>>>>> FileCheck will dump the input file in case of failure.  This mode will<br>
>>>>> be enabled by an environment variable "FILECHECK_VERY_VERBOSE".  If we<br>
>>>>> chose a command line option, we would have to edit all FileCheck-based<br>
>>>>> tests to use %FileCheck.<br>
>>>><br>
>>>> I think that this idea is good, but I'd prefer it be implemented a different way:<br>
>>>><br>
>>>>  - Filecheck should take a new flag -dump-input-on-error that causes it to... dump the input file on error.<br>
>>>>  - Lit should be the thing that checks the environment (or perhaps add a new option to lit), and adds the flag to FileCheck invocations.<br>
>>>><br>
>>>> I don't like it when the behavior of such a low-level tool like this changes based on environment variables.  It isn't discoverable in --help.  If for some reason, it is bad for lit to implicitly pass the option, I'd rather have a standard FILECHECK_COMMANDLINE environment variable, and have filecheck parse arbitrary options out of it using the cl::ParseEnvironmentOptions function.<br>

>>><br>
>>> I agree that a command line option would be better.  But in that case<br>
>>> all tests should be updated.  It is not an issue for me -- it is<br>
>>> mostly mechanical.  So should I change tests to use %FileCheck?<br>
>><br>
>> Here's a third attempt.<br>
>><br>
>> The new behavior is as follows:<br>
>><br>
>> 1. In case of errors we always dump output to a temporary file and print<br>
><br>
> Does it mean we get one more file in /tmp every time a test fails, and<br>
> it is not cleaned up automatically? I don't think this should happen<br>
> in the "default" mode of the tool.<br>
<br>
</div></div>Well, yes.  David requested that and I agreed that it is a good idea.<br>
Are you strongly opposed to it?<br>
<div class="HOEnZb"><div class="h5"><br>
Dmitri<br>
<br>
--<br>
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>*/<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>