RFR: 8318647: Serial: Refactor BlockOffsetTable [v2]

Wed Oct 25 12:08:29 UTC 2023

On Wed, 25 Oct 2023 11:15:42 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Note that region isn't defined very exactly. Here it is (apparently) the memory from `blk_start` to `blk_end`, but later it is also used as "area containing the same backskip value". Maybe there is some better wording to be found.
>
> "region" in this file always means "logarithmic region". If we refer to `[blk_start, blk_end]`, "block" should be used.

I noticed just now. It would be nice to define that somewhere. 👍

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16304#discussion_r1371630310