VIA-Research / uPIMulator

MIT License
103 stars 14 forks source link

Problem with 'seqread' instruction #2

Open marzieh-barkhordar opened 7 months ago

marzieh-barkhordar commented 7 months ago

I faced issues adding a new benchmark to uPIMulator. While working with the MRAM mapped onto the WRAM, using sequential readers (seqread), I realized the problem was with this instruction. After changing it to 'mram_read()', the issue was fixed. It seems 'seqread' doesn't function as expected, yet the simulator didn't crash. Can you confirm if 'seqread' is an unimplemented instruction?

Thank you.

bongjoonhyun commented 7 months ago

Hello,

we have not tested seqread function extensively, whereas mram_read function has been extensively tested. However, if the simulation is finished without crashing, seqread does not contain unimplemented instructions. If the simulator encounters an unimplemented instruction, the simulator crashes.

Thank you!