[PATCH] D147649: [Reland][ADT][ConcurrentHashTable] adapt ConcurrentHashTable and its users to LLVM_ENABLE_THREADS=0 mode.

[Alexey Lapshin via Phabricator via llvm-commits]

avl added inline comments.


================
Comment at: llvm/include/llvm/DWARFLinkerParallel/StringPool.h:25
 
-class PerThreadStringAllocator
-    : public AllocatorBase<PerThreadStringAllocator> {
+class SimpleStringAllocator : public AllocatorBase<SimpleStringAllocator> {
 public:
----------------
JDevlieghere wrote:
> Why not just `StringAllocator`. Do we expect to have a more "complex" allocator down the line?
I am planning to use allocator from D142318 later.

Will rename SimpleStringAllocator to StringAllocator as suggested.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147649