[cfe-commits] r171760 - in /cfe/trunk: include/clang/AST/CMakeLists.txt include/clang/AST/Makefile lib/AST/ASTDumper.cpp lib/AST/CMakeLists.txt test/Misc/ast-dump-attr.cpp test/Tooling/clang-check-ast-dump.cpp utils/TableGen/ClangAttrEmitter.cpp utils/TableGen/TableGen.cpp utils/TableGen/TableGenBackends.h

[Philip Craig]

On 8 January 2013 05:10, Alexander Kornienko <alexfh at google.com> 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 <gribozavr at gmail.com> wrote:
>>
>> On Mon, Jan 7, 2013 at 7:53 PM, Alexander Kornienko <alexfh at google.com>
>> 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 <gribozavr at gmail.com>*/
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>