[lworld+fp16] RFR: 8338061: Add support for FP16 unary and ternary operations [v2]

Bhavana Kilambi bkilambi at openjdk.org
Tue Aug 27 15:44:03 UTC 2024


On Tue, 27 Aug 2024 15:40:12 GMT, Bhavana Kilambi <bkilambi at openjdk.org> wrote:

>> This patch adds support for three unary operators - abs, neg and sqrt and a ternary operator - fma for FP16.
>> Both scalar and vector mid-end support along with aarch64 backend support are added.
>> Tested all FP16 tests successfully on aarch64.
>
> Bhavana Kilambi 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 three additional commits since the last revision:
> 
>  - Address review comments
>  - Merge branch 'lworld+fp16'
>  - 8338061: Add support for FP16 unary and ternary operations
>    
>    This patch adds support for three unary operators - abs, neg and sqrt
>    and a ternary operator - fma for FP16.
>    Both scalar and vector mid-end support along with aarch64 backend
>    support are added.
>    Tested all FP16 tests successfully on aarch64.

test/jdk/java/foreign/enablenativeaccess/org/openjdk/foreigntest/libLinkerInvokerUnnamed.cpp line 25:

> 23: 
> 24: #include "jni.h"
> 25: #include "testlib_threads.hpp"

I think this change was missed in the merge commit. I added it.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1211#discussion_r1733115571


More information about the valhalla-dev mailing list