[PATCH] D107933: [clang] Expose unreachable fallthrough annotation warning

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 16 12:55:41 PDT 2021


dblaikie added a comment.

Patch description probably needs an update - looks like it's out of date ("Add -Wimplicit-fallthrough-unreachable, which is default enabled with -Wimplicit-fallthrough" should read, I guess "Add -Wunreachable-code-fallthrough, which is default enabled with -Wunreachable-code" - also, I might avoid using the "default enabled" as it may lead to some confusion/sound like it's suggesting that "this subwarning is on by default as it's grouped under this other on-by-default warning", which isn't the case - maybe "which is enabled with -Wunreachable-code" would suffice? or "which is part of the -Wunreachable-code group")


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107933



More information about the cfe-commits mailing list