[PATCH v4 0/6] support subsets of Float-Point in Integer Registers extensions

Weiwei Li posted 6 patches 2 years, 3 months ago
Test checkpatch passed
Failed in applying to current master (apply log)
There is a newer version of this series
target/riscv/cpu.c                        |  17 ++
target/riscv/cpu.h                        |   4 +
target/riscv/cpu_helper.c                 |   6 +-
target/riscv/csr.c                        |  25 +-
target/riscv/fpu_helper.c                 | 178 ++++++------
target/riscv/helper.h                     |   4 +-
target/riscv/insn_trans/trans_rvd.c.inc   | 285 ++++++++++++++-----
target/riscv/insn_trans/trans_rvf.c.inc   | 314 +++++++++++++-------
target/riscv/insn_trans/trans_rvzfh.c.inc | 332 +++++++++++++++-------
target/riscv/internals.h                  |  32 ++-
target/riscv/translate.c                  | 154 ++++++++++
11 files changed, 980 insertions(+), 371 deletions(-)
[PATCH v4 0/6] support subsets of Float-Point in Integer Registers extensions
Posted by Weiwei Li 2 years, 3 months ago
This patchset implements RISC-V Float-Point in Integer Registers extensions(Version 1.0), which includes Zfinx, Zdinx, Zhinx and Zhinxmin extension. 

Specification:
https://github.com/riscv/riscv-zfinx/blob/main/zfinx-1.0.0.pdf

The port is available here:
https://github.com/plctlab/plct-qemu/tree/plct-zfinx-upstream-v4

To test this implementation, specify cpu argument with 'Zfinx =true,Zdinx=true,Zhinx=true,Zhinxmin=true' with 'g=false,f=false,d=false,Zfh=false,Zfhmin=false'
This implementation can pass gcc tests, ci result can be found in https://ci.rvperf.org/job/plct-qemu-zfinx-upstream/.

v4:
* combine register pair check for rv32 zdinx
* clear mstatus.FS when RVF is disabled by write_misa

v3:
* delete unused reset for mstatus.FS
* use positive test for RVF instead of negative test for ZFINX
* replace get_ol with get_xl
* use tcg_gen_concat_tl_i64 to unify tcg_gen_concat_i32_i64 and tcg_gen_deposit_i64

v2:
* hardwire mstatus.FS to zero when enable zfinx
* do register-pair check at the begin of translation
* optimize partial implemention as suggested

Weiwei Li (6):
  target/riscv: add cfg properties for zfinx, zdinx and zhinx{min}
  target/riscv: hardwire mstatus.FS to zero when enable zfinx
  target/riscv: add support for zfinx
  target/riscv: add support for zdinx
  target/riscv: add support for zhinx/zhinxmin
  target/riscv: expose zfinx, zdinx, zhinx{min} properties

 target/riscv/cpu.c                        |  17 ++
 target/riscv/cpu.h                        |   4 +
 target/riscv/cpu_helper.c                 |   6 +-
 target/riscv/csr.c                        |  25 +-
 target/riscv/fpu_helper.c                 | 178 ++++++------
 target/riscv/helper.h                     |   4 +-
 target/riscv/insn_trans/trans_rvd.c.inc   | 285 ++++++++++++++-----
 target/riscv/insn_trans/trans_rvf.c.inc   | 314 +++++++++++++-------
 target/riscv/insn_trans/trans_rvzfh.c.inc | 332 +++++++++++++++-------
 target/riscv/internals.h                  |  32 ++-
 target/riscv/translate.c                  | 154 ++++++++++
 11 files changed, 980 insertions(+), 371 deletions(-)

-- 
2.17.1