steveire added inline comments.
================
Comment at: clang/lib/AST/ParentMapContext.cpp:174
+ {
+ auto AncestorNodes = matchParents<VarDecl, DeclStmt,
CXXForRangeStmt>(
+ ParentList, this);
----------------
aaron.ballman wrote:
> Not needing to be solved in this patch, but do we eventually need to do
> something for `ObjCForCollectionStmt` the same as we do for `CXXForRangeStmt`?
Perhaps. I'm not familiar with it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96113/new/
https://reviews.llvm.org/D96113
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
