[PATCH] D47272: [clangd] Build index on preamble changes instead of the AST changes

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 23 11:35:56 PDT 2018


ilya-biryukov added a comment.

In https://reviews.llvm.org/D47272#1109909, @malaperle wrote:

> > We do not to rely on symbols from the main file anyway, since any info hat those provide can always be taken from the AST.
>
> I'll be adding those soon for workspace symbols... And also for document symbols.


I can add extra code to build pieces for the AST later. This is not hard to do, but would require rearranging some code a bit more.
Will try to send the change tomorrow for review tomorrow. Does that SG?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D47272





More information about the cfe-commits mailing list