RFR: 8322490: CastNode constructors accepts control node as input [v3]
Paul Hohensee
phh at openjdk.org
Thu Dec 21 23:49:48 UTC 2023
On Thu, 21 Dec 2023 09:17:16 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
>
> Joshua Cao has updated the pull request incrementally with one additional commit since the last revision:
>
> Move comment
Marked as reviewed by phh (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/17162#pullrequestreview-1793792668
More information about the hotspot-compiler-dev
mailing list