i965/vec4: Ignore swizzle of VGRF for use by var_range_end().
authorMatt Turner <mattst88@gmail.com>
Thu, 18 Aug 2016 22:54:47 +0000 (15:54 -0700)
committerMatt Turner <mattst88@gmail.com>
Fri, 19 Aug 2016 23:52:25 +0000 (16:52 -0700)
commite7c376adfdecd4c1333997c8be8bb066a87c67b4
tree74c63969607d898899afadec9d05fed723bb96fe
parent3ef31122d08fdf7e8e6a8d74a9d91006fe840f86
i965/vec4: Ignore swizzle of VGRF for use by var_range_end().

var_range_end(v, n) loops over the n components of variable number v and
finds the maximum value, giving the last use of any component of v.
Therefore it expects v to correspond to the variable associated with the
.x channel of the VGRF.

var_from_reg() however returns the variable for the first channel of the
VGRF, post-swizzle.

So, if the last register had a swizzle with y, z, or w in the swizzle
component, we would read out of bounds. For any other register, we would
read liveness information from the next register.

The fix is to convert the src_reg to a dst_reg in order to call the
dst_reg version of var_from_reg() that doesn't consider the swizzle.

Cc: mesa-stable@lists.freedesktop.org
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_vec4.cpp
src/mesa/drivers/dri/i965/brw_vec4_cse.cpp