Re: [PATCH v2] target/riscv: Fix LMUL check to use VLEN

2023-07-18 Thread Alistair Francis
On Tue, Jul 18, 2023 at 11:14 PM Rob Bradford wrote: > > The previous check was failing with: > > VLEN=128 ELEN = 64 SEW = 16 and LMUL = 1/8 which is a > valid combination. > > Fix the check to allow valid combinations when VLEN is a multiple of > ELEN. > > From the specification: > > "In

Re: [PATCH v2] target/riscv: Fix LMUL check to use VLEN

2023-07-18 Thread Weiwei Li
On 2023/7/18 21:11, Rob Bradford wrote: The previous check was failing with: VLEN=128 ELEN = 64 SEW = 16 and LMUL = 1/8 which is a valid combination. Fix the check to allow valid combinations when VLEN is a multiple of ELEN. From the specification: "In general, the requirement is to

[PATCH v2] target/riscv: Fix LMUL check to use VLEN

2023-07-18 Thread Rob Bradford
The previous check was failing with: VLEN=128 ELEN = 64 SEW = 16 and LMUL = 1/8 which is a valid combination. Fix the check to allow valid combinations when VLEN is a multiple of ELEN. >From the specification: "In general, the requirement is to support LMUL ≥ SEWMIN/ELEN, where SEWMIN is the