RFR: 8358483: G1: Remove G1HeapRegionManager::num_available_regions
Sangheon Kim
sangheki at openjdk.org
Wed Jun 4 23:23:50 UTC 2025
On Wed, 4 Jun 2025 13:44:25 GMT, Ivan Walulya <iwalulya at openjdk.org> wrote:
>> Simple cleanup of moving an API from `G1HeapRegionManager` to `G1CollectedHeap`.
>>
>> Test: tier1
>
> I would advocate for moving more APIs into `G1HeapRegionManager` instead of adding more to `G1CollectedHeap`.
I agree with @walulyai .
Adding convenient APIs on `G1CollectedHeap` is a matter of preference. But I dislike removing a basic API from `G1HeapRegionManager`. Maybe we disagree whether `num_available_regions() ` is a basic API or not?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25611#issuecomment-2941904091
More information about the hotspot-gc-dev
mailing list