<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 22, 2014 at 4:03 PM, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On 9 April 2014 18:40, Reid Kleckner <span dir="ltr"><<a href="mailto:reid@kleckner.net" target="_blank">reid@kleckner.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Author: rnk<br>
Date: Wed Apr  9 20:40:15 2014<br>
New Revision: 205948<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=205948&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=205948&view=rev</a><br>
Log:<br>
Avoid crashing when failing to emit a thunk<br>
<br>
If we crash, we raise a crash handler dialog, and that's really<br>
annoying.  Even though we can't emit correct IR until we have musttail,<br>
don't crash.<br></blockquote><div><br></div></div><div>Two questions, does report_fatal_error raise the dialog? And what happened to running the tests under KillTheDoctor to prevent this?</div></div></div></div></blockquote>
<div><br></div><div>No, we were dereferencing null here.</div><div><br></div><div>I wasn't running tests, I was running clang with fallback, so KillTheDoctor is no help.  We have *another* mechanism to suppress the dialogs, which is to add LLVM_DISABLE_CRASH_REPORT to the environment.  I can't use that because Chromium's build system resets all environment variables when running the compiler.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Third question, you say "don't crash" but then what *does* it do?</div>
</div></div></div></blockquote><div><br></div><div>We've already emitted an error.  This code will generate bogus IR that will fail verification.  If you review musttail, I can delete it.  ;)</div></div></div></div>