Tapani Pälli <tapani.pa...@intel.com> writes:

> Signed-off-by: Tapani Pälli <tapani.pa...@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

Attachment: signature.asc
Description: PGP signature

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

Reply via email to