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

Ilya Golovenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 3 05:23:32 PST 2020


ilya-golovenko added a comment.

In D92484#2430985 <https://reviews.llvm.org/D92484#2430985>, @sammccall wrote:

> 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)

Thank you for review!


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