[lworld] RFR: 8335256: [lworld] C2: Remove larval InlineTypeNode [v5]
Tobias Hartmann
thartmann at openjdk.org
Mon May 12 11:58:06 UTC 2025
On Fri, 9 May 2025 17:56:35 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:
>> src/hotspot/share/opto/inlinetypenode.cpp line 1100:
>>
>>> 1098: Node* base = is_loaded(phase);
>>> 1099: if (base != nullptr && !base->is_InlineType() && !phase->type(base)->maybe_null() && AllocateNode::Ideal_allocation(base) == nullptr) {
>>> 1100: if (oop != base || phase->type(is_buffered) != TypeInt::ONE) {
>>
>> Maybe use `InlineTypeNode::is_allocated` here instead?
>
> `InlineTypeNode::is_allocated` tries to see if an `InlineTypeNode` is buffered, while this check tries to see if the node has `is_buffered` being `true` and `oop` being exactly `base`, not a random value. So it cannot be used here.
Right, thanks for the clarification.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1447#discussion_r2084510165
More information about the valhalla-dev
mailing list