Integrated: 8300769: Remove G1CollectionSet::_inc_bytes_used_before

Thomas Schatzl tschatzl at openjdk.org
Tue Jan 24 17:38:33 UTC 2023


On Fri, 20 Jan 2023 13:02:10 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this removal of `G1CollectionSet::_inc_bytes_used_before`; this variable holds the amount of bytes used in eden regions, updated during mutator time. At the start of gc its value is copied to `G1CollectionSet::_bytes_used_before`, and then never touched again. The `_bytes_used_before` is read at end of gc, and then reset.
> 
> This change simply removes `_inc_bytes_used_before` and directly uses `_bytes_used_before` to accumulate eden region used bytes.
> 
> Testing: gha
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: 3be5758b
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/3be5758bb413fb6b4dc6191d78ca38332d5153f1
Stats:     57 lines in 5 files changed: 2 ins; 52 del; 3 mod

8300769: Remove G1CollectionSet::_inc_bytes_used_before

Reviewed-by: ayang, iwalulya

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

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


More information about the hotspot-gc-dev mailing list