On 8/17/20 1:49 AM, frank.ch...@sifive.com wrote: > From: Frank Chang <frank.ch...@sifive.com> > > Clear tail elements only if VTA is agnostic. > > Signed-off-by: Frank Chang <frank.ch...@sifive.com> > --- > target/riscv/insn32.decode | 20 ++++++++++---------- > target/riscv/insn_trans/trans_rvv.inc.c | 2 +- > target/riscv/vector_helper.c | 14 ++++---------- > 3 files changed, 15 insertions(+), 21 deletions(-)
The patch description is wrong, but the code is correct. One nit: > - uint8_t carry = vext_elem_mask(v0, i); \ > + uint8_t carry = !vm ? vext_elem_mask(v0, i) : 0; \ ETYPE carry = !vm && vext_elem_mask(v0, i); r~