[PATCH] D104071: [AST] Include the TranslationUnitDecl when traversing with TraversalScope
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 10 16:12:15 PDT 2021
sammccall added a comment.
This includes both the traversal change and the clangd changes to adapt to it, since there ended up being few.
I chickened out of the idea of changing the behavior of TranslationUnitDecl::decls(), for a couple of reasons:
- having it apply to RecursiveASTVisitor only is similar to the previous version, so this is a slightly less invasive/risky change
- it's actually slightly awkward to change the node's behavior as decls() is not virtual in DeclContext
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104071/new/
https://reviews.llvm.org/D104071
More information about the cfe-commits
mailing list