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.

Attachment: pgpHduuj0GO1U.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to