[PATCH] D119363: [clang] Add `ObjCProtocolLoc` to represent protocol references
David Goldman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 15 09:37:00 PST 2022
dgoldman added inline comments.
================
Comment at: clang/include/clang/AST/TypeLoc.h:2621
+ /// Evaluates true when this protocol loc is valid/non-empty.
+ explicit operator bool() const { return Protocol; }
+};
----------------
sammccall wrote:
> will we ever have invalid instances?
>From what I can tell, nope. This is explicitly used here though: https://github.com/llvm/llvm-project/blob/ceb5dc55c2e395c085cd9d16c4152c5a1923d7e2/clang/lib/AST/ParentMapContext.cpp#L405
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119363/new/
https://reviews.llvm.org/D119363
More information about the cfe-commits
mailing list