ioeric added inline comments.
================
Comment at: clangd/index/FileIndex.h:121
std::pair<SymbolSlab, RefSlab>
indexAST(ASTContext &AST, std::shared_ptr<Preprocessor> PP,
llvm::ArrayRef<std::string> URISchemes = {});
----------------
sammccall wrote:
> indexPreamble would be clearer I think.
What about `indexHeaderSymbols`? `indexPreamble` seems to require understanding
of how preamble is built, but it doesn't seem to matter here if `AST` was built
from preamble or not.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52222
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
