[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8299915: Remove ArrayAllocatorMallocLimit and associated code
From:       Afshin Zafari <azafari () openjdk ! org>
Date:       2023-09-27 8:30:23
Message-ID: o_F6Y7uzbmdjFtPue1nPa7wBEuEmcWw38TVKMlIltKI=.06b50dc2-e2b0-40be-9379-353b4b7e9976 () github ! com
[Download RAW message or body]

On Thu, 21 Sep 2023 12:02:24 GMT, Afshin Zafari <azafari@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 <azafari@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


[prev in list] [next in list] [prev in thread] [next in thread]