Integrated: 8322154: RISC-V: JDK-8315743 missed change in MacroAssembler::load_reserved
Gui Cao
gcao at openjdk.org
Tue Dec 19 07:48:46 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)
This pull request has now been integrated.
Changeset: 59073fa3
Author: Gui Cao <gcao at openjdk.org>
Committer: Robbin Ehn <rehn at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/59073fa3eb7d04d9e0f08fbef70c9db6ffde296a
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
8322154: RISC-V: JDK-8315743 missed change in MacroAssembler::load_reserved
Reviewed-by: fyang, rehn, luhenry
-------------
PR: https://git.openjdk.org/jdk/pull/17117
More information about the hotspot-dev
mailing list