RFR: 8322490: CastNode constructors accepts control node as input

Joshua Cao duke at openjdk.org
Tue Dec 19 20:31:55 UTC 2023


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

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

Commit messages:
 - 8322490: CastNode constructors accepts control node as input

Changes: https://git.openjdk.org/jdk/pull/17162/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17162&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8322490
  Stats: 65 lines in 6 files changed: 1 ins; 28 del; 36 mod
  Patch: https://git.openjdk.org/jdk/pull/17162.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17162/head:pull/17162

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


More information about the hotspot-compiler-dev mailing list