Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-06-28 02:29:43 +00:00
Code Issues Projects Releases Packages Wiki Activity

[WIN32K]

Browse source 
- Improve performance of XFORMOBJ_bXformFixPoints by getting the integer values from the FLOATOBJ once instead of on every iteration

svn path=/trunk/; revision=64261
This commit is contained in:
Timo Kreuzer 2014-09-24 21:27:47 +00:00
parent ae5b9eb418
commit cdb844e88c
1 changed files with 23 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

34
reactos/win32ss/gdi/ntgdi/xformobj.c
View file

@ -1,7 +1,7 @@
/* /*
* PROJECT: ReactOS win32 kernel mode subsystem * PROJECT: ReactOS win32 kernel mode subsystem
* LICENSE: GPL - See COPYING in the top level directory * LICENSE: GPL - See COPYING in the top level directory
* FILE: subsystems/win32/win32k/objects/xformobj.c * FILE: win32ss/gdi/ntgdi/xformobj.c
* PURPOSE: XFORMOBJ API * PURPOSE: XFORMOBJ API
* PROGRAMMER: Timo Kreuzer * PROGRAMMER: Timo Kreuzer
*/ */
@ -195,8 +195,8 @@ XFORMOBJ_iCombine(
MATRIX mx; MATRIX mx;
PMATRIX pmx, pmx1, pmx2; PMATRIX pmx, pmx1, pmx2;
pmx = XFORMOBJ_pmx(pxo); /* Get the source matrices */
pmx1 =XFORMOBJ_pmx(pxo1); pmx1 = XFORMOBJ_pmx(pxo1);
pmx2 = XFORMOBJ_pmx(pxo2); pmx2 = XFORMOBJ_pmx(pxo2);
/* Do a 3 x 3 matrix multiplication with mx as destinantion */ /* Do a 3 x 3 matrix multiplication with mx as destinantion */
@ -210,6 +210,7 @@ XFORMOBJ_iCombine(
FLOATOBJ_Add(&mx.efDy, &pmx2->efDy); FLOATOBJ_Add(&mx.efDy, &pmx2->efDy);
/* Copy back */ /* Copy back */
pmx = XFORMOBJ_pmx(pxo);
*pmx = mx; *pmx = mx;
/* Update accelerators and return complexity */ /* Update accelerators and return complexity */
@ -322,11 +323,14 @@ XFORMOBJ_bXformFixPoints(
if (flAccel & XFORM_UNITY) if (flAccel & XFORM_UNITY)
{ {
/* 1-scale integer transform */ /* 1-scale integer transform */
LONG lM12 = FLOATOBJ_GetLong(&pmx->efM12);
LONG lM21 = FLOATOBJ_GetLong(&pmx->efM21);
i = cPoints - 1; i = cPoints - 1;
do do
{ {
LONG x = pptIn[i].x + pptIn[i].y * FLOATOBJ_GetLong(&pmx->efM21); LONG x = pptIn[i].x + pptIn[i].y * lM21;
LONG y = pptIn[i].y + pptIn[i].x * FLOATOBJ_GetLong(&pmx->efM12); LONG y = pptIn[i].y + pptIn[i].x * lM12;
pptOut[i].y = y; pptOut[i].y = y;
pptOut[i].x = x; pptOut[i].x = x;
} }
@ -335,25 +339,33 @@ XFORMOBJ_bXformFixPoints(
else if (flAccel & XFORM_SCALE) else if (flAccel & XFORM_SCALE)
{ {
/* Diagonal integer transform */ /* Diagonal integer transform */
LONG lM11 = FLOATOBJ_GetLong(&pmx->efM11);
LONG lM22 = FLOATOBJ_GetLong(&pmx->efM22);
i = cPoints - 1; i = cPoints - 1;
do do
{ {
pptOut[i].x = pptIn[i].x * FLOATOBJ_GetLong(&pmx->efM11); pptOut[i].x = pptIn[i].x * lM11;
pptOut[i].y = pptIn[i].y * FLOATOBJ_GetLong(&pmx->efM22); pptOut[i].y = pptIn[i].y * lM22;
} }
while (--i >= 0); while (--i >= 0);
} }
else else
{ {
/* Full integer transform */ /* Full integer transform */
LONG lM11 = FLOATOBJ_GetLong(&pmx->efM11);
LONG lM12 = FLOATOBJ_GetLong(&pmx->efM12);
LONG lM21 = FLOATOBJ_GetLong(&pmx->efM21);
LONG lM22 = FLOATOBJ_GetLong(&pmx->efM22);
i = cPoints - 1; i = cPoints - 1;
do do
{ {
LONG x; LONG x;
x = pptIn[i].x * FLOATOBJ_GetLong(&pmx->efM11); x = pptIn[i].x * lM11;
x += pptIn[i].y * FLOATOBJ_GetLong(&pmx->efM21); x += pptIn[i].y * lM21;
pptOut[i].y = pptIn[i].y * FLOATOBJ_GetLong(&pmx->efM22); pptOut[i].y = pptIn[i].y * lM22;
pptOut[i].y += pptIn[i].x * FLOATOBJ_GetLong(&pmx->efM12); pptOut[i].y += pptIn[i].x * lM12;
pptOut[i].x = x; pptOut[i].x = x;
} }
while (--i >= 0); while (--i >= 0);