<div dir="ltr">For now I'm still making sure we don't assert on stuff.<div>We can eventually add the Diag-style error handling, but that'll end up “just” transforming these fixes I'm doing, it won't change them, so I'm continuing to go through my set of inputs that crash the reader.</div><div><br></div><div>These ones also end up using the error reporting that was already there, so it's not like they're report_fatal_error() calls. They actually tell the caller that there was an error and it can do something (but not that much) about it.</div><div><br></div><div>I might end up starting doing the Diag-style patch soon, but it depends on the other work I have to do. In the meantime I'll keep going over the crash fixes, since most of those are small-ish patches. (Unless there are objections to me doing it now)</div><div><br></div><div>  Filipe</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 2, 2015 at 3:12 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">are you ever planning on switching over to a more "diag" style interface like Rafael suggested? That would allow making these errors much more informative.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7325" target="_blank">http://reviews.llvm.org/D7325</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div><br></div></div>