[PATCH] D59269: ELF: Use bump pointer allocator for uncompressed section buffers. NFCI.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 12 12:40:48 PDT 2019


ruiu added a comment.

I thought that we could protect `BumpPtrAllocator::Allocate` with a mutex, but since StringSaver directly calls that function, there's no way to acquire a lock before calling that function. Maybe we should protect `StringSaver::save`? Or, we could also make BAlloc and StringSaver thread-local variables. That might be a better approach.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59269





More information about the llvm-commits mailing list