[PATCH] D35406: [clangd] Replace ASTUnit with manual AST management.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 14 08:08:03 PDT 2017


ilya-biryukov added a comment.

In https://reviews.llvm.org/D35406#809623, @malaperle wrote:

> For synching with what I am doing. I am "only" looking right now at the modeling of the index and its on-disk storage, not really on the speeding up of the parsing of the TUs (i.e. the input of the index). I use index::IndexDataConsumer to feed the index and I see that in your change it is still pretty much used the same way as before so there is no problem. I will post on the indexing thread from before with a more detailed proposal from before once it is a bit further along.


Great that this change is not interfering. Eager to see your proposal for the indexing.


https://reviews.llvm.org/D35406





More information about the cfe-commits mailing list