From: Matt Turner Date: Sun, 29 Jun 2014 01:38:03 +0000 (-0700) Subject: i965/fs: Predicate PLN instructions used in unlit centroid WA. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=4fe53ee5d7c418d1ed51c5e8dfe5a2b1f48127a3;p=mesa.git i965/fs: Predicate PLN instructions used in unlit centroid WA. Maybe lets us skip some PLN instructions if whole subspans are disabled? Reviewed-by: Kristian Høgsberg --- diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index d1ded33a6b0..45b75cb2f70 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp @@ -1168,9 +1168,6 @@ fs_visitor::emit_general_interpolation(ir_variable *ir) /* Smooth/noperspective interpolation case. */ for (unsigned int k = 0; k < type->vector_elements; k++) { struct brw_reg interp = interp_reg(location, k); - emit_linterp(attr, fs_reg(interp), interpolation_mode, - ir->data.centroid && !key->persample_shading, - ir->data.sample || key->persample_shading); if (brw->needs_unlit_centroid_workaround && ir->data.centroid) { /* Get the pixel/sample mask into f0 so that we know * which pixels are lit. Then, for each channel that is @@ -1178,11 +1175,22 @@ fs_visitor::emit_general_interpolation(ir_variable *ir) * data. */ emit(FS_OPCODE_MOV_DISPATCH_TO_FLAGS); - fs_inst *inst = emit_linterp(attr, fs_reg(interp), - interpolation_mode, - false, false); + + fs_inst *inst; + inst = emit_linterp(attr, fs_reg(interp), interpolation_mode, + ir->data.centroid && !key->persample_shading, + ir->data.sample || key->persample_shading); + inst->predicate = BRW_PREDICATE_NORMAL; + inst->predicate_inverse = false; + + inst = emit_linterp(attr, fs_reg(interp), interpolation_mode, + false, false); inst->predicate = BRW_PREDICATE_NORMAL; inst->predicate_inverse = true; + } else { + emit_linterp(attr, fs_reg(interp), interpolation_mode, + ir->data.centroid && !key->persample_shading, + ir->data.sample || key->persample_shading); } if (brw->gen < 6 && interpolation_mode == INTERP_QUALIFIER_SMOOTH) { emit(BRW_OPCODE_MUL, attr, attr, this->pixel_w);