[PATCH] D21257: Reduce size in memory. This is very helpful in a multi-module setups.

Lang Hames via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 12 18:55:39 PDT 2016


lhames added a comment.

Hi Vassil,

Sorry for the delayed reply.

What's the rationale behind this change? My (very limited) understanding of unordered_map is that '4' is the initial number of buckets to create, but I would have expected that to raise the memory consumption (assuming less than four are needed in practice) or leave it unaffected?

What do you mean by a multi-module use case? (Multiple modules in a single RuntimeDyld instance?)
Have you profiled other use-cases?

Cheers,
Lang.


Repository:
  rL LLVM

https://reviews.llvm.org/D21257





More information about the llvm-commits mailing list