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

[Aleksey Shipilev]

On Wed, 28 Jun 2023 05:53:31 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 incrementally with two additional commits since the last revision:
> 
>  - Aleksey's review comments
>  - Aleksey's review comments

Okay, this looks clean. Thanks!

Let's see if others have any more comments.

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

Marked as reviewed by shade (Reviewer).

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