OpenXiangShan / NEMU

Other
236 stars 90 forks source link

fix(vmvnr): vmvnr will not set vtype register, no need to check vtype.vill #511

Closed xyyy1420 closed 2 months ago

Ziyue-Zhang commented 2 months ago
image image

In spike, vmvnr uses require_vector(true). In require_vector(true), it checks vill.

xyyy1420 commented 2 months ago

image

emm, but the manual says "without needing to know or change 'vl' or 'vtype'"

Ziyue-Zhang commented 2 months ago

https://github.com/riscv-software-src/riscv-isa-sim/issues/1302

Aswaterman thought the spec is wrong

xyyy1420 commented 2 months ago

ok, i will close