[riscv-port] Integrated: 8277883: riscv: Fix a temp register usage in eden_allocate
zhengxiaolinX
duke at openjdk.java.net
Mon Nov 29 08:14:01 UTC 2021
On Mon, 29 Nov 2021 03:43:32 GMT, zhengxiaolinX <duke at openjdk.java.net> wrote:
> Hi team,
>
> A trivial fix for a small C1 crash - this issue could be directly reproduced by using `java -XX:+UseSerialGC -XX:-UseTLAB -XX:TieredStopAtLevel=1`. The reason is simple: the eden_allocate uses t2 as a register and zaps it, whereas C1 will use it as a register allocation candidate, leading to a crash. In this function, t0 never gets a use so we can use it safely. Tested in all cases. [The original patch](https://github.com/riscv-collab/riscv-openjdk/pull/15)
>
> Thanks,
> Xiaolin
This pull request has now been integrated.
Changeset: 1101c14f
Author: yunyao.zxl <yunyao.zxl at alibaba-inc.com>
Committer: Fei Yang <fyang at openjdk.org>
URL: https://git.openjdk.java.net/riscv-port/commit/1101c14fd5c4b5ea89e6f3868bf9917d772445fa
Stats: 4 lines in 1 file changed: 0 ins; 0 del; 4 mod
8277883: riscv: Fix a temp register usage in eden_allocate
Reviewed-by: fyang
-------------
PR: https://git.openjdk.java.net/riscv-port/pull/16
More information about the riscv-port-dev
mailing list