On Monday, January 19, 2015 03:31:06 PM Matt Turner wrote: > --- > src/mesa/drivers/dri/i965/brw_shader.cpp | 10 ++++++++++ > src/mesa/drivers/dri/i965/brw_shader.h | 1 + > src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 6 +----- > 3 files changed, 12 insertions(+), 5 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp > b/src/mesa/drivers/dri/i965/brw_shader.cpp > index cbdf976..c6fead7 100644 > --- a/src/mesa/drivers/dri/i965/brw_shader.cpp > +++ b/src/mesa/drivers/dri/i965/brw_shader.cpp > @@ -678,6 +678,16 @@ backend_reg::is_accumulator() const > } > > bool > +backend_instruction::is_3src() const > +{ > + return opcode == BRW_OPCODE_LRP || > + opcode == BRW_OPCODE_MAD || > + opcode == BRW_OPCODE_BFE || > + opcode == BRW_OPCODE_BFI2 || > + opcode == BRW_OPCODE_CSEL;
Pah, manual listings of things :) Let's do even better: return opcode < 128 && opcode_descs[op].nsrc == 3; That would get Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev