[PATCH] D145842: [clang][Sema] Avoid duplicate diagnostics for unreachable fallthrough attribute
Takuya Shimizu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 13 09:18:39 PDT 2023
hazohelet updated this revision to Diff 504697.
hazohelet added a comment.
Address comments from @aaron.ballman
- Dropped `const` qualifier from value objects for style consistency
- Passed boolean `HasFallThroughAttr` to callback and handled warning suppression in `HandleUnreachable` in order to make code cleaner.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145842/new/
https://reviews.llvm.org/D145842
Files:
clang/include/clang/Analysis/Analyses/ReachableCode.h
clang/lib/Analysis/ReachableCode.cpp
clang/lib/Sema/AnalysisBasedWarnings.cpp
clang/test/Sema/warn-unreachable-fallthrough.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145842.504697.patch
Type: text/x-patch
Size: 4685 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230313/c5666289/attachment.bin>
More information about the cfe-commits
mailing list