Re: [PATCH v5 0/7] riscv: set vstart_eq_zero on mark_vs_dirty

[Daniel Henrique Barboza]

Richard,

On 2/20/24 23:22, Daniel Henrique Barboza wrote:
> 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.


I missed your acks from v4 in patches 1 and 2 . I'll add them back in the next
version.


Thanks,


Daniel

> 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: 
> 20240220222510.209448-1-dbarboza@ventanamicro.com/">https://lore.kernel.org/qemu-riscv/20240220222510.209448-1-dbarboza@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(-)