RFR: 8376187: [VectorAPI] Define new lane type constants and pass them to intrinsic entries [v7]

Quan Anh Mai qamai at openjdk.org
Wed Feb 4 18:16:17 UTC 2026


On Tue, 3 Feb 2026 03:31:52 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> As per [discussions ](https://github.com/openjdk/jdk/pull/28002#issuecomment-3789507594) on JDK-8370691 pull request, splitting out portion of PR#28002 into a separate patch in preparation of Float16 vector API support.
>> 
>> Patch add new lane type constants and pass them to vector intrinsic entry points.
>> 
>> All existing Vector API jtreg test are passing with the patch.
>> 
>> Kindly review and share your feedback.
>> 
>> Best Regards,
>> Jatin
>
> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments resolution

src/hotspot/share/prims/vectorSupport.hpp line 141:

> 139: 
> 140:   // Values in this enum correspond to the jdk.incubator.vector.LaneType ordinal values.
> 141:   enum LaneType {

Please specify the underlying type of this enum: `enum LaneType : int`. Otherwise, this will not work:

    VectorSupport::LaneType vltype = static_cast<VectorSupport::LaneType>(laneType->get_con());

It is because casting an integer to an unscoped enum is UB if `laneType->get_con()` does not fit into the range of the enum values.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29481#discussion_r2765349355


More information about the core-libs-dev mailing list