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