[PATCH] D56325: Sort symbols in .bss by size.

[Vic Yang via Phabricator via llvm-commits]

victoryang added a comment.

In D56325#1367138 <https://reviews.llvm.org/D56325#1367138>, @ruiu wrote:

> George,
>
> 0.5% is not a micro-optimization if it works for many applications. If you find a heuristic that can generally reduce memory usage by 0.5%, that's not negligible. Imagine for example 0.5% memory usage reduction in a huge cloud application. You might be able to save multi million dollars.
>
> Vic,
>
> That said, the numbers you've shown so far don't honestly seem very convincing. The number of data points is too few that I can't see if it generally works. I believe I can test your patch myself and see how it works for Linux applications by parsing /proc/self/smaps (which contains information about the number of dirty pages). Let me try to do that for a few applications like clang.


Rui,

Did you get a chance to try this out?


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

https://reviews.llvm.org/D56325