RFR: 8322154: RISC-V: JDK-8315743 missed change in MacroAssembler::load_reserved

Fri Dec 15 09:19:38 UTC 2023

On Fri, 15 Dec 2023 03:56:04 GMT, Gui Cao <gcao at openjdk.org> wrote:

> The fix for https://bugs.openjdk.org/browse/JDK-8315743 touches MacroAssembler::load_reserved replacing `t0` with `dst`. But it missed change for the third case (that is `uint32`) of the switch in this assember function. We should also replace `t0` used in `zero_extend` with `dst`. @robehn can you help confirm this?
> 
> - [x]  Run tier1 tests on qemu 8.1.50 with UseRVV (release)

I notice some of RV PRs goes a bit fast. Note https://openjdk.org/guide/ -> "Allow enough time for review".
Hence I can sponsor, but not now.

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

PR Comment: https://git.openjdk.org/jdk/pull/17117#issuecomment-1857540340