> Hi, please consider.
> 
> This provides the necessary JVM changes (interpreter, c1 and c2) for a 
> specification-compliant Valhalla port on linux-riscv64 platform. Major change 
> is made in the riscv CPU part. We need a very small change to hotspot c1 
> shared code. The reason is that there doesn't exist a real flag register for 
> the riscv64 ISA. So we have to pass the result in a temp register. That's a 
> major difference from other ISAs like aarch64 & amd64.
> 
> Calling convention flattening feature has not been implemented for this 
> platform. If either `-InlineTypePassFieldsAsArgs` or 
> `-XX:-InlineTypeReturnedAsFields` is specified on the command line, JVM will 
> issue a warning and continue with these two options disabled. This helps 
> enable most of Valhalla tests on this platform so that we could have a better 
> test coverage.
> 
> 
> 
> Testing on linux-riscv64:
> - [x] GHA
> - [x] `make test TEST="hotspot_valhalla jdk_valhalla valhalla_adopted"` 
> (release & fastdebug)
> - [x] `make test TEST="tier1 tier2 tier3 tier4"` (release)
> - [x] `make test TEST="hotspot:tier1 hotspot:tier2 hotspot:tier3 
> hotspot:tier4"` (fastdebug)
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Fei Yang has updated the pull request incrementally with one additional commit 
since the last revision:

  More register improvement

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2531/files
  - new: https://git.openjdk.org/valhalla/pull/2531/files/48fe0f02..e4fa8baf

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2531&range=02
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2531&range=01-02

  Stats: 3 lines in 2 files changed: 1 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/valhalla/pull/2531.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2531/head:pull/2531

PR: https://git.openjdk.org/valhalla/pull/2531

Reply via email to