From: Eric Anholt Date: Wed, 26 Mar 2014 17:06:53 +0000 (-0700) Subject: i965/fs: Fix vgrf0 live interval when no interpolation was done. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=caa2605db52da7303c4571bddc424a613786f56f;p=mesa.git i965/fs: Fix vgrf0 live interval when no interpolation was done. When you've got a simple solid-color shader that doesn't generate pixel_x/y interpolation, we were deciding that the first vgrf was both the undefined pixel_x and pixel_y, and extending its live interval to avoid the stride problem. That tricked other optimization that tries to see if a particular instruction is the last use of a variable. Reviewed-by: Kenneth Graunke --- diff --git a/src/mesa/drivers/dri/i965/brw_fs_live_variables.cpp b/src/mesa/drivers/dri/i965/brw_fs_live_variables.cpp index 61517299247..c7b1f2513ab 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_live_variables.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_live_variables.cpp @@ -86,8 +86,10 @@ fs_live_variables::setup_one_read(bblock_t *block, fs_inst *inst, */ int end_ip = ip; if (v->dispatch_width == 16 && (reg.stride == 0 || - (v->pixel_x.reg == reg.reg || - v->pixel_y.reg == reg.reg))) { + ((v->pixel_x.file == GRF && + v->pixel_x.reg == reg.reg) || + (v->pixel_y.file == GRF && + v->pixel_y.reg == reg.reg)))) { end_ip++; }