[PATCH] D58271: [ELF] --gdb-index: split off GdbSymbol::CuVector and add a separate CuVectors

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

> I have a strong belief: (I actually wanted to call it impossible but just didn't want to make an absolute assertion :) ) we can't decrease the memory usage of `.gdb_index` without a performance hit. I sorta blame the function has been optimized tell well for performance :( If we can emphasize less on performance, https://reviews.llvm.org/D58276 (and https://reviews.llvm.org/differential/diff/187004) is a more feasible direction to decrease memory usage (internally, we care memory usage a lot. The performance is already very good (2x ~ 4x faster compared with gold for a wide range of applications). Some sacrifice on it is totally acceptable, but on the other hand, we have some hard memory usage limits and the current memory footprint characterstics make some huge targets unable to link).

That is my perception too; it seems nearly impossible to reduce the memory usage without sacrificing speed.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D58271