[PATCH] D142318: [Support] Add PerThreadBumpPtrAllocator class.

[Alexey Lapshin via Phabricator via llvm-commits]

avl added a comment.

In D142318#4269662 <https://reviews.llvm.org/D142318#4269662>, @dexonsmith wrote:

> In D142318#4269644 <https://reviews.llvm.org/D142318#4269644>, @avl wrote:
>
>> I am OK to do that separate patch right after the current patch. Just do not have a good idea for this at the moment.
>
> WDYT of the idea above, to have a Boolean flag that checks whether `getDefaultExecutor()` has been called, and assert on that in `getThreadIndex()`?

I think this solves only part of the problem: it checks the fact that executor is already created when getThreadIndex() is requested. But it does not check that thread index is valid. If thread was created not by ThreadPoolExecutor then it would have zero index which clashes with thread index of main thread and Thread0. I thought we want to check that other threads were not used with getThreadIndex.

Checking ThreadPoolExecutor existence still useful check and it would be good to implement it. If we found a good way to check thread indexes it would also be useful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142318