Looks great.  It looks like your script assumes that none of the header files 
in include have methods that issue diagnostics.  Is this true?

On Feb 13, 2012, at 9:40 AM, Dmitri Gribenko <[email protected]> wrote:

> Hi,
> 
> The first attached patch removes unused diagnostics from
> include/clang/Basic/Diagnostic*.td files.
> 
> The only unused diagnostic I left was err_expected_lsquare because it
> fits nicely within a list in an exhaustive list in
> DiagnosticParseKinds.td.
> 
> The second patch adds an utils/find-unused-diagnostics.sh script to
> find such diagnostics automatically.
> 
> Dmitri
> 
> -- 
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/
> <remove-unused-diagnostics-v1.patch><find-unused-diagnostics-tool-v1.patch>_______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to