[PATCH] D92201: [clangd] Make sure project-aware index is up-to-date for estimateMemoryUsage()

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added a comment.

Hm, yes, this makes sense. I just found it awkward that memory usage estimate will spike right after the first request but that's actually the reality (loading the index is deferred until the first actual index request). My confusion here is that I don't really understand whether deferring the index loading is a good thing because we're practically moving the delay from the warm-up to the working state which is suboptimal. I was trying to cause the index to load in the initialization stage for D92198 <https://reviews.llvm.org/D92198> but couldn't figure out where exactly the config is loaded and what could be the best place for triggering index loading.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92201/new/

https://reviews.llvm.org/D92201