[jdk17u-dev] RFR: 8276799: Implementation of JEP 422: Linux/RISC-V Port [v9]

Ludovic Henry luhenry at openjdk.org
Fri Jul 7 18:04:12 UTC 2023


On Fri, 7 Jul 2023 14:00:40 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> The RISC-V port was originally developed at Huawei Technologies, then integrated into OpenJDK 19.
>> 
>> The 17u version of the port has continued to be maintained in the openjdk/riscv-port-jdk17u repo later
>> and has been tested for several months. There are few changes to shared HotSpot code (mostly the main
>> one is C1 conditional move/branch support for RISC-V). As required by 17u maintainer, changes to shared
>> code has been kept to a minimum. Only enabling shared changes are incorporated and these changes are
>> properly guarded with macro RISCV. So this 17u port should not breaking existing code and, although it is
>> a large patch, finally integrating it into 17u upstream should be low risk.
>> 
>> Testing on linux-riscv64 platform:
>> - [x] Bootcycle (release & fastdebug build)
>> - [x] Tier1-4 tests (release build)
>> - [x] Benchmark workloads (Dacapo, SPECJbb2015, SPECJVM2008, Renaissance. release build)
>
> Fei Yang 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 15 additional commits since the last revision:
> 
>  - Merge branch 'master' into jep422-17u-backport
>  - 8277417: C1 LIR instruction for load-klass
>  - Merge remote-tracking branch 'upstream/master' into jep422-17u-backport
>  - Merge branch 'master' into jep422-17u-backport
>  - Aleksey's review comments
>  - Aleksey's review comments
>  - Merge branch 'master' into jep422-17u-backport
>  - 8285630: Fix a configure error in RISC-V cross build
>  - Adds back commit ef86ea2842b1a204834291d9d6665bfcd7b75fbc
>  - Merge branch 'master' into jep422-17u-backport
>  - ... and 5 more: https://git.openjdk.org/jdk17u-dev/compare/19327f2e...7d2a51d5

Marked as reviewed by luhenry (no project role).

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

PR Review: https://git.openjdk.org/jdk17u-dev/pull/1427#pullrequestreview-1519412409


More information about the jdk-updates-dev mailing list