[prev in list] [next in list] [prev in thread] [next in thread]
List: cfe-commits
Subject: Re: [PATCH] Fix crash when both Parser::ExpectAndConsume and ConsumeAnyToken throw a diagnostic
From: Alp Toker <alp () nuanti ! com>
Date: 2014-02-22 18:31:34
Message-ID: 5308ED06.3040605 () nuanti ! com
[Download RAW message or body]
On 22/02/2014 14:08, Olivier Goffart wrote:
> Hi,
>
> I believe this is a regression from r198270
>
Indeed the lifetime of the DiagnosticBuilder shouldn't have overlapped
the call to ConsumeAnyToken().
> --- /dev/null
> +++ b/test/Parser/diag-crash.c
> @@ -0,0 +1,5 @@
> +// RUN: %clang_cc1 -fsyntax-only -verify %s
> +
> +// Don't crash if the preprocessor is throwing a diagnostic from Parser::ExceptAndConsume
Typo in the comment.
> +int foo: // expected-error {{expected ';' after top level declarator}}
> +#endif // expected-error {{#endif without #if}}
> \ No newline at end of file
Looks good otherwise, do you have commit access?
Alp.
--
http://www.nuanti.com
the browser experts
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic