OSDN Git Service

target/riscv: Fix LMUL check to use VLEN
authorRob Bradford <rbradford@rivosinc.com>
Tue, 18 Jul 2023 13:11:44 +0000 (14:11 +0100)
committerAlistair Francis <alistair.francis@wdc.com>
Wed, 19 Jul 2023 04:37:26 +0000 (14:37 +1000)
commit32be32509987fbe42cf5c2fd3cea3c2ad6eae179
tree9c4908a65064bca27fc47a0ce57df6f88e389d1c
parenta916dc954bb5d5aebe5bfcc222cbe9f984118442
target/riscv: Fix LMUL check to use VLEN

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 narrowest supported SEW value and ELEN is the widest
supported SEW value. In the standard extensions, SEWMIN=8. For standard
vector extensions with ELEN=32, fractional LMULs of 1/2 and 1/4 must be
supported. For standard vector extensions with ELEN=64, fractional LMULs
of 1/2, 1/4, and 1/8 must be supported." Elsewhere in the specification
it makes clear that VLEN>=ELEN.

From inspection this new check allows:

VLEN=ELEN=64 1/2, 1/4, 1/8 for SEW >=8
VLEN=ELEN=32 1/2, 1/4 for SEW >=8

Fixes: d9b7609a1fb2 ("target/riscv: rvv-1.0: configure instructions")
Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
Reviewed-by: Weiwei Li <liweiwei@iscas.ac.cn>
Message-Id: <20230718131316.12283-2-rbradford@rivosinc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
target/riscv/vector_helper.c