[PATCH] D142318: [Support] Add PerThreadBumpPtrAllocator class.
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 14 10:42:43 PDT 2023
dexonsmith added a comment.
In D142318#4269070 <https://reviews.llvm.org/D142318#4269070>, @dexonsmith wrote:
> In D142318#4268765 <https://reviews.llvm.org/D142318#4268765>, @MaskRay wrote:
>
>> In D142318#4268729 <https://reviews.llvm.org/D142318#4268729>, @avl wrote:
>>
>>> so far I suggest to implement safety check as a separate patch. After having a good solution for this.
>>
>> Looks good to me.
>
> I think if we don't add the check now it's unlikely to happen later.
(Although I do think an independent patch makes sense so it can be reverted independently if it causes trouble.)
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