[cfe-commits] [PATCH] Remove unused diagnostics

Ted Kremenek kremenek at apple.com
Mon Feb 13 10:33:13 PST 2012


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 <gribozavr at gmail.com> 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 <gribozavr at gmail.com>*/
> <remove-unused-diagnostics-v1.patch><find-unused-diagnostics-tool-v1.patch>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list