[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 04/16] target/arm: Remove unnecessary range check for VSHL
|
From: |
Peter Maydell |
|
Subject: |
Re: [PATCH v3 04/16] target/arm: Remove unnecessary range check for VSHL |
|
Date: |
Tue, 12 May 2020 14:53:58 +0100 |
On Fri, 8 May 2020 at 16:22, Richard Henderson
<address@hidden> wrote:
>
> In 1dc8425e551, while converting to gvec, I added an extra range check
> against the shift count. This was unnecessary because the encoding of
> the shift count produces 0 to the element size - 1.
>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> target/arm/translate.c | 12 ++----------
> 1 file changed, 2 insertions(+), 10 deletions(-)
>
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- Re: [PATCH v3 01/16] target/arm: Create gen_gvec_[us]sra, (continued)
- [PATCH v3 03/16] target/arm: Create gen_gvec_{sri,sli}, Richard Henderson, 2020/05/08
- [PATCH v3 04/16] target/arm: Remove unnecessary range check for VSHL, Richard Henderson, 2020/05/08
- Re: [PATCH v3 04/16] target/arm: Remove unnecessary range check for VSHL,
Peter Maydell <=
- [PATCH v3 05/16] target/arm: Tidy handle_vec_simd_shri, Richard Henderson, 2020/05/08
- [PATCH v3 08/16] target/arm: Swap argument order for VSHL during decode, Richard Henderson, 2020/05/08
- [PATCH v3 07/16] target/arm: Create gen_gvec_{mla,mls}, Richard Henderson, 2020/05/08
- [PATCH v3 13/16] target/arm: Pass pointer to qc to qrdmla/qrdmls, Richard Henderson, 2020/05/08
- [PATCH v3 14/16] target/arm: Clear tail in gvec_fmul_idx_*, gvec_fmla_idx_*, Richard Henderson, 2020/05/08