<div dir="ltr">I think this leaks memory and a handle if you never wait on the program now.<div><br></div><div style>I'm working on a more complete fix.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jun 13, 2013 at 10:42 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think I've fixed the issue in r183904 -- there was an uninitialized<br>
variable being passed into Execute, so anything we tried to execute<br>
would end up crashing.  I'll watch the build bots.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Thu, Jun 13, 2013 at 10:32 AM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
> For me they center around 'not.exe' crashing.  I don't have a stack trace<br>
> yet because I have to rebuild with debug info =/.<br>
><br>
><br>
> On Thu, Jun 13, 2013 at 6:35 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>><br>
> wrote:<br>
>><br>
>> Hi Rafael,<br>
>><br>
>> Even though Clang now builds fine on Windows, I observe crashes while<br>
>> building simple Windows tests on my local bot.<br>
>> Unfortunately, I'm out of the office today so can't help you diagnose.<br>
>><br>
>> Reid, Takumi - can you help Rafael?<br>
>><br>
>> 2013/6/13 Rafael Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>>:<br>
>> > Author: rafael<br>
>> > Date: Wed Jun 12 16:25:04 2013<br>
>> > New Revision: 183868<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=183868&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=183868&view=rev</a><br>
>> > Log:<br>
>> > [Win] Put ChangeStdinToBinary ChangeStdoutToBinary ChangeStderrToBinary<br>
>> > in sys.<br>
>> ><br>
>> > Modified:<br>
>> >     llvm/trunk/lib/Support/Windows/Program.inc<br>
>> ><br>
>> > Modified: llvm/trunk/lib/Support/Windows/Program.inc<br>
>> > URL:<br>
>> > <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Windows/Program.inc?rev=183868&r1=183867&r2=183868&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Windows/Program.inc?rev=183868&r1=183867&r2=183868&view=diff</a><br>

>> ><br>
>> > ==============================================================================<br>
>> > --- llvm/trunk/lib/Support/Windows/Program.inc (original)<br>
>> > +++ llvm/trunk/lib/Support/Windows/Program.inc Wed Jun 12 16:25:04 2013<br>
>> > @@ -421,21 +421,21 @@ static int Wait(void *&Data, const Path<br>
>> >  }<br>
>> ><br>
>> >  namespace llvm {<br>
>> > -error_code ChangeStdinToBinary(){<br>
>> > +error_code sys::ChangeStdinToBinary(){<br>
>> >    int result = _setmode( _fileno(stdin), _O_BINARY );<br>
>> >    if (result == -1)<br>
>> >      return error_code(errno, generic_category());<br>
>> >    return make_error_code(errc::success);<br>
>> >  }<br>
>> ><br>
>> > -error_code ChangeStdoutToBinary(){<br>
>> > +error_code sys::ChangeStdoutToBinary(){<br>
>> >    int result = _setmode( _fileno(stdout), _O_BINARY );<br>
>> >    if (result == -1)<br>
>> >      return error_code(errno, generic_category());<br>
>> >    return make_error_code(errc::success);<br>
>> >  }<br>
>> ><br>
>> > -error_code ChangeStderrToBinary(){<br>
>> > +error_code sys::ChangeStderrToBinary(){<br>
>> >    int result = _setmode( _fileno(stderr), _O_BINARY );<br>
>> >    if (result == -1)<br>
>> >      return error_code(errno, generic_category());<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>
>> 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>
><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>