[Mesa-dev] [PATCH 02/13] i965: remove ARB_compute_shader extension override

[Emil Velikov]

On 7 November 2017 at 11:54, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
>
> Checking the override was useful in the early stages of developing the
> extension.
>
> Now that everything is wired, where possible, we can drop the check.
> Doing so allows us to simplify some of the related code.
>
> Cc: Jordan Justen <jordan.l.justen at intel.com>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
>  src/mesa/drivers/dri/i965/brw_context.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
> index 1bf39b07382..3947a71a46a 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.c
> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> @@ -348,8 +348,7 @@ brw_initialize_context_constants(struct brw_context *brw)
>           (_mesa_is_desktop_gl(ctx) &&
>            ctx->Const.MaxComputeWorkGroupSize[0] >= 1024) ||
>           (ctx->API == API_OPENGLES2 &&
> -          ctx->Const.MaxComputeWorkGroupSize[0] >= 128) ||
> -         _mesa_extension_override_enables.ARB_compute_shader,
> +          ctx->Const.MaxComputeWorkGroupSize[0] >= 128),

Jordan can you throw an Ack on this patch if it makes sense?
Brian already reviewed the series, but I'd appreciate input from
someone familiar with the Intel specifics.

Thanks
Emil