dcoughlin added a comment.

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;
    }
  }


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

Reply via email to