Integrated: 8354815: RISC-V: Change type of bitwise rotation shift to iRegIorL2I

Anjian-Wen duke at openjdk.org
Fri Apr 18 02:23:46 UTC 2025


On Mon, 14 Apr 2025 09:46:34 GMT, Anjian-Wen <duke at openjdk.org> wrote:

> There is no need to do a type conversion when the shift amount of bitwise rotation is an integer converted from long (ConvL2I).
> There reason is that these instruction performs a rotate right/left of source by the amount in the least-significant 5/6 bits
> of the shift amount depending on the width of the operation (32-bit/64-bit). For 32-bit operations, the resulting 32-bit
> value is sign-extended by copying bit 31 to all of the more-significant bits. This means that we could use iRegIorL2I type for
> source for these 32-bit operations as well.
> 
> Jtreg
> hs:tier1-hs:tier3 tested on linux-riscv64 platform equipped with Zbb.

This pull request has now been integrated.

Changeset: 0995b940
Author:    Anjian-Wen <wenanjian at bytedance.com>
Committer: Fei Yang <fyang at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/0995b9409d910d816276673b5c06fdf7826bfac7
Stats:     27 lines in 3 files changed: 14 ins; 2 del; 11 mod

8354815: RISC-V: Change type of bitwise rotation shift to iRegIorL2I

Reviewed-by: fyang, fjiang

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

PR: https://git.openjdk.org/jdk/pull/24618


More information about the hotspot-compiler-dev mailing list