<div dir="ltr">No worries. Necessity is an excellent motivator, and I had a huge test case to reduce. ;)<div><br></div><div>- Lang.</div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 7, 2014 at 3:21 PM, Adam Nemet <span dir="ltr"><<a href="mailto:anemet@apple.com" target="_blank">anemet@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for tracking this down, Lang!  I’ve encountered this in the past but didn’t get a chance to look into it.<br>
<span class="HOEnZb"><font color="#888888"><br>
Adam<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Oct 7, 2014, at 2:47 PM, Lang Hames <<a href="mailto:lhames@gmail.com">lhames@gmail.com</a>> wrote:<br>
<br>
> Author: lhames<br>
> Date: Tue Oct  7 16:47:23 2014<br>
> New Revision: 219244<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=219244&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=219244&view=rev</a><br>
> Log:<br>
> [Bugpoint] Close error log in ProcessFailure. Bugpoint had been failing to close<br>
> this, and in some circumstances (e.g. reducing particularly large test-cases)<br>
> this was causing bugpoint to be killed for hitting open file-handle limits.<br>
><br>
> No test case: I was only able to trigger this with test cases taking upwards of<br>
> 10 mins to run.<br>
><br>
><br>
> Modified:<br>
>    llvm/trunk/tools/bugpoint/ToolRunner.cpp<br>
><br>
> Modified: llvm/trunk/tools/bugpoint/ToolRunner.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/bugpoint/ToolRunner.cpp?rev=219244&r1=219243&r2=219244&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/bugpoint/ToolRunner.cpp?rev=219244&r1=219243&r2=219244&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/tools/bugpoint/ToolRunner.cpp (original)<br>
> +++ llvm/trunk/tools/bugpoint/ToolRunner.cpp Tue Oct  7 16:47:23 2014<br>
> @@ -141,21 +141,13 @@ static std::string ProcessFailure(String<br>
><br>
>   // Rerun the compiler, capturing any error messages to print them.<br>
>   SmallString<128> ErrorFilename;<br>
> -  int ErrorFD;<br>
>   std::error_code EC = sys::fs::createTemporaryFile(<br>
> -      "bugpoint.program_error_messages", "", ErrorFD, ErrorFilename);<br>
> +      "bugpoint.program_error_messages", "", ErrorFilename);<br>
>   if (EC) {<br>
>     errs() << "Error making unique filename: " << EC.message() << "\n";<br>
>     exit(1);<br>
>   }<br>
><br>
> -#ifdef _WIN32<br>
> -  // Close ErrorFD immediately, or it couldn't be reopened on Win32.<br>
> -  // FIXME: We may have an option in openFileForWrite(), not to use ResultFD<br>
> -  // but to close it.<br>
> -  delete new raw_fd_ostream(ErrorFD, true);<br>
> -#endif<br>
> -<br>
>   RunProgramWithTimeout(ProgPath, Args, "", ErrorFilename.str(),<br>
>                         ErrorFilename.str(), Timeout, MemoryLimit);<br>
>   // FIXME: check return code ?<br>
><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>
<br>
</div></div></blockquote></div><br></div></div></div>