[PATCH] D119784: [Symbolize] LRU cache binaries in llvm-symbolizer.

[Daniel Thornburgh via Phabricator via llvm-commits]

mysterymath added a comment.

In D119784#3334359 <https://reviews.llvm.org/D119784#3334359>, @dblaikie wrote:

> Or would it be OK to evict the last thing even as the next thing is being added to the cache - before all the derived objects are created from the new object? Would that be adequate?

I think it'd be pretty reasonable to load in the second binary before we evict the first; mapping in two large binaries isn't all that worse than one, especially since there currently isn't any bound at all.
If that's OK, we can get this effect by just always keeping the most recently used binary around. This should be much simpler to code, and it'll only slightly delay evicting the binary to the next `pruneCache()` call.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119784