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

[William Kemper]

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

>> src/hotspot/share/gc/shenandoah/shenandoahConcurrentGC.cpp line 196:
>> 
>>> 194: 
>>> 195:     // Evacuation is complete, retire gc labs
>>> 196:     heap->concurrent_prepare_for_update_refs();
>> 
>> For consistency with other related method naming, can we use "updaterefs" instead of "update_refs" (makes IDE searches easier to locate related methods).
>
> I'm all for making this consistent, but it seems that `update_refs` is more commonly used in method and variable declarations:
> 
> [0] % grep -r --include "*.hpp" updaterefs src/hotspot/share/gc/shenandoah | wc -l 
> 17
> 
> [0] % grep -r --include "*.hpp" update_refs src/hotspot/share/gc/shenandoah | wc -l
> 27

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.

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

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