Integrated: 8267875: Shenandoah: Duplicated code in ShenandoahBarrierSetC2::ideal_node()

Zhengyu Gu zgu at openjdk.java.net
Mon Jun 7 14:03:25 UTC 2021


On Thu, 27 May 2021 18:29:40 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

> Node::set_req_X definition as:
> void Node::set_req_X(uint i, Node *n, PhaseGVN *gvn) {
>   PhaseIterGVN* igvn = gvn->is_IterGVN();
>   if (igvn == NULL) {
>     set_req(i, n);
>     return;
>   }
>   set_req_X(i, n, igvn);
> }
> 
> ShenandoahBarrierSetC2::ideal_node() duplicates above logic.
> 
> 
> Test:
>   hotspot_gc_shenandoah

This pull request has now been integrated.

Changeset: ea8274fb
Author:    Zhengyu Gu <zgu at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/ea8274fbf07d7308bcef5aacbe553dc7b37887e3
Stats:     11 lines in 1 file changed: 0 ins; 9 del; 2 mod

8267875: Shenandoah: Duplicated code in ShenandoahBarrierSetC2::ideal_node()

Reviewed-by: rkennke, roland

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

PR: https://git.openjdk.java.net/jdk/pull/4229


More information about the hotspot-gc-dev mailing list