RFR: 8322154: RISC-V: JDK-8315743 missed change in MacroAssembler::load_reserved
Fei Yang
fyang at openjdk.org
Fri Dec 15 04:07:48 UTC 2023
On Fri, 15 Dec 2023 03:56:04 GMT, Gui Cao <gcao at openjdk.org> wrote:
> The fix for JDK-8315743 touches MacroAssembler::load_reserved replacing `t0` with `dst`. But it missed change for the third case (that is `uint32`) if the switch in this assember function. We should also replace `t0` used in `zero_extend` with `dst`. @robehn can you help confirm this?
>
> - [ ] Run tier1 tests on qemu 8.1.50 with UseRVV (release)
Good catch! I think we should also fix this for JDK-22.
-------------
Marked as reviewed by fyang (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/17117#pullrequestreview-1783089797
More information about the hotspot-dev
mailing list