On Wed, Dec 9, 2015 at 4:15 AM, Iago Toral Quiroga <[email protected]> wrote: > --- > src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c > b/src/mesa/drivers/dri/i965/brw_eu_emit.c > index f8c0f80..9543d5e 100644 > --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c > +++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c > @@ -1256,6 +1256,7 @@ brw_F16TO32(struct brw_codegen *p, struct brw_reg dst, > struct brw_reg src) > void brw_NOP(struct brw_codegen *p) > { > brw_inst *insn = next_insn(p, BRW_OPCODE_NOP); > + brw_inst_set_exec_size(p->devinfo, insn, BRW_EXECUTE_4);
I don't follow this change. Was this implicitly set before? At least in newer documentation, NOP is defined to have nearly all fields 0 which would mean execution size must be 1. _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
