Kenneth Graunke <kenn...@whitecape.org> writes: > This is a straight port from gen7_wm_state.c; I haven't looked into > whether we can do both. > > v2: Actually do it right. > > Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> @@ -205,13 +201,39 @@ upload_ps_state(struct brw_context *brw) > else > dw6 |= GEN7_PS_POSOFFSET_NONE; > > - dw7 |= > - brw->wm.prog_data->first_curbe_grf << > GEN7_PS_DISPATCH_START_GRF_SHIFT_0 | > - brw->wm.prog_data->first_curbe_grf_16<< > GEN7_PS_DISPATCH_START_GRF_SHIFT_2; > + /* In case of non 1x per sample shading, only one of SIMD8 and SIMD16 > + * should be enabled. We do 'SIMD16 only' dispatch if a SIMD16 shader > + * is successfully compiled. In majority of the cases that bring us > + * better performance than 'SIMD8 only' dispatch. > + */ > + int min_invocations_per_fragment = > + _mesa_get_min_invocations_per_fragment(ctx, brw->fragment_program, > false); > + assert(min_invocations_per_fragment >= 1); Same _NEW_MULTISAMPLE comment.
pgpHduuj0GO1U.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev