[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