RFR: 8341260: Add Float16 to jdk.incubator.vector [v16]
Raffaello Giulietti
rgiulietti at openjdk.org
Wed Nov 13 11:18:31 UTC 2024
On Wed, 13 Nov 2024 10:41:34 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:
>> Joe Darcy has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Implement code review feedback.
>> - Misc cleanups.
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Float16.java line 150:
>
>> 148: * A constant holding the largest positive finite value of type
>> 149: * {@code Float16},
>> 150: * (2-2<sup>-10</sup>)·2<sup>15</sup>, numerically equal to 65504.0.
>
> I'm curious why `MAX_VALUE` includes the "numerically equal to", while `MIN_NORMAL` and `MIN_VALUE` do not?
>
> I see java.lang.Float does not use "numerically equal" in the corresponding constants. Float instead says "It is equal to the hexadecimal floating-point literal.."
Not sure if this is the reason, but in decimal notation `MIN_NORMAL` and `MIN_VALUE` are relatively unreadable: `0.00006103515625` and `5.9604644775390625E-8`, resp.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21574#discussion_r1840063297
More information about the core-libs-dev
mailing list