[Mesa-dev] [PATCH 1/3] mesa: Add _mesa_has_compute_shaders

[Ben Widawsky]

On Tue, Feb 17, 2015 at 11:37:44AM -0800, Ian Romanick wrote:
> On 02/13/2015 11:02 PM, Ben Widawsky wrote:
> > From: Jordan Justen <jordan.l.justen at intel.com>
> > 
> > Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> > Reviewed-by: Ben Widawsky <ben at bwidawsk.net>
> > ---
> >  src/mesa/main/context.h | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> > 
> > diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h
> > index d902ea7..69389ae 100644
> > --- a/src/mesa/main/context.h
> > +++ b/src/mesa/main/context.h
> > @@ -326,6 +326,17 @@ _mesa_has_geometry_shaders(const struct gl_context *ctx)
> >  }
> >  
> >  
> > +/**
> > + * Checks if the context supports compute shaders.
> > + */
> > +static inline GLboolean
> 
> bool
> 

I don't mind changing this for Jordan before I push, but bool does seem
inconsistent with the existing _mesa_has_foo. So long as you're aware...

> > +_mesa_has_compute_shaders(const struct gl_context *ctx)
> > +{
> > +   return ctx->Extensions.ARB_compute_shader ||
> > +      (ctx->API == API_OPENGLES2 && ctx->Version >= 31);
> > +}
> > +
> > +
> >  #ifdef __cplusplus
> >  }
> >  #endif
> > 
> 

-- 
Ben Widawsky, Intel Open Source Technology Center