[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'm sorry, I sent it prematurely.)

That is my perception too; it seems nearly impossible to reduce the memory usage without sacrificing speed. If memory consumption is a problem for most users, we probably should choose memory reduction over speed, but I don't think our use case within Google is strong enough to change that design choice. In lld, we parallelize things if we are handling a massive number of the same kind of objects, and this perfectly matches that pattern. It'd be pretty odd if we don't do this only this place.

Fortunately there is a workaround for it: we could build a binary without --gdb-index and then add it using gdb as a post-processing. Maybe we should live with that.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D58271