On Monday, May 16, 2016 12:24:23 PM PDT Ian Romanick wrote:
> From: Ian Romanick <ian.d.roman...@intel.com>
> 
> A nearly identical block already exists in the gen >= 6 block above.
> 
> Signed-off-by: Ian Romanick <ian.d.roman...@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 362a7e5..a6a080f 100644
> --- a/src/mesa/drivers/dri/i965/intel_extensions.c
> +++ b/src/mesa/drivers/dri/i965/intel_extensions.c
> @@ -358,15 +358,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->intelScreen->has_mi_math_and_lrr) {
> 

Weird.  You're right though, we're setting them on Gen6+ and setting
them a second time on Gen7+.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

You and Tim Rowley are probably going to have merge conflicts on GL3.txt...

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to