[Mesa-dev] [PATCH 01/16] i965: Sets missing vertex shader constant values for HighInt format

Ian Romanick idr at freedesktop.org
Thu Dec 11 18:18:07 PST 2014


This patch is

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 12/11/2014 02:34 PM, Eduardo Lima Mitev wrote:
> The range's min and max, and the precision value are not set correctly for the
> vertex shader constants.
> 
> Fixes 1 dEQP test: dEQP-GLES3.functional.state_query.shader.precision_vertex_highp_int
> ---
>  src/mesa/drivers/dri/i965/brw_context.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
> index 860ee22d..03b4ee3 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.c
> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> @@ -480,6 +480,12 @@ brw_initialize_context_constants(struct brw_context *brw)
>     ctx->Const.Program[MESA_SHADER_FRAGMENT].HighInt = ctx->Const.Program[MESA_SHADER_FRAGMENT].LowInt;
>     ctx->Const.Program[MESA_SHADER_FRAGMENT].MediumInt = ctx->Const.Program[MESA_SHADER_FRAGMENT].LowInt;
>  
> +   ctx->Const.Program[MESA_SHADER_VERTEX].LowInt.RangeMin = 31;
> +   ctx->Const.Program[MESA_SHADER_VERTEX].LowInt.RangeMax = 30;
> +   ctx->Const.Program[MESA_SHADER_VERTEX].LowInt.Precision = 0;
> +   ctx->Const.Program[MESA_SHADER_VERTEX].HighInt = ctx->Const.Program[MESA_SHADER_VERTEX].LowInt;
> +   ctx->Const.Program[MESA_SHADER_VERTEX].MediumInt = ctx->Const.Program[MESA_SHADER_VERTEX].LowInt;
> +
>     if (brw->gen >= 7) {
>        ctx->Const.Program[MESA_SHADER_FRAGMENT].MaxAtomicCounters = MAX_ATOMIC_COUNTERS;
>        ctx->Const.Program[MESA_SHADER_VERTEX].MaxAtomicCounters = MAX_ATOMIC_COUNTERS;
> 



More information about the mesa-dev mailing list