[riscv-port] RFR: 8278994: riscv: RVC support [v14]
Fei Yang
fyang at openjdk.java.net
Wed Jan 12 06:21:45 UTC 2022
On Wed, 12 Jan 2022 02:58:59 GMT, Xiaolin Zheng <xlinzheng at openjdk.org> wrote:
> >
>
> I have tested this patch under whole `test/jdk` and `test/hotspot/jtreg` folders without Monday's change through last weekend on a Hifive Unleashed and only revealed [JDK-8279664](https://bugs.openjdk.java.net/browse/JDK-8279664), which has nothing to do with this patch, for the [RISC-V backend initial-load patch](https://github.com/openjdk/riscv-port/commit/7148c39832fe522bad91df77ef4b5c9371193c35) could also reveal this problem on the Unleashed board I'm currently using. This one is a bit weird and I need more time to dig deep into that.
>
> About Monday's incremental changesets I tested `test/hotspot/jtreg/compiler`, `test/hotspot/jtreg/gc/z`, and `test/hotspot/jtreg/gc/shenandoah` with or without RVC and revealed no errors. Also SPECjbb2015 is tested and the result seems to have no regression found under RVC - the result on Unleashed remains a `max-jOPs` level of score 370 with minor fluctuations.
>
> I think maybe this patch could safely move forward -- except I need to take a look at JDK-8279664 to find whether it is a board-related issue because I cannot reproduce it on C910 and qemu. And thanks for the scrupulous reviews and the delicious suggestions, Felix and Yadong. :-)
What about codesize improvement of the C2 JIT code? BTW: You need to rebase the patch on the latest repo.
-------------
PR: https://git.openjdk.java.net/riscv-port/pull/34
More information about the riscv-port-dev
mailing list