[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
Thu Mar 1 10:44:53 PST 2018
JonasToth added a comment.
After long inactivity (sorry!) i had a chance to look at it again:
switch(i) {
case 0:;
case 1:;
case 2:;
...
}
does *NOT* lead to the stack overflow. This is most likely an issue in the AST:
https://godbolt.org/g/vZw2BD
Empty case labels do nest, an empty statement prevents this. The nesting leads most likely to the deep recursion. I will file a bug for it.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D40737
More information about the cfe-commits
mailing list