Re: [PATCH v2] target/riscv: Fix shift count overflow

2024-02-25 Thread Philippe Mathieu-Daudé
On 25/2/24 04:27, demin.han wrote: The result of (8 - 3 - vlmul) is negtive when vlmul >= 6, Typo "negative". and results in wrong vill. Signed-off-by: demin.han --- target/riscv/vector_helper.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/target/riscv/vector_

Re: [PATCH v2] target/riscv: Fix shift count overflow

2024-02-25 Thread Daniel Henrique Barboza
On 2/25/24 00:27, demin.han wrote: The result of (8 - 3 - vlmul) is negtive when vlmul >= 6, and results in wrong vill. Signed-off-by: demin.han --- Reviewed-by: Daniel Henrique Barboza target/riscv/vector_helper.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --gi

[PATCH v2] target/riscv: Fix shift count overflow

2024-02-24 Thread demin.han
The result of (8 - 3 - vlmul) is negtive when vlmul >= 6, and results in wrong vill. Signed-off-by: demin.han --- target/riscv/vector_helper.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/target/riscv/vector_helper.c b/target/riscv/vector_helper.c index 84cec73eb2..fe