[cfe-commits] [patch] (Was: Assertion failure on invalid code.)

[Argyrios Kyrtzidis]

On Jan 9, 2013, at 2:24 AM, Enea Zaffanella <zaffanella at cs.unipr.it> wrote:

> On 01/08/2013 07:28 PM, Argyrios Kyrtzidis wrote:
>> On Jan 8, 2013, at 5:26 AM, Enea Zaffanella <zaffanella at cs.unipr.it> wrote:
> [...]
>>> Please find attached a patch fixing the crash mentioned above.
>>> The patch includes minor changes to 3 tests, where (as expected) we no longer get further diagnostics on the code already marked as invalid.
>> 
>> Could you also add a test case for the crash ?
> 
> Here is the revised patch.
> The currently crashing test (which is a minor variant of the one posted) is test5_inst at the end of test/SemaCXX/condition.cpp.

LGTM.

> 
> Enea.
> 
> <CrashFix-2.patch>