RFR: 8344049: Shenandoah: Eliminate init-update-refs safepoint [v3]

Y. Srinivas Ramakrishna ysr at openjdk.org
Tue Jan 14 20:29:29 UTC 2025


On Mon, 13 Jan 2025 18:53:20 GMT, William Kemper <wkemper at openjdk.org> wrote:

>> If it's okay with you, I will do this on a separate PR so that this current PR is not cluttered by the change.
>
> https://bugs.openjdk.org/browse/JDK-8347617

Yes, that's OK. I didn't realize there were so many `update_refs` and `updaterefs` each... I had only looked at the `[[[vm_]op_]init_updaterefs}` class of names (including the ones right below here) and incorrectly assumed there was consistency in the naming, but for this new variant that you had introduced in this PR.

Thanks for filing a separate ticket for rename, and it makes sense to do that kind of wholesale renaming later. (The suggestion here had been just for this name, but I see that it isn't inconsistent with the lack of a single convention, so I take back my comment.)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22688#discussion_r1915221534


More information about the hotspot-gc-dev mailing list