[lworld+vector] RFR: 8311610: [lworld+vector] Clean-up of vector allocation in class VectorSupport [v3]
Xiaohong Gong
xgong at openjdk.org
Wed Jul 12 02:53:20 UTC 2023
On Wed, 12 Jul 2023 01:35:52 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:
>> Please also relax the condition on line # 1605 to just check for Location::vector
>
> Yes, relaxing the condition is fine. And we'd better to add the assertion that `secondary_fields_count > 1` ?
Or, is the vector supported for 1 lanecount (e.g. 64-bit shape for `long/double` type) on x86 systems?
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/882#discussion_r1260503207
More information about the valhalla-dev
mailing list