[Lldb-commits] [PATCH] D117914: [lldb] Add ConstString memory usage statistics

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere added a comment.

In D117914#3262467 <https://reviews.llvm.org/D117914#3262467>, @kastiglione wrote:

> I find "Unallocated" ambiguous. To the Allocator, maybe it's not, but to VM it is. What about Total/Used/Unused? Note that the docs for `BytesAllocated` say it can be used to calculated "wasted" space.
>
>   /// How many bytes we've allocated.
>   ///
>   /// Used so that we can compute how much space was wasted.
>   size_t BytesAllocated = 0;

I got the original terminology from the dumpStats function on the BumpPtrAllocator. I like Total/Used/Unused because there's less ambiguity.


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

https://reviews.llvm.org/D117914