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

2019-04-12 Thread Ilya Biryukov via cfe-commits
Replies inline. On Thu, Apr 11, 2019 at 7:59 PM Sam McCall wrote: > Thanks - I think there's quite a lot of background missing here: > Indeed, here are clarifications. > what/who is this for, > Building the index for clangd without actually running clangd it seems useful, e.g. for cases when yo

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

2019-04-11 Thread Sam McCall via cfe-commits
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