Integrated: 8322490: cleanup CastNode construction

Joshua Cao duke at openjdk.org
Fri Dec 22 21:11:54 UTC 2023


On Tue, 19 Dec 2023 20:27:06 GMT, Joshua Cao <duke at openjdk.org> wrote:

> It is a common pattern to have:
> 
> 
> Node* n = new CastNode(...);
> n->set_req(control_node);
> 
> 
> We can modify the constructor to set the control node. It makes the code a little tidier.
> 
> Passes tier1 locally on my Linux machine

This pull request has now been integrated.

Changeset: 7263e25d
Author:    Joshua Cao <joshcao at amazon.com>
Committer: Paul Hohensee <phh at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7263e25d9b69d67697992a284c75454c479b6ec3
Stats:     120 lines in 10 files changed: 8 ins; 64 del; 48 mod

8322490: cleanup CastNode construction

Reviewed-by: chagedorn, phh

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

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


More information about the hotspot-compiler-dev mailing list