> 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:

  Simplify check in TemplateTable::if_acmp

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2531/files
  - new: https://git.openjdk.org/valhalla/pull/2531/files/7aea74ee..d880d4a2

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

  Stats: 6 lines in 1 file changed: 0 ins; 0 del; 6 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