openhwgroup / corev-binutils-gdb

GNU General Public License v2.0
9 stars 26 forks source link

Post-Increment Register-Immediate Load should be updated #87

Closed ChunyuLiao closed 1 year ago

ChunyuLiao commented 1 year ago

https://github.com/openhwgroup/corev-binutils-gdb/blob/development/gas/testsuite/gas/riscv/cv-mem-lb.s cv.lb t0, -2048(t2) cv.lb t4, 15(t1) cv.lb t3, 2047(t5)

these instructions should be updated to: cv.lb t0, -2048(t2!) cv.lb t4, 15(t1!) cv.lb t3, 2047(t5!)

image
NandniJamnadas commented 1 year ago

This is a mistake in the GNU assembler and in the testing. We have accidentally defined cv.lb rd, imm(rs) as a synonym of standard RISC-V lb. The same appears for all the other CORE-V loads and stores. We will remove this and update the tests.

NandniJamnadas commented 1 year ago

This issue is now fixed with pull request #90 CV32E40Pv2 Update Post Increment Load/Store.