RFR: 8300769: Remove G1CollectionSet::_inc_bytes_used_before [v2]
Thomas Schatzl
tschatzl at openjdk.org
Mon Jan 23 13:45:15 UTC 2023
> 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
Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
ayang review, remove redundant _bytes_used_before
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/12113/files
- new: https://git.openjdk.org/jdk/pull/12113/files/ec60055c..b15485bb
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=12113&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=12113&range=00-01
Stats: 33 lines in 4 files changed: 1 ins; 32 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/12113.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/12113/head:pull/12113
PR: https://git.openjdk.org/jdk/pull/12113
More information about the hotspot-gc-dev
mailing list