riscv-software-src / riscv-isa-sim

Spike, a RISC-V ISA Simulator
Other
2.43k stars 855 forks source link

Vector amo instruction support #1260

Open hanfeng0114 opened 1 year ago

hanfeng0114 commented 1 year ago

It seems the latest vector spec has excluded vector amo instructions. Will spike excludes the instructions?

aswaterman commented 1 year ago

cc @chihminchao: we will probably need to remove these at some point anyway, since a future subword AMO extension is likely to use that opcode space.

hanfeng0114 commented 1 year ago

Will the final solution be settled soon?

aswaterman commented 1 year ago

Spike will eventually exclude the instructions, yes. But one of the maintainers of the vector support needs to do the work.

chihminchao commented 1 year ago

I agree to remove them and we could dig out the implementation from log in future.

hanfeng0114 commented 1 year ago

So will it be removed soon?

chihminchao commented 1 year ago

@hanfeng0114 Could you send a PR for that ?