[lworld+vector] RFR: 8304310: Initial compilers and runtime handling for multifield backed vectors. [v14]
Xiaohong Gong
xgong at openjdk.org
Wed May 10 02:28:48 UTC 2023
On Mon, 20 Mar 2023 07:21:00 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/vectornode.cpp line 861:
>
>> 859: return n->in(1)->bottom_type() == TypeF::ONE;
>> 860: case Op_ReplicateD:
>> 861: return n->in(1)->bottom_type() == TypeD::ONE;
>
> `is_all_ones_vector` here means all bits are `1` (i.e. `-1` for integer types). So `TypeF::ONE` is not the right value here?
Missed this comment?
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/833#discussion_r1189291070
More information about the valhalla-dev
mailing list