On Mon, 6 Apr 2026 07:55:10 GMT, Jatin Bhateja <[email protected]> wrote:

>> Add a new  Float16lVector type and corresponding concrete vector classes, in 
>> addition to existing primitive vector types, maintaining operation parity 
>> with the FloatVector type.
>> - Add necessary inline expander support.
>>    - Enable intrinsification for a few vector operations, namely 
>> ADD/SUB/MUL/DIV/MAX/MIN/FMA.
>> - Use existing Float16 vector IR and backend support.
>> - Extended the existing VectorAPI JTREG test suite for the newly added 
>> Float16Vector operations.
>>  
>> The idea here is to first be at par with Float16 auto-vectorization support 
>> before intrinsifying new operations (conversions, reduction, etc).
>> 
>> The following are the performance numbers for some of the selected 
>> Float16Vector benchmarking kernels compared to equivalent auto-vectorized 
>> Float16OperationsBenchmark kernels.
>> 
>> <img width="1344" height="532" alt="image" 
>> src="https://github.com/user-attachments/assets/c8157c3c-22b0-4bc1-9de9-7a68cadb7b2a";
>>  />
>> 
>> Initial RFP[1] was floated on the panama-dev mailing list.
>> 
>> Kindly review the draft PR and share your feedback.
>> 
>> Best Regards,
>> Jatin
>> 
>> [1] https://mail.openjdk.org/pipermail/panama-dev/2025-August/021100.html
>
> Jatin Bhateja has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains 49 commits:
> 
>  - Post merge fixes
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8370691
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8370691
>  - Fix float16 handling for toIntegralChecked routine, some other cleanups
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8370691
>  - More cleanups
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8370691
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8370691
>  - More cleanups
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8370691
>  - ... and 39 more: https://git.openjdk.org/jdk/compare/1d290109...c95e3c6e

src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Float16Vector.java 
line 52:

> 50: /**
> 51:  * A specialized {@link Vector} representing an ordered immutable 
> sequence of
> 52:  * {@code short} values.

I’m not sure whether this has been discussed and decided before, but should we 
use “sequence of Float16 values” instead of “short”?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28002#discussion_r3061332861

Reply via email to