[PATCH] D40737: [clang-tidy] Resubmit hicpp-multiway-paths-covered without breaking test
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Mar 18 07:32:40 PDT 2018
JonasToth added a comment.
I think the check is ready to land. I did check run it over LLVM and found some interesting code parts that could benefit. I extracted all the warnings and sorted them into the categories `missing default/covered codepath(uncovered.txt)`, `better use if/else(better_if.txt)` and `use if(mandatory_if.txt)`. Some warnings come from generated files but some live in usercode. In total there are 540 warnings.
Please note, that i identified a common pattern to leave out the `default` label and add an ending `llvm_unreachable` that is outside the switch but ensures there is no unwanted fall_through. Most of the warnings for the first category are like this, but i could not check all of them!
F5899267: uncovered.txt <https://reviews.llvm.org/F5899267>
F5899266: better_if.txt <https://reviews.llvm.org/F5899266>
F5899265: mandatory_if.txt <https://reviews.llvm.org/F5899265>
rCTE Clang Tools Extra
More information about the cfe-commits