[PATCH] D142318: [Support] Add PerThreadBumpPtrAllocator class.
Alexey Lapshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 18 04:58:46 PDT 2023
avl added a comment.
In D142318#4277110 <https://reviews.llvm.org/D142318#4277110>, @andrewng wrote:
>> 2. remove optimization for NumItems to avoid executing on the main thread.
>
> Sorry but I'm quite busy right now, so haven't been able to give this much attention. I did notice this whilst scanning the updates to this review and IIRC this was quite an important optimisation for some use case (can't remember the details). So might want to take some care before removing it.
yep. I am going to create an appropriate set of reviews, so that it would be possible to pay attention to each aspect separately.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142318/new/
https://reviews.llvm.org/D142318
More information about the llvm-commits
mailing list