RFR: 8287555: Tighten G1 G1BlockOffsetTable::block_start() code [v7]

Thomas Schatzl tschatzl at openjdk.org
Fri Jul 8 08:25:27 UTC 2022


> Hi all,
> 
>   can I have reviews for this change that specializes `G1BlockOffsetTablePart::block_size` with a variant used for card aligned addresses including some cleanup of comments? This saves some unnecessary lookup of the block size of the object found as for refinement the start address is always aligned.
> 
> Testing: tier1, gha
> 
> Thanks,
>   Thomas

Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:

  Refactoring

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/9059/files
  - new: https://git.openjdk.org/jdk/pull/9059/files/ecda6c96..c848f00c

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9059&range=06
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9059&range=05-06

  Stats: 48 lines in 2 files changed: 17 ins; 18 del; 13 mod
  Patch: https://git.openjdk.org/jdk/pull/9059.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9059/head:pull/9059

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



More information about the hotspot-gc-dev mailing list