[PATCH] D53032: [clangd] Minimal implementation of automatic static index, behind a flag.

[Sam McCall via Phabricator via cfe-commits]

sammccall added inline comments.


================
Comment at: clangd/ClangdLSPServer.h:117
+        llvm::Optional<Path> CompileCommandsDir,
+        std::function<void(llvm::StringRef,
+                           const tooling::CompilationDatabase &)>);
----------------
ioeric wrote:
> please document what the callback is for and how often it's called.
Documented at the callsite and in GlobalCompilationDatabase.h.
This class here is really just plumbing (it shouldn't be in the header at all)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53032