fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-07 09:18:04 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

prog_optimize: Fix reallocating registers for shaders with loops

Browse source 
Registers that are used inside of loops need to be considered live
starting with the first instruction of the outermost loop.

https://bugs.freedesktop.org/show_bug.cgi?id=34370

(cherry picked from commit 18dcbd358f)

Reviewed-by: Eric Anholt <eric@anholt.net>
This commit is contained in:
Tom Stellard 2011-03-27 01:17:43 -07:00
parent e3dcb53e6e
commit 9e2c9cdf48
1 changed files with 14 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/mesa/program/prog_optimize.c
View file

@ -937,24 +937,35 @@ update_interval(GLint intBegin[], GLint intEnd[],
GLuint index, GLuint ic)
{
int i;
GLuint begin = ic;
GLuint end = ic;
/* If the register is used in a loop, extend its lifetime through the end
* of the outermost loop that doesn't contain its definition.
*/
for (i = 0; i < loopStackDepth; i++) {
if (intBegin[index] < loopStack[i].Start) {
ic = loopStack[i].End;
end = loopStack[i].End;
break;
}
}
/* Variables that are live at the end of a loop will also be live at the
* beginning, so an instruction inside of a loop should have its live
* interval begin at the start of the outermost loop.
*/
if (loopStackDepth > 0 && ic > loopStack[0].Start && ic < loopStack[0].End) {
begin = loopStack[0].Start;
}
ASSERT(index < REG_ALLOCATE_MAX_PROGRAM_TEMPS);
if (intBegin[index] == -1) {
ASSERT(intEnd[index] == -1);
intBegin[index] = intEnd[index] = ic;
intBegin[index] = begin;
intEnd[index] = end;
}
else {
intEnd[index] = ic;
intEnd[index] = end;
}
}