[lworld+fp16] RFR: 8330021: AArch64: Add backend support for FP16 add operation [v4]

Bhavana Kilambi bkilambi at openjdk.org
Wed Jul 10 08:22:15 UTC 2024


On Fri, 5 Jul 2024 13:28:02 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> Bhavana Kilambi has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>> 
>>  - Merge branch 'lworld+fp16'
>>  - Remove couple of IR testcases + add --enable-preview flag to tests
>>  - Merge branch 'lworld+fp16'
>>  - Add a missing return statement in the ad file
>>  - 8330021: AArch64: Add backend support for FP16 add operation
>>    
>>    This commit [1] adds initial support for FP16 operations and adds
>>    backend support for FP16 add operation for X86. This task adds backend
>>    support for scalar and vector FP16 add operation on aarch64.
>>    
>>    [1] https://github.com/openjdk/valhalla/commit/f03fb4e4ee4d59ed692d0c26ddce260511f544e7#diff-a799ce8da7f3062bb3699beb65aae504840c649942032e325c2a50f88a2869ad
>
> Non target specific part looks ok to me.

Hi @jatin-bhateja , I have uploaded a new patch addressing your comments. Can you please review? Thanks!

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

PR Comment: https://git.openjdk.org/valhalla/pull/1096#issuecomment-2219854037


More information about the valhalla-dev mailing list