RFR: 8287805: Shenandoah: consolidate evacuate-update-root closures [v6]

Aleksey Shipilev shade at openjdk.org
Mon Jul 18 13:10:05 UTC 2022


On Mon, 18 Jul 2022 13:05:01 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

>> ShenandoahEvacuateUpdateMetadataClosure and ShenandoahEvacuateUpdateRootsClosure are mostly same, can be consolidated.
>> 
>> Also, only uses of ShenandoahEvacuateUpdateMetadataClosure all pass MO_UNORDERED template parameter, so it can be eliminated.
>> 
>> Test:
>> 
>> - [x] hotspot_gc_shenandoah
>
> Zhengyu Gu has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 13 additional commits since the last revision:
> 
>  - Merge branch 'master' into JDK-8287805-evac-updt-closures
>  - Move get thread closer to the use
>  - Aleksey's comment
>  - Aleksey's comment
>  - Merge branch 'master' into JDK-8287805-evac-updt-closures
>  - v1
>  - Remove unused impl
>  - Review feedback
>  - Merge branch 'master' into JDK-8287805-evac-updt-closures
>  - 8287805: Shenandoah: consolidate evacuate-update-root closures
>  - ... and 3 more: https://git.openjdk.org/jdk/compare/3fc125de...8c12588d

Marked as reviewed by shade (Reviewer).

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

PR: https://git.openjdk.org/jdk/pull/9023



More information about the hotspot-gc-dev mailing list