[Mesa-dev] [PATCH 11/11] i965: Delete redundant extension enables

[Kenneth Graunke]

On Tuesday, June 14, 2016 7:01:59 PM PDT Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
> 
> A nearly identical block already exists in the gen >= 6 block above.
> 
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  src/mesa/drivers/dri/i965/intel_extensions.c | 9 ---------
>  1 file changed, 9 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c b/src/mesa/drivers/dri/i965/intel_extensions.c
> index 5be4787..b55fed2 100644
> --- a/src/mesa/drivers/dri/i965/intel_extensions.c
> +++ b/src/mesa/drivers/dri/i965/intel_extensions.c
> @@ -360,15 +360,6 @@ intelInitExtensions(struct gl_context *ctx)
>           if (brw->intelScreen->cmd_parser_version >= 2)
>              brw->predicate.supported = true;
>        }
> -
> -      /* Only enable this in core profile because other parts of Mesa behave
> -       * slightly differently when the extension is enabled.
> -       */
> -      if (ctx->API == API_OPENGL_CORE) {
> -         ctx->Extensions.ARB_viewport_array = true;
> -         ctx->Extensions.AMD_vertex_shader_viewport_index = true;
> -         ctx->Extensions.ARB_shader_subroutine = true;
> -      }
>     }
>  
>     if (brw->gen >= 8 || brw->is_haswell || brw->is_baytrail) {
> 

I actually reviewed this a month ago :)  Patch 11 is still:

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160615/f9f2a91c/attachment.sig>