OpenRISC 1000
l.mtspr incorrectly implemented in Or1ksim #100
Closed jeremybennett opened this issue over 14 years ago
jeremybennett commented over 14 years ago
<p>
The implementation of <tt>l.mtspr</tt> in Or1ksim is incorrect. The immediate values should be ORed with the contents of rA, not added to it.
</p>
<p>
In practice this causes little difficulty, since invariably r0 is used for rA and addition and ORing then yield the same result.
<p>
Reported in <em>Implementation and Verification of a CPU Subsystem for Multimode RF Transceivers</em> by Waqas Ahmed of the Royal Institute of Technology, Sweden.
</p>
<p>
Jeremy
</p>
<p>
-- <br />
Tel: +44 (1590) 610184<br />
Cell: +44 (7970) 676050<br />
SkypeID: jeremybennett<br />
Email: <a href="mailto:jeremy.bennett@embecosm.com">jeremy.bennett@embecosm.com</a><br />
Web: <a href="http://www.embecosm.com">www.embecosm.com</a>
</p>
jeremybennett was assigned over 14 years ago
jeremybennett commented over 14 years ago
<p>
The same problem also applies to <tt>l.mfspr</tt>.
</p>
<p>
Jeremy
</p>
jeremybennett commented over 14 years ago
<p>
Corrected in top of tree. Regression tests added.
</p>
<p>
Marking closed.
</p>
<p>
Jeremy
</p>
jeremybennett closed this over 14 years ago
© copyright 1999-2018 OpenCores.org, equivalent to Oliscience, all rights reserved. OpenCores®, registered trademark.