[PATCH] D55640: [clang-tidy] Implement a check for large Objective-C type encodings 🔍
Stephane Moore via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 28 20:24:23 PST 2019
stephanemoore planned changes to this revision.
stephanemoore added a comment.
Reiterating outstanding action items:
• Evaluate using hasDeclContext instead of hasAncestor.
• Include type encoding size information in diagnostic messages.
I've been observing some unexpected behaviors with matching in the proposed test sample that I am still investigating (as I recall the issue pertained to interfaces/protocols without interfaces).
Including the type encoding size information in diagnostic messages shouldn't be too hard (assuming we can get precise type encoding size information by analyzing the AST).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55640/new/
https://reviews.llvm.org/D55640
More information about the cfe-commits
mailing list