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