Hi, In this v4 we changed patch 1 to do mark_vs_dirty() after both loads and stores, not just stores.
A new patch, patch 2, was added to clean-up the now unused 'is_store' argument from these functions. All other patches unchanged. Patches missing reviews/acks: patches 1 and 2. Changes from v3: - patch 1: do mark_vs_dirty() for both loads and stores - patch 2 (new): remove the now unused 'is_store' arg from load/store functions - v3 link: https://lore.kernel.org/qemu-riscv/20240220192607.141880-1-dbarb...@ventanamicro.com/ Daniel Henrique Barboza (5): 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/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 | 6 +- target/riscv/insn_trans/trans_rvv.c.inc | 292 ++++++--------------- target/riscv/insn_trans/trans_rvvk.c.inc | 30 +-- target/riscv/translate.c | 6 + target/riscv/vector_helper.c | 1 + 5 files changed, 99 insertions(+), 236 deletions(-) -- 2.43.2
