sammccall added a comment.

(I do think the outstanding issues are not related to this patch, so this is 
ready for review)

In D138219#3945954 <https://reviews.llvm.org/D138219#3945954>, @hokein wrote:

> E.g. for the following case, the `UsingShadowDecl` refers to the *primary* 
> template decl, which is not marked as used or referenced. The used/referenced 
> bit is only set for specialized `FunctionDecl`, so WalkAST doesn't report 
> this UsingDecl location. (`EnumDecl` also has this problem).

That makes sense, I can imagine a few ways to fix this (change the bits in the 
AST, walk over specializations, drop the used||referenced approach).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138219

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

Reply via email to