diff options
authorEric Anholt <>2013-02-20 18:00:47 -0800
committerIan Romanick <>2013-03-05 14:57:09 -0800
commita11201fd2fea07401b9d88c551bd3dfe661b1554 (patch)
parentba4f4cead47449fd855cbcdba5738bc14791b23d (diff)
i965/fs: Fix broken math on values loaded from uniform buffers on gen6.
In a debug build this led to assertion failures, but on a non-debug build the hardware would just reference the whole vec8 instead of the same channel 8 times. Fixes the new piglit glsl-1.40/uniform-buffer/fs-exp2. Bugzilla: Note: This is a candidate for the stable branches Reviewed-by: Kenneth Graunke <> (cherry picked from commit 7b0731d940c758ca9c1e883cdea454d8787255c1)
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 200307e3d9b..f80219e3d37 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -1929,6 +1929,7 @@ fs_visitor::register_coalesce()
bool has_source_modifiers = (inst->src[0].abs ||
inst->src[0].negate ||
+ inst->src[0].smear != -1 ||
inst->src[0].file == UNIFORM);
/* Found a move of a GRF to a GRF. Let's see if we can coalesce