On 7 November 2017 at 11:54, Emil Velikov <emil.l.veli...@gmail.com> wrote: > From: Emil Velikov <emil.veli...@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.jus...@intel.com> > Signed-off-by: Emil Velikov <emil.veli...@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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
