[PATCH] D35284: Support: Add llvm::format_memory to convert number of bytes to human readable string.

Frederich Munch via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 11:57:17 PDT 2017


marsupial added a comment.

@ruiu
What about an optional argument to specific IEC suffixes.
Using IEC by default is fine by me, just that there is a way to not do so.

I agree the name is probably too general,  but format_binary_prefix is a bit obtuse.
What about **format_memory_size** instead?


https://reviews.llvm.org/D35284





More information about the llvm-commits mailing list