[Mesa-dev] [PATCH 3/4] i965: use UnrollSamplerArrayDynamicIndexing for gen < 7

[Francisco Jerez]

Tapani Pälli <tapani.palli at intel.com> writes:

> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_context.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
> index 652d9a3..38c6f72 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.c
> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> @@ -578,6 +578,10 @@ brw_initialize_context_constants(struct brw_context *brw)
>  	 (i == MESA_SHADER_FRAGMENT);
>        ctx->Const.ShaderCompilerOptions[i].EmitNoIndirectUniform = false;
>        ctx->Const.ShaderCompilerOptions[i].LowerClipDistance = true;
> +
> +      /* !ARB_gpu_shader5 */
> +      if (brw->gen < 7)
> +         ctx->Const.ShaderCompilerOptions[i].UnrollSamplerArrayDynamicIndexing = true;

It would be nice to have this flag default to true on context init if
the back-end doesn't support ARB_gpu_shader5 while the rest of
ShaderCompilerOptions are initialized on context creation, otherwise we
should probably make a similar change in the i915 driver and the Mesa
state tracker at least.

>     }
>  
>     ctx->Const.ShaderCompilerOptions[MESA_SHADER_VERTEX].OptimizeForAOS = true;
> -- 
> 2.1.0
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150609/1371fb63/attachment.sig>