On Monday, June 01, 2015 11:35:03 AM Dave Airlie wrote: > From: Dave Airlie <[email protected]> > > I'm not sure if we shouldn't enable this everywhere > and rip out the API checks, > > discuss, > > Signed-off-by: Dave Airlie <[email protected]> > --- > src/mesa/state_tracker/st_extensions.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/mesa/state_tracker/st_extensions.c > b/src/mesa/state_tracker/st_extensions.c > index b1057f3..3b828fa 100644 > --- a/src/mesa/state_tracker/st_extensions.c > +++ b/src/mesa/state_tracker/st_extensions.c > @@ -598,6 +598,7 @@ void st_init_extensions(struct pipe_screen *screen, > extensions->ARB_half_float_vertex = GL_TRUE; > extensions->ARB_internalformat_query = GL_TRUE; > extensions->ARB_map_buffer_range = GL_TRUE; > + extensions->ARB_shader_subroutine = GL_TRUE; > extensions->ARB_texture_border_clamp = GL_TRUE; /* XXX temp */ > extensions->ARB_texture_cube_map = GL_TRUE; > extensions->ARB_texture_env_combine = GL_TRUE; >
I know people have talked about enabling it everywhere 1.30+ is available. That makes some sense. Here's an argument for keeping it optional: We don't want people using this extension, so there's no value - in fact, negative value - to exposing it unless we have to. Drivers that can't do 4.0, such as Sandybridge, could avoid advertising it if we kept it optional. I'm happy to defer to majority opinion.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
