[cfe-commits] [PATCH] expected-no-diagnostics for -verify (was: Re: r164677 - )

[Andy Gibbs]

** bump ** :o)

I've had to resync and attach the patch again here; a test-case needed to be 
updated to reflect changes in trunk.  The patch now is taken against 
r165609.

Cheers
Andy


On Saturday, October 06, 2012 10:59 PM, Andy Gibbs wrote:
> Hi,
>
> Was there any interest in this patch?  Please let me know if there are any 
> changes that should be made, or whether it is good to be committed? (or 
> even whether I was barking up the wrong tree...?!)
>
> (If any were unable to receive the patch -- I know that sometimes Apple 
> users have problems with my attachments! -- then it can be viewed directly 
> at [...snip...])
>
> Cheers
> Andy
>
>
> On Tuesday, October 02, 2012 5:32 PM, Andy Gibbs wrote:
>> All,
>>
>> Attached is a patch that adds an 'expected-no-diagnostics' directive
>> to -verify.  This can then be used, as discussed, to flag files that are 
>> not
>> expected to produce any diagnostics at all.  -verify has then been 
>> changed
>> to generate an error if no expected-* directives are found (for example, 
>> if
>> the RUN line doesn't specify the source file!).  I've also made it so 
>> that
>> the user cannot put 'expected-no-diagnostics' in with other expected-*
>> directives (to avoid mistakes!).
>>
>> The complete patch is rather large, but I've grouped the changes to
>> VerifyDiagnosticConsumer to the top of the file along with the additional
>> test-case.  The rest of the patch makes the necessary changes to fix 580
>> test-cases which now (in almost all cases) need to have the
>> 'expected-no-diagnostics' directive.
>>
>> Comments / questions welcome!
>>
>> Andy 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: verify.diff
Type: application/octet-stream
Size: 277806 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121010/ace3da29/attachment.obj>