Re: [PATCH] target/riscv: Fix LMUL check to use minimum SEW

2023-07-17 Thread LIU Zhiwei
On 2023/7/18 8:43, Weiwei Li wrote: On 2023/7/17 23:13, Rob Bradford wrote: On Thu, 2023-07-06 at 21:22 +0800, Weiwei Li wrote: On 2023/7/6 18:44, Rob Bradford wrote: The previous check was failing with: ELEN = 64 SEW = 16 and LMUL = 1/8 (encoded as 5) which is a valid combination. Fix

Re: [PATCH] target/riscv: Fix LMUL check to use minimum SEW

2023-07-17 Thread Weiwei Li
On 2023/7/17 23:13, Rob Bradford wrote: On Thu, 2023-07-06 at 21:22 +0800, Weiwei Li wrote: On 2023/7/6 18:44, Rob Bradford wrote: The previous check was failing with: ELEN = 64 SEW = 16 and LMUL = 1/8 (encoded as 5) which is a valid combination. Fix the check to correctly match the

Re: [PATCH] target/riscv: Fix LMUL check to use minimum SEW

2023-07-17 Thread Rob Bradford
On Thu, 2023-07-06 at 21:22 +0800, Weiwei Li wrote: > > On 2023/7/6 18:44, Rob Bradford wrote: > > The previous check was failing with: > > > > ELEN = 64 SEW = 16 and LMUL = 1/8 (encoded as 5) which is a valid > > combination. > > > > Fix the check to correctly match the specification by using

Re: [PATCH] target/riscv: Fix LMUL check to use minimum SEW

2023-07-06 Thread Weiwei Li
On 2023/7/6 18:44, Rob Bradford wrote: The previous check was failing with: ELEN = 64 SEW = 16 and LMUL = 1/8 (encoded as 5) which is a valid combination. Fix the check to correctly match the specification by using minimum SEW rather than the active SEW. From the specification: "In

Re: [PATCH] target/riscv: Fix LMUL check to use minimum SEW

2023-07-06 Thread Daniel Henrique Barboza
On 7/6/23 07:44, Rob Bradford wrote: The previous check was failing with: ELEN = 64 SEW = 16 and LMUL = 1/8 (encoded as 5) which is a valid combination. Fix the check to correctly match the specification by using minimum SEW rather than the active SEW. From the specification: "In

[PATCH] target/riscv: Fix LMUL check to use minimum SEW

2023-07-06 Thread Rob Bradford
The previous check was failing with: ELEN = 64 SEW = 16 and LMUL = 1/8 (encoded as 5) which is a valid combination. Fix the check to correctly match the specification by using minimum SEW rather than the active SEW. >From the specification: "In general, the requirement is to support LMUL ≥