[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