RFR: 8368985: Small Float16 refactorings [v2]
Raffaello Giulietti
rgiulietti at openjdk.org
Wed Oct 1 16:59:18 UTC 2025
On Wed, 1 Oct 2025 16:06:45 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>> A few more small cleanups to Float16 and its tests.
>
> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
>
> Respond to review feedback.
src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Float16.java line 756:
> 754: public static boolean isInfinite(Float16 f16) {
> 755: return ((float16ToRawShortBits(f16) ^ float16ToRawShortBits(POSITIVE_INFINITY)) &
> 756: (EXP_BIT_MASK | SIGNIF_BIT_MASK)) == 0;
IMHO, `isNan()` and `isInfinite()` should look almost identical, except for the significand bits.
Maybe for another time.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27587#discussion_r2395223913
More information about the core-libs-dev
mailing list