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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
