RFR: 8321010: RISC-V: C2 RoundVF [v7]

T Busch duke at openjdk.org
Fri Jul 5 14:02:29 UTC 2024


On Tue, 11 Jun 2024 21:00:39 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Hi,
>> Can you have a review on this patch to add RoundVF/RoundDF intrinsics?
>> Thanks!
>> 
>> ## Tests
>> 
>> test/hotspot/jtreg/compiler/vectorization/TestRoundVectRiscv64.java test/hotspot/jtreg/compiler/c2/cr6340864/TestFloatVect.java test/hotspot/jtreg/compiler/c2/cr6340864/TestDoubleVect.java test/hotspot/jtreg/compiler/floatingpoint/TestRound.java
>> 
>> test/jdk/java/lang/Math/RoundTests.java
>
> Hamlin Li has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 15 commits:
> 
>  - merge master
>  - Merge branch 'master' into round-F+D-v
>  - Merge branch 'master' into round-F+D-v
>  - restore round mode back to rne
>  - Merge branch 'master' into round-F+D-v
>  - fix minors
>  - merge master
>  - fix space
>  - add tests
>  - add test cases
>  - ... and 5 more: https://git.openjdk.org/jdk/compare/83b34410...a9faee89

src/hotspot/cpu/riscv/c2_MacroAssembler_riscv.cpp line 1971:

> 1969: //    float >= Integer.MAX_VALUE,
> 1970: //    float <= Integer.MIN_VALUE.
> 1971: void C2_MacroAssembler::java_round_float_v(VectorRegister dst, VectorRegister src, FloatRegister ftmp) {

Void

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17745#discussion_r1585865241


More information about the hotspot-compiler-dev mailing list