On 8 January 2013 05:10, Alexander Kornienko <[email protected]> wrote: > Should be fixed in r171768. > > Thank you for noticing this!
Thanks for the fix. Is there some way I should have been testing the build in order to enable that warning? I didn't see it in my cmake/ninja build. > On Mon, Jan 7, 2013 at 7:08 PM, Dmitri Gribenko <[email protected]> wrote: >> >> On Mon, Jan 7, 2013 at 7:53 PM, Alexander Kornienko <[email protected]> >> wrote: >> > Author: alexfh >> > Date: Mon Jan 7 11:53:08 2013 >> > New Revision: 171760 >> > >> > URL: http://llvm.org/viewvc/llvm-project?rev=171760&view=rev >> > Log: >> > Implement Attr dumping for -ast-dump. >> > http://llvm-reviews.chandlerc.com/D234 >> > >> > Patch by Philip Craig! >> >> Hello Alexander, >> >> It looks like this commit introduced a few warnings like this: >> >> tools/clang/include/clang/AST/AttrDump.inc:80:5: warning: default >> label in switch which covers all enumeration values >> [-Wcovered-switch-default] >> default: >> ^ >> >> 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]>*/ > > > _______________________________________________ > 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
