[PATCH] D138219: [include-cleaner] Show includes matched by refs in HTML report.Demo: https://htmlpreview.github.io/?https://gist.githubusercontent.com/sam-mccall/ecee6869e37af3db28089b64d8dce806/raw/8736e64c45af411e2c2d72adaed2dfc4410a5b36/ASTTests.html%25202

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 23 02:01:05 PST 2022


sammccall added a comment.

In D138219#3946020 <https://reviews.llvm.org/D138219#3946020>, @sammccall wrote:

> 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).

Filed https://github.com/llvm/llvm-project/issues/59147


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138219



More information about the cfe-commits mailing list