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