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

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 27 03:03:40 PST 2020


sammccall added a comment.

Why do we want to do this?
The existing estimate covers the existing loaded data.
This change prompts it to load data that we think we might need soon before estimating, but I'm not sure why that's a good thing.

(actually I don't think it'll work as if it causes new indexes to be loaded, they'll load asynchronously and report 0 here)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92201



More information about the cfe-commits mailing list