On Thu, Jun 26, 2008 at 12:19 PM, Michal Simek [off-list ref] wrote:

quoted

On Thu, Jun 26, 2008 at 5:29 AM,  [off-list ref] wrote:

quoted

+ex_sw:
+       /* Get the destination register number into r5 */
+       lbui    r5, r0, ex_reg_op;
+       /* Form store_word jump table offset (sw_table + (8 * regnum)) */
+       la      r6, r0, sw_table;
+       add     r5, r5, r5;
+       add     r5, r5, r5;
+       add     r5, r5, r5;
+       add     r5, r5, r6;
+       bra     r5;

Possibly stupid question: This is part of the unaligned store word
exception handler, yes? Shouldn't the above add's be addk's to
preserve the state of the carry register pre/post store?

I don't think that addk is important. I have some tests for exception, I want to
cover full exception handling.

Okay. It doesn't match your other exception handlers, though, which is
why I noticed it in the first place (they use addk).