[cfe-commits] r164677 -

[Andy Gibbs]

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 
http://lists.cs.uiuc.edu/pipermail/cfe-commits/attachments/20121002/824367ac/attachment-0001.obj)

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