RFR: 8357056: RISC-V: Asm fixes - load/store width [v2]
Robbin Ehn
rehn at openjdk.org
Tue May 20 12:02:43 UTC 2025
> Hi, please consider.
>
> While working on https://github.com/openjdk/jdk/pull/25252, I notice:
> - Major op code was just repeat
> - Width coded in binary
> - Stores have mixed up rs1 and rs2
> - Bonus, fsd used a macro for no reason
>
> I think this improves readability.
>
> Tested tier1
>
> Thanks, Robbin
Robbin Ehn has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
- Fixed flh/flw/fld
- Merge branch 'master' into asm_fixes
- Fixes
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25253/files
- new: https://git.openjdk.org/jdk/pull/25253/files/841f85da..2d658948
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25253&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25253&range=00-01
Stats: 18652 lines in 431 files changed: 6820 ins; 9951 del; 1881 mod
Patch: https://git.openjdk.org/jdk/pull/25253.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25253/head:pull/25253
PR: https://git.openjdk.org/jdk/pull/25253
More information about the hotspot-compiler-dev
mailing list