On 7/22/20 2:16 AM, frank.ch...@sifive.com wrote: > From: Frank Chang <frank.ch...@sifive.com> > > Sign-extend vmselu.vi and vmsgtu.vi immediate values. > > Signed-off-by: Frank Chang <frank.ch...@sifive.com> > --- > target/riscv/insn_trans/trans_rvv.inc.c | 4 +- > target/riscv/vector_helper.c | 86 +++++++++++++------------ > 2 files changed, 48 insertions(+), 42 deletions(-)
FWIW, not what spike does, but this time at least the manual *does* say the immediate is sign-extended. > + if (vta == 1) { \ > + for (; i < vlmax; i++) { \ > + vext_set_elem_mask(vd, i, 1); \ > + } \ > + } \ This is wrong, but I guess we already talked about vta... r~