[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