Re: [Mesa-dev] [PATCH] i965: Consider SEL.{GE, L} to be commutative operations.

2015-01-08 Thread Kenneth Graunke
On Thursday, January 08, 2015 10:53:22 AM Matt Turner wrote: > --- > v2: Made some small changes to the surrounding vec4 code to better match > the fs code. > > No shader-db changes. > > src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 14 ++ > src/mesa/drivers/dri/i965/brw_vec4_cse.

[Mesa-dev] [PATCH] i965: Consider SEL.{GE, L} to be commutative operations.

2015-01-08 Thread Matt Turner
--- v2: Made some small changes to the surrounding vec4 code to better match the fs code. No shader-db changes. src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 14 ++ src/mesa/drivers/dri/i965/brw_vec4_cse.cpp | 21 +++-- 2 files changed, 25 insertions(+), 10 deletio

Re: [Mesa-dev] [PATCH] i965: Consider SEL.{GE, L} to be commutative operations.

2014-12-21 Thread Matt Turner
On Sun, Dec 21, 2014 at 7:50 PM, Kenneth Graunke wrote: > On Sunday, December 21, 2014 03:37:25 PM Matt Turner wrote: >> --- >> No shader-db changes, unfortunately. > > Unsurprising :) This patch doesn't implement what you meant. > > 1. instructions_match() checks a->conditional_mod == b->conditio

Re: [Mesa-dev] [PATCH] i965: Consider SEL.{GE, L} to be commutative operations.

2014-12-21 Thread Kenneth Graunke
On Sunday, December 21, 2014 03:37:25 PM Matt Turner wrote: > --- > No shader-db changes, unfortunately. Unsurprising :) This patch doesn't implement what you meant. 1. instructions_match() checks a->conditional_mod == b->conditional_mod. So you won't ever see mixed conditional mods in operand

[Mesa-dev] [PATCH] i965: Consider SEL.{GE, L} to be commutative operations.

2014-12-21 Thread Matt Turner
--- No shader-db changes, unfortunately. src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 14 ++ src/mesa/drivers/dri/i965/brw_vec4_cse.cpp | 18 -- 2 files changed, 22 insertions(+), 10 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_cse.cpp b/src/mesa/driver