i965/vs: Fix incorrect subscript when resetting copy propagation records.
authorKenneth Graunke <kenneth@whitecape.org>
Sat, 24 Dec 2011 03:57:08 +0000 (19:57 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 27 Dec 2011 22:33:37 +0000 (14:33 -0800)
commit443c8d1ab7ddad9392046e041e4e9a4fda7cd6e7
tree3e656dbcb2afafa113567175884195c256b84440
parent7420c9dab4aaf87e6b840410226c296c4668a48f
i965/vs: Fix incorrect subscript when resetting copy propagation records.

In this code, 'i' loops over the number of virtual GRFs, while 'j' loops
over the number of vector components (0 <= j <= 3).

It can't possibly be correct to see if bit 'i' is set in the destination
writemask, as it will have values much larger than 3.  Clearly this is
supposed to be 'j'.

Found by inspection.

Tested-by: Matt Turner <mattst88@gmail.com>
Tested-by: Christopher James Halse Rogers <christopher.halse.rogers@canonical.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp