[Mesa-dev] [PATCH 09/13] i965: Only use the SIMD16 program for per-sample shading on Broadwell.

[Eric Anholt]

Kenneth Graunke <kenneth at 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 <kenneth at 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140219/9a9e92c9/attachment.pgp>