OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

glsl: Optimize redundant pack(unpack()) and unpack(pack()) combinations

Browse Source 
The lowering passes 64-bit integer operations will generate a lot of
these.

v2: Modify the HANDLE_PACK_UNPACK_INVERSE so that the breaks apply to
the switch instead of the 'do { } while(true)' loop.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
This commit is contained in:
Ian Romanick
2016-10-12 15:31:22 -07:00
parent 7122d851aa
commit 81952814a3
1 changed files with 28 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
src/compiler/glsl/opt_algebraic.cpp
View File

@@ -472,6 +472,34 @@ ir_algebraic_visitor::handle_expression(ir_expression *ir)
}
break;
/* This macro CANNOT use the do { } while(true) mechanism because
* then the breaks apply to the loop instead of the switch!
*/
#define HANDLE_PACK_UNPACK_INVERSE(inverse_operation) \
{ \
ir_expression *const op = ir->operands[0]->as_expression(); \
if (op == NULL) \
break; \
if (op->operation == (inverse_operation)) \
return op->operands[0]; \
break; \
}
case ir_unop_unpack_uint_2x32:
HANDLE_PACK_UNPACK_INVERSE(ir_unop_pack_uint_2x32);
case ir_unop_pack_uint_2x32:
HANDLE_PACK_UNPACK_INVERSE(ir_unop_unpack_uint_2x32);
case ir_unop_unpack_int_2x32:
HANDLE_PACK_UNPACK_INVERSE(ir_unop_pack_int_2x32);
case ir_unop_pack_int_2x32:
HANDLE_PACK_UNPACK_INVERSE(ir_unop_unpack_int_2x32);
case ir_unop_unpack_double_2x32:
HANDLE_PACK_UNPACK_INVERSE(ir_unop_pack_double_2x32);
case ir_unop_pack_double_2x32:
HANDLE_PACK_UNPACK_INVERSE(ir_unop_unpack_double_2x32);
#undef HANDLE_PACK_UNPACK_INVERSE
case ir_binop_add:
if (is_vec_zero(op_const[0]))
return ir->operands[1];