[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 06:24:52 PDT 2019


xbolva00 added a comment.

Some coding guidelines may require switch to have always default label. Even if devs know that default is not reachable, they can add default: abort(); or assert to increase safety (and warning will be silenced).

Yes, it not suitable to be enabled by default, but I still think it is good to have it.


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

https://reviews.llvm.org/D61288





More information about the cfe-commits mailing list