[PATCH] D47063: [clangd] Keep only a limited number of idle ASTs in memory

Marc-Andre Laperle via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 18 07:56:10 PDT 2018


malaperle added subscribers: arphaman, malaperle.
malaperle added a comment.

In https://reviews.llvm.org/D47063#1104417, @ilya-biryukov wrote:

> Maybe we can even store enough information to not need the AST for navigation at all and build it only for features like refactorings.
>  @sammccall, let me know what are your thoughts on all of this.


That's what I was thinking. I think I also had similar discussion with @arphaman. I think storing a limited number of ASTs is a good interim solution.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D47063





More information about the cfe-commits mailing list