RFR (XXS): 8252086: G1: Remove g1_rs in G1CollectedHeap::initialize
stefan.johansson at oracle.com
stefan.johansson at oracle.com
Thu Aug 20 10:12:51 UTC 2020
Hi Thomas,
On 2020-08-20 12:06, Thomas Schatzl wrote:
> Hi,
>
> can I have reviews for this small change that removes an intermediate
> local variable from G1CollectedHeap::initialize?
>
> This makes it easier to follow that CollectedHeap::_reserved is the same
> as HeapRegionManager::reserved().
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8252086
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8252086/webrev/
Looks great, thanks for cleaning up the code.
Cheers,
Stefan
> Testing:
> local compilation, local jtreg gc/g1 directory
>
> Thanks,
> Thomas
More information about the hotspot-gc-dev
mailing list