[lworld] RFR: 8371357: [lworld] Remove EnableValhalla [v2]

Paul Hübner phubner at openjdk.org
Mon Dec 15 16:39:06 UTC 2025


On Mon, 15 Dec 2025 14:48:48 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Paul Hübner has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Remove wrong comment.
>>  - Fix imports.
>
> src/hotspot/share/opto/callnode.cpp line 1948:
> 
>> 1946:     Node* proto_adr = phase->transform(new AddPNode(klass_node, klass_node, phase->MakeConX(in_bytes(Klass::prototype_header_offset()))));
>> 1947:     mark_node = LoadNode::make(*phase, control, mem, proto_adr, TypeRawPtr::BOTTOM, TypeX_X, TypeX_X->basic_type(), MemNode::unordered);
>> 1948:     // EnavleValhalla legacy
> 
> Suggestion:

Good shout, I missed that thanks.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1759#discussion_r2620091976


More information about the valhalla-dev mailing list