<div dir="ltr">Barring a test case that should successfully perform typo correction, the change seems reasonable for avoiding the null pointer dereference. My only nit is that it would be better to have both test cases in the same file (or even better, in an existing test file with their own enclosing namespace, if there is an appropriate file for it) instead of adding two more files since there is already so many test case files, and IMHO having all the test cases for the same code path in the same file makes it easier to keep track of the test cases.<div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 1, 2015 at 12:33 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Kaelyn, should we do typo correction sooner here?</p><div class="HOEnZb"><div class="h5">
<div class="gmail_quote">On 1 Jun 2015 11:41 am, "Dimitry Andric" <<a href="mailto:dimitry@andric.com" target="_blank">dimitry@andric.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It looks good to me, and it fixes both the original test case in bug 23189, and the reduced test case I added there.<br>
<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D9880&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=ojB8jU_6YRgWlpeGKT-JnWyFR4JaZhGXYEi5pKoWVyw&s=VbolNpISULK5MZPEqLid4NWF4dliBxxOk3KYosqj1Rs&e=" target="_blank">http://reviews.llvm.org/D9880</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=ojB8jU_6YRgWlpeGKT-JnWyFR4JaZhGXYEi5pKoWVyw&s=B2OkqwNrHLLiS5MEFncCqsglC0TUOCZTzTHfIlfk7n0&e=" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div>
</div></div></blockquote></div><br></div></div>