[riscv-port] RFR: 8277883: riscv: Fix a temp register usage in eden_allocate

zhengxiaolinX duke at openjdk.java.net
Mon Nov 29 03:51:47 UTC 2021


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

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

Commit messages:
 - Fix a temp register usage in eden_allocate

Changes: https://git.openjdk.java.net/riscv-port/pull/16/files
 Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=16&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8277883
  Stats: 6 lines in 1 file changed: 2 ins; 0 del; 4 mod
  Patch: https://git.openjdk.java.net/riscv-port/pull/16.diff
  Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/16/head:pull/16

PR: https://git.openjdk.java.net/riscv-port/pull/16


More information about the riscv-port-dev mailing list