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

SoftRasterizer: Oops! Fix a performance regression in SoftRasterizerRenderer_SSE2::ClearUsingValues() where the framebuffer was accidentally being cleared twice. (Regression from commit 7509d46.)

This commit is contained in:
rogerman 2018-02-12 13:42:42 -08:00
parent 7509d469b9
commit ab18de05ef
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
desmume/src/rasterize.cpp
View File

@ -2384,7 +2384,6 @@ Render3DError SoftRasterizerRenderer_SSE2::ClearUsingValues(const FragmentColor
this->_clearAttrIsFogged_v128u8 = _mm_set1_epi8(clearAttributes.isFogged); this->_clearAttrIsFogged_v128u8 = _mm_set1_epi8(clearAttributes.isFogged);
this->_clearAttrIsTranslucentPoly_v128u8 = _mm_set1_epi8(clearAttributes.isTranslucentPoly); this->_clearAttrIsTranslucentPoly_v128u8 = _mm_set1_epi8(clearAttributes.isTranslucentPoly);
size_t i = 0;
const bool doMultithreadedClear = (this->_threadCount > 0); const bool doMultithreadedClear = (this->_threadCount > 0);
if (doMultithreadedClear) if (doMultithreadedClear)
@ -2397,13 +2396,12 @@ Render3DError SoftRasterizerRenderer_SSE2::ClearUsingValues(const FragmentColor
else else
{ {
this->ClearUsingValuesLoop(0, this->_framebufferSIMDPixCount); this->ClearUsingValuesLoop(0, this->_framebufferSIMDPixCount);
i = this->_framebufferSIMDPixCount;
} }
#ifdef ENABLE_SSE2 #ifdef ENABLE_SSE2
#pragma LOOPVECTORIZE_DISABLE #pragma LOOPVECTORIZE_DISABLE
#endif #endif
for (; i < this->_framebufferPixCount; i++) for (size_t i = this->_framebufferSIMDPixCount; i < this->_framebufferPixCount; i++)
{ {
this->_framebufferColor[i] = clearColor6665; this->_framebufferColor[i] = clearColor6665;
this->_framebufferAttributes->SetAtIndex(i, clearAttributes); this->_framebufferAttributes->SetAtIndex(i, clearAttributes);