[lworld+vector] RFR: 8304310: Initial compilers and runtime handling for multifield backed vectors. [v14]
Jatin Bhateja
jbhateja at openjdk.org
Wed May 10 10:32:59 UTC 2023
On Wed, 10 May 2023 10:05:24 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:
>> Just to give meaningful name to wrapper, both are anyways annotated by @ForceInline hence will always get inlined in caller context.
>
> The reason that I asked this question is: when I was modifying the VectorMask related code, I will use `VectorPayloadMF.createVectPayloadInstance(...)` when I need to create a payload for mask. Then I was thinking maybe `VectorPayloadMF.newInstanceFactory()` is better with two reasons:
>
> 1. We have already contained "VectorPayloadMF" in the class name.
> 2. The later is shorter.
No problem, please modify as you feel appropriate during your implementation.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/833#discussion_r1189705705
More information about the valhalla-dev
mailing list