danielmarjamaki added a comment. In https://reviews.llvm.org/D24905#557573, @dcoughlin wrote:
> Sorry, missed this patch. > > I think it would good to add a test to make sure we do warn when the var decl > has an initializer, since that will not be executed. > > void varDecl(int X) { > switch (X) { > int A = 12; // We do want a warning here, since the variable will be > uninitialized in C (This is not allowed in C++). > case 1: > ... > break; > } > } > I added such test case with 283096. Repository: rL LLVM https://reviews.llvm.org/D24905 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits