r218780 - c++ error recovery. Build a valid AST when trying

jahanian fjahanian at apple.com
Wed Oct 1 14:43:51 PDT 2014


On Oct 1, 2014, at 1:47 PM, Richard Smith <richard at metafoo.co.uk> wrote:

> On Wed, Oct 1, 2014 at 11:03 AM, Fariborz Jahanian <fjahanian at apple.com> wrote:
> Author: fjahanian
> Date: Wed Oct  1 13:03:51 2014
> New Revision: 218780
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=218780&view=rev
> Log:
> c++ error recovery. Build a valid AST when trying
> to recover from parse error parsing the default
> argument. Patch prevents crash after spewing 100s
> of errors caused by someone who forgot to compile in c++11
> mode. So no test.
> 
> I don't follow; why no test here? A testcase like this:
> 
>   struct S { void f(int &r = error); };

Thanks. This will trigger it.
In r218830
- Fariborz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141001/601fa147/attachment.html>


More information about the cfe-commits mailing list