From 3daa0fae4b39a271f50f473edbe44712b6c8f040 Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Wed, 6 Jul 2016 20:49:58 -0700 Subject: [PATCH] i965/fs: Don't CSE render target messages with different target index. We weren't checking the fs_inst::target field when comparing whether two instructions are equal. For FB writes it doesn't matter because they aren't CSE-able anyway, but this would have become a problem with FB reads which are expression-like instructions. Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp index befbf140a5e..0c769ddfb45 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp @@ -191,6 +191,7 @@ instructions_match(fs_inst *a, fs_inst *b, bool *negate) a->header_size == b->header_size && a->shadow_compare == b->shadow_compare && a->pi_noperspective == b->pi_noperspective && + a->target == b->target && a->sources == b->sources && operands_match(a, b, negate); } -- 2.30.2