RFR: 8370807: G1: Improve region attribute table method naming

Albert Mingkun Yang ayang at openjdk.org
Wed Oct 29 11:52:22 UTC 2025


On Wed, 29 Oct 2025 11:40:26 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> So the given region is already(*) in the collection set

If that's indeed precondition, the new name makes sense.

There is `assert(!hr->in_collection_set(),` in the caller, which is conflicting with this semantic.

> If you really want I can do that in this change too.

I'd prefer doing that (new name + new precondition) in another PR, as the current title imply naming changes only, but up to you.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28038#discussion_r2472697834


More information about the hotspot-gc-dev mailing list