On Thu, Aug 28, 2014 at 12:50 PM, Matt Turner <matts...@gmail.com> wrote:
> 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.
Ping?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
