RFR: 8241153: Refactor HeapRegionManager::find_unavailable_from_idx to simplify expand_at

Thomas Schatzl thomas.schatzl at oracle.com
Thu Apr 23 13:59:00 UTC 2020


Hi,

On 21.04.20 10:49, Stefan Johansson wrote:
> Hi,
> 
> Please review this enhancement to simplify the expand code in G1.
> 
> Webrev: http://cr.openjdk.java.net/~sjohanss/8241153/00/
> JBS: https://bugs.openjdk.java.net/browse/JDK-8241153
> 
> Summary:
> Use the bitmap search functions to find candidates in 
> find_unavailable_from_idx and return a range to expand.
> 
> Testing:
> Mach5 tier1-4

  - the parameterless constructor for HeapRegionRange can be deleted, 
it's unused.

No need for a re-review for this change.

Looks good.

Thomas



More information about the hotspot-gc-dev mailing list