[PATCH] D51539: [clangd] Add symbol slab size to index memory consumption estimates

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet added inline comments.


================
Comment at: clang-tools-extra/clangd/index/dex/DexIndex.cpp:184
+  size_t Bytes = PairedSlabSize;
+  Bytes += LookupTable.size() * sizeof(std::pair<SymbolID, const Symbol *>);
   Bytes += SymbolQuality.size() * sizeof(std::pair<const Symbol *, float>);
----------------
Why not use `LookupTable.getMemorySize()` directly? Is it intentionally for not counting non-used but allocated buckets? Same for the below two as well.


https://reviews.llvm.org/D51539