[PATCH] D92484: [clangd] Relation slabs should not be accounted when computing backing storage size

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 3 05:18:07 PST 2020


sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

Thanks, you're right. There's no "payload" memory here, and MemIndex::estimateMemoryUsage should already be accounting for all the memory used.

(Sorry, it's been a while since I looked at this code)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92484



More information about the cfe-commits mailing list