[riscv-port] Integrated: 8276989: riscv: C1 should not allocate x3, which is the gp register
duke at openjdk.java.net
Mon Nov 15 09:28:59 UTC 2021
On Thu, 11 Nov 2021 09:28:13 GMT, zhengxiaolinX <duke at openjdk.java.net> wrote:
> Hi team,
> x3 is a special register on RISCV and we should not use it as a register allocation candidate - if users write JNI programs referencing global variables, where ld could link global reference loading logic to an instruction like 'ld $Rtmp, $offset(x3)', and try to call vm functions, C1 will zap x3's value and causes a crash. It seems C2 doesn't use x3 so we are just aligning the logic with C2 properly.
This pull request has now been integrated.
Author: yunyao.zxl <yunyao.zxl at alibaba-inc.com>
Committer: Fei Yang <fyang at openjdk.org>
Stats: 8 lines in 3 files changed: 2 ins; 3 del; 3 mod
8276989: riscv: C1 should not allocate x3, which is the gp register
Reviewed-by: fyang, shade
More information about the riscv-port-dev