[lworld+fp16] RFR: 8339473: Add support for FP16 isFinite, isInfinite and isNaN [v3]

[Bhavana Kilambi]

On Thu, 19 Sep 2024 17:19:10 GMT, Joe Darcy <darcy at openjdk.org> wrote:

>> Bhavana Kilambi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Change java side implementation instead of adding intrinsics
>
> src/java.base/share/classes/java/lang/Float16.java line 446:
> 
>> 444:      * @see Double#isNaN(double)
>> 445:      */
>> 446:     public static boolean isNaN(Float16 f16) {
> 
> With changing the implementation if isNaN, isFinite, etc. to bit-based, the regression tests should be updated to cover more cases. In particular, different NaN bit patterns should be tested.

Hi @jddarcy , thank you for the review. I will add more input patterns to be tested.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1239#discussion_r1768253608