Hi, In this new version we removed the remaining brconds() from trans_rvbf16.c.inc like Richard suggested in patch 3. Richard, I kept your ack in that patch.
This change caused the 'cpu_vstart' global to be unused. A new patch (4) was added to remove it. Series based on alistair/riscv-to-apply.next. Patches missing acks: patch 4. Changes from v4: - patch 3: - remove vstart >= vl and vl = 0 brconds from trans_fvbf16.c.inc - patch 4 (new): - remove cpu_vstart - v4 link: https://lore.kernel.org/qemu-riscv/20240220222510.209448-1-dbarb...@ventanamicro.com/ Daniel Henrique Barboza (6): trans_rvv.c.inc: mark_vs_dirty() before loads and stores trans_rvv.c.inc: remove 'is_store' bool from load/store fns target/riscv: remove 'over' brconds from vector trans target/riscv/translate.c: remove 'cpu_vstart' global target/riscv/vector_helper.c: set vstart = 0 in GEN_VEXT_VSLIDEUP_VX() trans_rvv.c.inc: remove redundant mark_vs_dirty() calls Ivan Klokov (1): target/riscv: Clear vstart_qe_zero flag target/riscv/insn_trans/trans_rvbf16.c.inc | 18 +- target/riscv/insn_trans/trans_rvv.c.inc | 292 ++++++--------------- target/riscv/insn_trans/trans_rvvk.c.inc | 30 +-- target/riscv/translate.c | 10 +- target/riscv/vector_helper.c | 1 + 5 files changed, 100 insertions(+), 251 deletions(-) -- 2.43.2
