On Thu, Jun 26, 2008 at 12:19 PM, Michal Simek <monstr@xxxxxxxxx> wrote: >> On Thu, Jun 26, 2008 at 5:29 AM, <monstr@xxxxxxxxx> wrote: >>> +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). -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
