RFR: 8357056: RISC-V: Asm fixes - load/store width [v3]

Robbin Ehn rehn at openjdk.org
Thu May 22 06:56:36 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 five additional commits since the last revision:

 - Rd/Rs->Rs2/Rs1
 - Merge branch 'master' into asm_fixes
 - 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/2d658948..aec6511a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25253&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25253&range=01-02

  Stats: 40289 lines in 432 files changed: 27716 ins; 9995 del; 2578 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