Re: [PATCH] target/riscv/vector_helper.c: create vext_set_tail_elems_1s()

Thanks for the reviews! I decided to take this patch (acks included) and send it in together with a cleanup of the env_archcpu() usages in vector_helper.c: https://lists.gnu.org/archive/html/qemu-devel/2023-02/msg07566.html Thanks, Daniel On 2/21/23 15:45, Daniel Henrique Barboza wrote:

Re: [PATCH] target/riscv/vector_helper.c: create vext_set_tail_elems_1s()

Reviewed-by: Frank Chang On Wed, Feb 22, 2023 at 2:46 AM Daniel Henrique Barboza < dbarb...@ventanamicro.com> wrote: > Commit 752614cab8e6 ("target/riscv: rvv: Add tail agnostic for vector > load / store instructions") added code to set the tail elements to 1 in > the end of vext_ldst_stride(),

Re: [PATCH] target/riscv/vector_helper.c: create vext_set_tail_elems_1s()

On 2023/2/22 02:45, Daniel Henrique Barboza wrote: Commit 752614cab8e6 ("target/riscv: rvv: Add tail agnostic for vector load / store instructions") added code to set the tail elements to 1 in the end of vext_ldst_stride(), vext_ldst_us(), vext_ldst_index() and vext_ldff(). Aside from a env->vl

[PATCH] target/riscv/vector_helper.c: create vext_set_tail_elems_1s()

Commit 752614cab8e6 ("target/riscv: rvv: Add tail agnostic for vector load / store instructions") added code to set the tail elements to 1 in the end of vext_ldst_stride(), vext_ldst_us(), vext_ldst_index() and vext_ldff(). Aside from a env->vl versus an evl value being used in the first loop, the