nridge added a comment.

Thanks for the testcases!

The indirect one is particularly interesting, and suggests that we may want a 
"seen decls" set (similar to `SeenTemplates` here 
<https://searchfox.org/llvm/rev/412141663cfcc7883c24217bcb88780e4572543b/clang-tools-extra/clangd/AST.cpp#944>)
 in the `kindForDecl()` recursion?

(I'd move the check into `resolveUsingValueDecl()`, but that goes against the 
grain of putting the more thorough check in the caller -- or should we do both?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134827

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to