[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