[PATCH] D104071: [AST] Include the TranslationUnitDecl when traversing with TraversalScope

[Sam McCall via Phabricator via cfe-commits]

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