[PATCH] D41276: [clangd] Build in-memory index on symbols in files.

[Sam McCall via Phabricator via cfe-commits]

sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

I don't see `FileSymbols.{h,cpp}` being deleted, but maybe I'm bad at reading diffs.



================
Comment at: clangd/index/FileIndex.h:8
+//
+//===----------------------------------------------------------------------===//
+
----------------
Add a file-comment about what this is and how it's going to be used?
At the moment almost all the comments are on FileSymbols and implementation-focused, while FileIndex is the interesting class.


================
Comment at: clangd/index/FileIndex.h:57
+
+  bool fuzzyFind(Context &Ctx, const FuzzyFindRequest &Req,
+                 std::function<void(const Symbol &)> Callback) const;
----------------
override


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41276