RFR: 8341260: Add Float16 to jdk.incubator.vector [v6]

[Joe Darcy]

On Wed, 23 Oct 2024 01:03:38 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add equals/hashCode implementation; tests to follow.
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Float16.java line 865:
> 
>> 863:     public static Float16 min(Float16 a, Float16 b) {
>> 864:         return shortBitsToFloat16(floatToFloat16(Math.min(a.floatValue(),
>> 865:                                                           b.floatValue()) ));
> 
> I assume we will optimize these min/max implementations in the future. Otherwise, the extra space should be removed.

The general intention is that the back-end intrinsification efforts that use native hardware instructions, where present, will provide the main speed boost. Therefore, the preference has been to keep the Java sources in Float16 as straightforward and "obviously correct" as possible.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21574#discussion_r1813074826