RFR: 8299915: Remove ArrayAllocatorMallocLimit and associated code [v2]

Afshin Zafari azafari at openjdk.org
Mon Sep 25 09:20:20 UTC 2023


> 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.

Afshin Zafari has updated the pull request incrementally with one additional commit since the last revision:

  other size_t flags than the ArrayAllocatorMallocLimit are used in tests.

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/15859/files
  - new: https://git.openjdk.org/jdk/pull/15859/files/3371127f..623d076f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15859&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15859&range=00-01

  Stats: 55 lines in 2 files changed: 55 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/15859.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15859/head:pull/15859

PR: https://git.openjdk.org/jdk/pull/15859


More information about the serviceability-dev mailing list