[lworld+vector] RFR: 8304310: Initial compilers and runtime handling for multifield backed vectors. [v14]

Jatin Bhateja jbhateja at openjdk.org
Wed May 10 09:20:47 UTC 2023


On Wed, 10 May 2023 02:08:46 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review comments resolutions.
>
> src/hotspot/share/opto/cfgnode.cpp line 2704:
> 
>> 2702:   ciInlineKlass* payload = vk->declared_nonstatic_field_at(0)->type()->as_inline_klass();
>> 2703: 
>> 2704:   Node* payload_oop = payload->is_initialized() ? InlineTypeNode::default_oop(*igvn, payload) : igvn->zerocon(T_PRIMITIVE_OBJECT);
> 
> `payload_oop` is not used and can be removed?

This is already handled within  _`make_uninitialized`_

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/833#discussion_r1189610381



More information about the valhalla-dev mailing list