<div dir="ltr">Changing to exit(-3) sounds good to me.<div><br></div><div>There's more work here to be done here, though.<div><div><br></div><div>I complicated things by adding yet another env var called LLVM_ENABLE_CRT_REPORT, which tries to suppress the "abort, retry, ignore" dialog raised by assertion failures when using the Visual CRT.  I think both of these vars are basically trying to achieve the same behavior of suppressing dialogs and slow system forensics gathering, so they could be merged.</div>
<div><br></div><div>Currently, LLVM_DISABLE_CRASH_REPORTING turns off stack dumping on Windows but not on Mac.  This seems unintentional to me, and it means we don't get crash stack traces from 'check-clang' on Windows.  It'd be nice to get those without a dialog.</div>
</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 25, 2013 at 9:40 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@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="im">On 25 July 2013 11:54, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>

> Wow, this code is pretty old (r82537 by ddunbar)!  I'm not certain<br>
> removing it is the correct approach though -- the point to that is to<br>
> disable crash reporting, and by removing it, the crash dialogs will<br>
> appear again on system with post mortem debuggers installed.<br>
<br>
</div>ah, so that is why I was not getting a dialog. Does MSVC ship with a<br>
post mortem debugger? Do I have to do something to enable it?<br>
<div class="im"><br>
> On Windows, crashing really does exit much like exiting with a regular<br>
> error, IIRC.  The resulting code (checked with %ERRORLEVEL%) is<br>
> usually the exception code, but there's nothing magical about process<br>
> return values except for 0.<br>
<br>
</div>I have attached a log showing the difference I get in one problematic<br>
test (badpch.c). The lest invasive fix would probably be to change<br>
<br>
 _exit(3);<br>
<br>
to<br>
 _exit(-3);<br>
<br>
but we would still be trowing away a bit of information.<br>
<div class="im"><br>
> I'm CCing in some folks who may have an opinion.<br>
><br>
> ~Aaron<br>
<br>
</div>Cheers,<br>
Rafael<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></blockquote></div><br></div>