[Mesa-dev] [PATCH v2 2/3] i965: De-duplicate is_expression_commutative() functions.

Pohjolainen, Topi topi.pohjolainen at intel.com
Sun Mar 15 00:40:04 PDT 2015


On Fri, Mar 13, 2015 at 04:23:40PM -0700, Kenneth Graunke wrote:
> Create a backend_inst::is_commutative() method to replace two static
> functions that did the exact same thing.
> 
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  src/mesa/drivers/dri/i965/brw_fs_cse.cpp   | 24 +-----------------------
>  src/mesa/drivers/dri/i965/brw_shader.cpp   | 22 ++++++++++++++++++++++
>  src/mesa/drivers/dri/i965/brw_shader.h     |  1 +
>  src/mesa/drivers/dri/i965/brw_vec4_cse.cpp | 24 +-----------------------
>  4 files changed, 25 insertions(+), 46 deletions(-)

This is clear improvement on its own:

Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>


More information about the mesa-dev mailing list