Integrated: JDK-8318228: RISC-V: C2 ConvF2HF

[Hamlin Li]

On Tue, 16 Jan 2024 18:43:03 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you review the patch to add ConvF2HF intrinsic to JDK for riscv？
> Thanks!
> 
> ## Test
> ### Functionality
> #### hotspot tests
> test/hotspot/jtreg/compiler/intrinsics/ 
> test/hotspot/jtreg/compiler/c2/irTests
> 
> #### jdk tests
> test/jdk/java/lang/Float/Binary16Conversion*.java
> 
> ### Performance
> tested on licheepi.
> 
> #### with UseZfh enabled
> 
> Benchmark                                     (size)  Mode  Cnt     Score   Error  Units
> Fp16ConversionBenchmark.floatToFloat16          2048  avgt    2  4170.549          ns/op
> Fp16ConversionBenchmark.floatToFloat16Memory    2048  avgt    2    21.492          ns/op
> 
> 
> #### with UseZfh disabled
> (i.e. disable the intrinsic)
> 
> Benchmark                                     (size)  Mode  Cnt      Score   Error  Units
> Fp16ConversionBenchmark.floatToFloat16          2048  avgt    2  25036.647          ns/op
> Fp16ConversionBenchmark.floatToFloat16Memory    2048  avgt    2     27.326          ns/op

This pull request has now been integrated.

Changeset: bcaad515
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/bcaad515fdedd0c41a719d2a88b2da3036c766a3
Stats:     57 lines in 3 files changed: 57 ins; 0 del; 0 mod

8318228: RISC-V: C2 ConvF2HF

Reviewed-by: fyang, vkempik

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

PR: https://git.openjdk.org/jdk/pull/17450