[PATCH] D73852: [clang] detect switch fallthrough marked by a comment (PR43465)
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 2 08:42:03 PST 2020
rsmith added a comment.
I'm also pretty concerned about using comments to drive warning behavior. We discussed this when first adding our gallery fallthrough warning and its suppression mechanism and made an explicit decision that we did not want comments to affect our behaviour. I don't think anything has changed in that regard.
We shouldn't enable the warning under -Wextra in language modes where there's no standard way to suppress it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73852/new/
https://reviews.llvm.org/D73852
More information about the cfe-commits
mailing list