On Mon, 3 Aug 2026 05:52:53 GMT, Jatin Bhateja <[email protected]> wrote:

>> Patch optimizes Float16 to integral conversion operations. Currently, its a 
>> two step process where by first a Float16 value is
>> converted to a single precision floating point value followed by a 
>> conversion to an integral value.
>> 
>> x86 targets supporting AVX512-FP16 feature (Intel Sapphire Rapids+ and 
>> upcoming AMD Zen6) provides direct instruction to convert a Float16 value to 
>> integral value.
>> 
>> Following are the performance numbers of micro benchmark included with the 
>> patch on Granite Rapids with and without auto-vectorization.
>> 
>> <img width="1125" height="636" alt="image" 
>> src="https://github.com/user-attachments/assets/ca6e6757-1579-475f-8307-9454c7c025c1";
>>  />
>> 
>> Kindly review and share your feedback.
>> 
>> Best Regards,
>> Jatin
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Jatin Bhateja has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains seven additional 
> commits since the last revision:
> 
>  - Merge branch 'master' of http://github.com/openjdk/jdk into JDK-8382523
>  - Review comments resolution
>  - Review comments resolution
>  - Review comments resolution
>  - Review comments resolution
>  - Review comments resolution
>  - 8382523: Optimize Float16 to integral conversion operations for 
> AVX512-FP16 targets

> Change must be properly reviewed (2 reviews required, with at least 1 
> [Reviewer](https://openjdk.org/bylaws#reviewer), 1 
> [Author](https://openjdk.org/bylaws#author))

^ @jatin-bhateja

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

PR Comment: https://git.openjdk.org/jdk/pull/30928#issuecomment-5179048941

Reply via email to