ShuriZma
/
pcsx2
mirror of https://github.com/PCSX2/pcsx2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

superVU: Fix missing breaks. 

-This was actually a bug, may improve some games that were buggy in superVU, but these functions aren't often used.
-Coverity CID 146865 & 146864:  In recVUMI_ESIN(VURegs *, int): Missing break statement between cases in switch statement (CWE-484)
-Coverity CID 146863 & 146862: In recVUMI_EEXP(VURegs *, int): Missing break statement between cases in switch statement (CWE-484)
-Coverity CID 146855 & 146854: In recVUMI_EATAN(VURegs *, int): Missing break statement between cases in switch statement (CWE-484)
This commit is contained in:
refractionpcsx2 2015-10-04 15:04:34 +01:00
parent 3618983aaa
commit ad784a56ec
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
pcsx2/x86/sVU_Lower.cpp
View File

@ -1847,9 +1847,9 @@ void recVUMI_ESIN( VURegs *VU, int info )
//Console.WriteLn("recVUMI_ESIN"); //Console.WriteLn("recVUMI_ESIN");
if( (xmmregs[EEREC_S].mode & MODE_WRITE) && (xmmregs[EEREC_S].mode&MODE_NOFLUSH) ) { if( (xmmregs[EEREC_S].mode & MODE_WRITE) && (xmmregs[EEREC_S].mode&MODE_NOFLUSH) ) {
switch(_Fsf_) { switch(_Fsf_) {
case 0: SSE_MOVSS_XMM_to_M32((uptr)s_tempmem, EEREC_S); case 0: SSE_MOVSS_XMM_to_M32((uptr)s_tempmem, EEREC_S); break;
case 1: SSE_MOVLPS_XMM_to_M64((uptr)s_tempmem, EEREC_S); case 1: SSE_MOVLPS_XMM_to_M64((uptr)s_tempmem, EEREC_S); break;
default: SSE_MOVHPS_XMM_to_M64((uptr)&s_tempmem[2], EEREC_S); default: SSE_MOVHPS_XMM_to_M64((uptr)&s_tempmem[2], EEREC_S); break;
} }
FLD32((uptr)&s_tempmem[_Fsf_]); FLD32((uptr)&s_tempmem[_Fsf_]);
} }
@ -1878,9 +1878,9 @@ void recVUMI_EATAN( VURegs *VU, int info )
//Console.WriteLn("recVUMI_EATAN"); //Console.WriteLn("recVUMI_EATAN");
if( (xmmregs[EEREC_S].mode & MODE_WRITE) && (xmmregs[EEREC_S].mode&MODE_NOFLUSH) ) { if( (xmmregs[EEREC_S].mode & MODE_WRITE) && (xmmregs[EEREC_S].mode&MODE_NOFLUSH) ) {
switch(_Fsf_) { switch(_Fsf_) {
case 0: SSE_MOVSS_XMM_to_M32((uptr)s_tempmem, EEREC_S); case 0: SSE_MOVSS_XMM_to_M32((uptr)s_tempmem, EEREC_S); break;
case 1: SSE_MOVLPS_XMM_to_M64((uptr)s_tempmem, EEREC_S); case 1: SSE_MOVLPS_XMM_to_M64((uptr)s_tempmem, EEREC_S); break;
default: SSE_MOVHPS_XMM_to_M64((uptr)&s_tempmem[2], EEREC_S); default: SSE_MOVHPS_XMM_to_M64((uptr)&s_tempmem[2], EEREC_S); break;
} }
FLD32((uptr)&s_tempmem[_Fsf_]); FLD32((uptr)&s_tempmem[_Fsf_]);
} }
@ -1910,9 +1910,9 @@ void recVUMI_EEXP( VURegs *VU, int info )
if( (xmmregs[EEREC_S].mode & MODE_WRITE) && (xmmregs[EEREC_S].mode&MODE_NOFLUSH) ) { if( (xmmregs[EEREC_S].mode & MODE_WRITE) && (xmmregs[EEREC_S].mode&MODE_NOFLUSH) ) {
switch(_Fsf_) { switch(_Fsf_) {
case 0: SSE_MOVSS_XMM_to_M32((uptr)s_tempmem, EEREC_S); case 0: SSE_MOVSS_XMM_to_M32((uptr)s_tempmem, EEREC_S); break;
case 1: SSE_MOVLPS_XMM_to_M64((uptr)s_tempmem, EEREC_S); case 1: SSE_MOVLPS_XMM_to_M64((uptr)s_tempmem, EEREC_S); break;
default: SSE_MOVHPS_XMM_to_M64((uptr)&s_tempmem[2], EEREC_S); default: SSE_MOVHPS_XMM_to_M64((uptr)&s_tempmem[2], EEREC_S); break;
} }
FMUL32((uptr)&s_tempmem[_Fsf_]); FMUL32((uptr)&s_tempmem[_Fsf_]);
} }