[lworld+vector] RFR: 8304310: Initial compilers and runtime handling for multifield backed vectors. [v13]
Vladimir Ivanov
vlivanov at openjdk.org
Sat May 6 01:07:43 UTC 2023
On Sat, 6 May 2023 00:10:38 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Handling multifield access through Unsafe.get API, some more cleanups.
>
> src/hotspot/share/runtime/deoptimization.cpp line 1146:
>
>> 1144: if (EnableVectorSupport && VectorSupport::is_vector(ik)) {
>> 1145: obj = VectorSupport::allocate_vector(ik, fr, reg_map, sv, THREAD);
>> 1146: } else if (EnableVectorSupport && VectorSupport::is_vector_payload_mf(ik)) {
>
> Why do you need to handle both cases now? Anything special needed for `VectorSupport::is_vector()` case?
Another question: is there a lurking problem in the original implementation? Can a backing array leak into debug info, so reallocation has to handle it? I believe it's not the case since vector scalarization logic relies on typed vector box to pass type information, but want to double-check with you.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/833#discussion_r1186565966
More information about the valhalla-dev
mailing list