ra: don't consider nodes for spilling we don't need to
authorConnor Abbott <cwabbott0@gmail.com>
Fri, 1 Aug 2014 01:57:22 +0000 (18:57 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 13 Aug 2014 18:43:30 +0000 (11:43 -0700)
commit03f4084d28d8a6cd271cad58f5148e1127f83757
tree108aff51d377739f219ae1bbd2fe64dab60b5106
parent567e2769b81863b6dffdac3826a6b729ce6ea37c
ra: don't consider nodes for spilling we don't need to

Previously, we would consider any optimistically colored nodes for
spilling. However, spilling any optimistically colored nodes below the
node that we failed to color on the stack wouldn't help us make
progress, since it wouldn't help with allowing us to find a color for
the node currently failing to get colored. Only consider nodes
which were above the failing node on the stack for spilling, which
simplifies the logic, and comment the code better so people know what's
going on here.

No shader-db changes with BRW_MAX_GRF reduced to 90 (or with the normal
number of GRF's).

Signed-off-by: Connor Abbott <connor.abbott@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/program/register_allocate.c