[riscv-port-jdk17u:riscv-port] RFR: 8297715: RISC-V: C2: Use single-bit instructions from the Zbs extension
Fei Yang
fyang at openjdk.org
Mon Mar 27 02:09:00 UTC 2023
On Thu, 23 Mar 2023 16:17:36 GMT, Dingli Zhang <dzhang at openjdk.org> wrote:
> Please review this backport to riscv-port-jdk17u.
> Backport of [JDK-8297715](https://bugs.openjdk.org/browse/JDK-8297715).
> The original patch cannot be directly applied because of the line number, but there are no other additional changes.
>
> The effect is that we could then optimize C2 JIT code for methods like (print with case `xml.validation` in SPECjvm2008):
> Before:
>
> 03c lhu R28, [R11, #12] # short, #@loadUS ! Field: com/sun/org/apache/xerces/internal/dom/NodeImpl.flags
> 040 + andi R7, R28, #2 #@andI_reg_imm
> 044 + snez R10, R7 #@convI2Bool
>
> After:
>
> 03c lhu R28, [R11, #12] # short, #@loadUS ! Field: com/sun/org/apache/xerces/internal/dom/NodeImpl.flags
> 040 + bexti R10, R28, #2 #@convI2Bool_andI_reg_immIpowerOf2
>
>
> Testing:
> - Tier1 passed without new failure on qemu (release) using `-XX:+UnlockExperimentalVMOptions -XX:+UseZbs`.
Looks good.
-------------
Marked as reviewed by fyang (Committer).
PR Review: https://git.openjdk.org/riscv-port-jdk17u/pull/26#pullrequestreview-1358220859
More information about the riscv-port-dev
mailing list