Merge pull request #420 from lioncash/qflag

dyncom: Actually set the Q flag for SMLABB/SMLABT/SMLATB/SMLATT
10 years ago · fb0c749223
parent 8b1ec1a82a e08a39a2f4
commit fb0c749223
1 changed files with 2 additions and 1 deletions
--- a/src/core/arm/dyncom/arm_dyncom_interpreter.cpp
+++ b/src/core/arm/dyncom/arm_dyncom_interpreter.cpp
@ -5571,7 +5571,8 @@ unsigned InterpreterMainLoop(ARMul_State* state) {
                operand2 = (BIT(RS, 31)) ? (BITS(RS, 16, 31) | 0xffff0000) : BITS(RS, 16, 31);
            RD = operand1 * operand2 + RN;

-            // TODO: FIXME: UPDATE Q FLAGS
+            if (AddOverflow(operand1 * operand2, RN, RD))
+                cpu->Cpsr |= (1 << 27);
        }
        cpu->Reg[15] += GET_INST_SIZE(cpu);
        INC_PC(sizeof(smla_inst));