<div dir="ltr">I don't see any obvious problem where we would leak an FD, but maybe this exposed underlying deadlock conditions (which the code already calls out are possible in some cases).<div><br></div><div>I fixed this logic to only capture the output when there are actual failures, which was my original intent anyway, and my guess is that this will help.</div><div><br></div><div>Can you retest the failing scenario after r272365 and let me know if you still see it?</div><div><br></div><div> - Daniel<br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 9, 2016 at 6:21 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</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, Jun 9, 2016 at 1:36 PM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
> On Tue, Jun 7, 2016 at 9:13 AM, Daniel Dunbar via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> Author: ddunbar<br>
>> Date: Tue Jun  7 11:13:40 2016<br>
>> New Revision: 272021<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=272021&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=272021&view=rev</a><br>
>> Log:<br>
>> [lit] Improve logging with file redirection.<br>
>><br>
>>  - This will cause lit to automatically include the first 1K of data in<br>
>>    redirected output files when a command fails (previously if the command<br>
>>    failed, but the main point of the test was, say, a `FileCheck` later on, then<br>
>>    the log wasn't helpful in showing why the command failed).<br>
><br>
> This makes the test suite hang for me on 32-bit Windows builds. It<br>
> runs fine at r272019, but hangs at this revision (and also on<br>
> ToT/r272292).<br>
><br>
> I'm not sure how to debug this. It seems after a while, python.exe<br>
> just sits there waiting for something, with 1, 2, or 0 FileCheck<br>
> processes opened.<br>
<br>
</div></div>Maybe it's a problem of running out of file descriptors. I'm not sure<br>
why this is happening on my machine but not on buildbots, though. If I<br>
run lit with -j16, or set NUMBER_OF_PROCESSORS in the environment, the<br>
tests pass, so I'm not blocked on this anymore.<br>
</blockquote></div><br></div>