Doesn't this make patch 3 irrelevant? FWIW, I like this better. On 12/15/2015 04:08 PM, Jordan Justen wrote: > Previously we were checking the desktop OpenGL ARB_compute_shader > requirements, but for OpenGLES 3.1, the requirements are lower. > > Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Marta Lofstedt <marta.lofst...@intel.com> > --- > src/mesa/drivers/dri/i965/brw_context.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_context.c > b/src/mesa/drivers/dri/i965/brw_context.c > index 0abe601..5105625 100644 > --- a/src/mesa/drivers/dri/i965/brw_context.c > +++ b/src/mesa/drivers/dri/i965/brw_context.c > @@ -377,7 +377,10 @@ brw_initialize_context_constants(struct brw_context *brw) > [MESA_SHADER_GEOMETRY] = brw->gen >= 6, > [MESA_SHADER_FRAGMENT] = true, > [MESA_SHADER_COMPUTE] = > - (ctx->Const.MaxComputeWorkGroupSize[0] >= 1024) || > + (ctx->API == API_OPENGL_CORE && > + ctx->Const.MaxComputeWorkGroupSize[0] >= 1024) || > + (ctx->API == API_OPENGLES2 && > + ctx->Const.MaxComputeWorkGroupSize[0] >= 128) || > _mesa_extension_override_enables.ARB_compute_shader, > }; > >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev