Updated patch with test case added.  Thanks for catching that!<div><br></div><div><br><div class="gmail_quote">On Sat, Oct 27, 2012 at 1:16 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for the patch!<br>
<br>
Instead of adding a bool, can you just check if Ident is not 0? Also,<br>
can you please add a test case? You can add the whole test suite with<br>
`make test` and an individual test e.g. like `../../utils/lit/lit.py<br>
-v test/Parser/cxx-member-crash.cpp`. Maybe you can just create<br>
test/Parser/cxx-crash.cpp which can become a collection of parser<br>
crash tests.<br>
<br>
Nico<br>
<div class="im"><br>
On Fri, Oct 26, 2012 at 10:07 PM, Brian Brooks <<a href="mailto:brooks.brian@gmail.com">brooks.brian@gmail.com</a>> wrote:<br>
> Attached is a patch for the following bug:<br>
><br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=14085" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14085</a><br>
><br>
><br>
</div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div>