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...
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