[PATCH] D122922: [lld][common][lld-macho] Support per-thread allocators and StringSavers

[Vy Nguyen via Phabricator via llvm-commits]

oontvoo added inline comments.


================
Comment at: lld/include/lld/Common/CommonLinkerContext.h:73-74
 // in a single process.
+// TODO: How would this affect the TLS variable's addresses?
+// (hopefully not much)
 CommonLinkerContext &commonContext();
----------------
int3 wrote:
> not sure what this comment is about. what's the concern here?
that was from the previous impl (with the mapping). removed now


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122922