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

Sam McCall via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 11 10:59:13 PDT 2019


Thanks - I think there's quite a lot of background missing here: what/who
is this for, why a separate binary from clangd-indexer (or and how do they
relate, how much of the BackgroundIndex code structure makes sense with
multiple users.

I assume there's been some offline discussion about this, but i missed it
:-)

On Thu, Apr 11, 2019, 18:16 Ilya Biryukov via Phabricator <
reviews at reviews.llvm.org> wrote:

> ilya-biryukov added a comment.
>
> To make it clear, I think the question is not just "which part of
> functionality is missing in BackgroundIndex", it's rather "which part of
> BackgroundIndex we **don't** need".
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D59605/new/
>
> https://reviews.llvm.org/D59605
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190411/d6a5be2a/attachment.html>


More information about the cfe-commits mailing list