[PATCH] D59605: [clangd] Introduce background-indexer

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 11 09:13:36 PDT 2019


ilya-biryukov added a subscriber: sammccall.
ilya-biryukov added a comment.

To reiterate the offline discussion: the tool seems useful to me, but it would be best to keep the client code simpler, it's currently fighting with `BackgroundIndex` because it's trying to hijack some of its functionality.
More specifically, I propose to add a function to `BackgroundIndex.h` that would be a one-shot update of the index, the client code would become significantly simpler and we would have more flexibility in how we move code around in `BackgroundIndex.cpp` to actually make this happen.

Adding @sammccall, who might also be interested in this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59605/new/

https://reviews.llvm.org/D59605





More information about the cfe-commits mailing list