On Thu, 21 Sep 2023 12:02:24 GMT, Afshin Zafari <azaf...@openjdk.org> wrote:
> 1. `ArrayAllocatorMallocLimit` is removed. The test cases that tested it also
> are removed.
> 2. `AllocArrayAllocator` instances are replaced with `MallocArrayAllocator`.
> 3. The signature of `CHeapBitMap::free(ptr, size)` is kept as it is, since it
> is called in this way from `GrowableBitMap<T>::resize`, where `T` can be also
> `ArenaBitMap` and `ResourceBitMap`. However, it uses
> `MallocArrayAllocator::free(ptr)` and ignores the `size`:
> ```C++
> void CHeapBitMap::free(bm_word_t* map, idx_t size_in_words) const {
> MallocArrayAllocator<bm_word_t>::free(map);
> }
>
> ### Test
> tiers1-4 passed on all platforms.
This pull request has now been integrated.
Changeset: 45a145e5
Author: Afshin Zafari <azaf...@openjdk.org>
URL:
https://git.openjdk.org/jdk/commit/45a145e5bc3d3216bb03379896f66a3b719a06dc
Stats: 213 lines in 8 files changed: 2 ins; 202 del; 9 mod
8299915: Remove ArrayAllocatorMallocLimit and associated code
Reviewed-by: dholmes, coleenp
-------------
PR: https://git.openjdk.org/jdk/pull/15859
