On Thursday, August 21, 2014 11:10:26 PM Matt Turner wrote: > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82846 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82929 > --- > Planning to commit this to master as well as to 10.3 directly, > since BDW is just broken without it. > > src/mesa/drivers/dri/i965/brw_eu_compact.c | 14 ++++++-------- > 1 file changed, 6 insertions(+), 8 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_eu_compact.c > b/src/mesa/drivers/dri/i965/brw_eu_compact.c > index 3680c55..4f509dd 100644 > --- a/src/mesa/drivers/dri/i965/brw_eu_compact.c > +++ b/src/mesa/drivers/dri/i965/brw_eu_compact.c > @@ -1029,19 +1029,17 @@ update_uip_jip(struct brw_context *brw, brw_inst > *insn, > { > int scale = brw->gen >= 8 ? sizeof(brw_compact_inst) : 1; > > - int32_t jip = brw_inst_jip(brw, insn); > - jip -= scale * > - compacted_between(this_old_ip, this_old_ip + jip, compacted_counts); > - brw_inst_set_jip(brw, insn, jip); > + int32_t jip = brw_inst_jip(brw, insn) / scale; > + jip -= compacted_between(this_old_ip, this_old_ip + jip, > compacted_counts); > + brw_inst_set_jip(brw, insn, jip * scale); > > if (brw_inst_opcode(brw, insn) == BRW_OPCODE_ENDIF || > brw_inst_opcode(brw, insn) == BRW_OPCODE_WHILE) > return; > > - int32_t uip = brw_inst_uip(brw, insn); > - uip -= scale * > - compacted_between(this_old_ip, this_old_ip + uip, compacted_counts); > - brw_inst_set_uip(brw, insn, uip); > + int32_t uip = brw_inst_uip(brw, insn) / scale; > + uip -= compacted_between(this_old_ip, this_old_ip + uip, > compacted_counts); > + brw_inst_set_uip(brw, insn, uip * scale); > } > > void >
Seems okay to me. 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
