On Tue, Aug 12, 2014 at 9:48 AM, Kenneth Graunke <kenn...@whitecape.org> wrote: > diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > index d3509a0..19f7ef8 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > +++ b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp > @@ -186,7 +186,7 @@ fs_generator::generate_fb_write(fs_inst *inst) > brw_imm_ud(0x1 << 11)); > } > > - if (inst->target > 0) { > + if (fp && inst->target > 0) { > /* Set the render target index for choosing BLEND_STATE. */ > brw_MOV(p, retype(brw_vec1_reg(BRW_MESSAGE_REGISTER_FILE, > inst->base_mrf, 2),
I don't understand this change. I don't see fp being used elsewhere in this patch. Patch 2 is Reviewed-by: Matt Turner <matts...@gmail.com> and so is this one once this hunk is sorted out. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev