ilya-biryukov added a comment. In https://reviews.llvm.org/D47272#1110958, @ioeric wrote:
> Would it be possible to add some integration tests for file index plus > preamble? Sure, will do ================ Comment at: clangd/index/FileIndex.cpp:37 + std::vector<Decl *> TopLevelDecls( + AST.getTranslationUnitDecl()->decls().begin(), + AST.getTranslationUnitDecl()->decls().end()); ---------------- ioeric wrote: > Would this give us decls in namespaces? It seems that > `AST.getTranslationUnitDecl()->decls()` are only decls that are immediately > inside the TU context. I'll double check and add a test for that, but I think the indexer visits the namespaces that we provide, I'm not sure if we have tests though. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47272 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
