Update to bring this bug to attention. Finally found the bug that causes Soul Nomad damage calculations being wrong. I imagine it has a very high impact on game compatibility.

git-svn-id: http://pcsx2-playground.googlecode.com/svn/trunk@338 a6443dda-0b58-4228-96e9-037be469359c
This commit is contained in:
ramapcsx2 2008-11-18 15:57:28 +00:00 committed by Gregory Hainaut
parent 098e6b509b
commit c26c73c44b
1 changed files with 6 additions and 1 deletions

View File

@ -682,13 +682,18 @@ void rpropSPECIAL(EEINST* prev, EEINST* pinst)
case 24: // mult
// can do unsigned mult only if HI isn't used
//using this allocation for temp causes the emu to crash
//temp = (pinst->regs[XMMGPR_HI]&(EEINST_LIVE0|EEINST_LIVE1))?0:EEINST_MMX;
temp = 0;
rpropSetWrite(XMMGPR_LO, EEINST_LIVE1);
rpropSetWrite(XMMGPR_HI, EEINST_LIVE1);
rpropSetWrite(_Rd_, EEINST_LIVE1);
// fixme - temp is always 0, so I doubt the next three lines are right.
// fixme - temp is always 0, so I doubt the next three lines are right. (arcum42)
// Yep, its wrong. Using always 0 causes the wrong damage calculations in Soul Nomad.
// This bug i very important to fix!! (rama)
rpropSetRead(_Rs_, temp);
rpropSetRead(_Rt_, temp);
pinst->info |= temp;