[PATCH] D61288: [Diagnostics] Implemented support for -Wswitch-default

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 1 07:01:45 PDT 2019


xbolva00 added a comment.

I am not familiar with clang-tidy’s codebase and I personally prefer good compiler warnings than dependency on another tool (clang-tidy). I mean the cases when diagnostic check is easy to do in the compiler.

And in semaexpr we have all we need and it is simple to do it.

But if majority of reviewers disagree with compiler warning, I will close this revision.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61288/new/

https://reviews.llvm.org/D61288





More information about the cfe-commits mailing list