From b65bd9583b9fe0080b5664df2a80d2084499e3c2 Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Sun, 26 Oct 2014 10:08:40 -0700 Subject: [PATCH] i965/fs: Perform CSE on MAD instructions with final arguments switched. Multiplication is commutative. instructions in affected programs: 48314 -> 47954 (-0.75%) Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp index 801200118d7..5fdbf463984 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_cse.cpp @@ -128,7 +128,11 @@ operands_match(fs_inst *a, fs_inst *b) fs_reg *xs = a->src; fs_reg *ys = b->src; - if (!is_expression_commutative(a->opcode)) { + if (a->opcode == BRW_OPCODE_MAD) { + return xs[0].equals(ys[0]) && + ((xs[1].equals(ys[1]) && xs[2].equals(ys[2])) || + (xs[2].equals(ys[1]) && xs[1].equals(ys[2]))); + } else if (!is_expression_commutative(a->opcode)) { bool match = true; for (int i = 0; i < a->sources; i++) { if (!xs[i].equals(ys[i])) { -- 2.30.2