On 08/17/2015 09:45 AM, Francisco Jerez wrote: > There is no GL_R8 image format in GLES, according to the state table > 20.32 of the GLES 3.1 spec the default value should be GL_R32UI.
Was any conformance test affected by this change? > --- > src/mesa/main/shaderimage.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/main/shaderimage.c b/src/mesa/main/shaderimage.c > index 48e3e09..5a3c47a 100644 > --- a/src/mesa/main/shaderimage.c > +++ b/src/mesa/main/shaderimage.c > @@ -397,10 +397,11 @@ is_image_format_supported(const struct gl_context *ctx, > GLenum format) > struct gl_image_unit > _mesa_default_image_unit(struct gl_context *ctx) > { > + const GLenum format = _mesa_is_desktop_gl(ctx) ? GL_R8 : GL_R32UI; > const struct gl_image_unit u = { > .Access = GL_READ_ONLY, > - .Format = GL_R8, > - ._ActualFormat = _mesa_get_shader_image_format(GL_R8) > + .Format = format, > + ._ActualFormat = _mesa_get_shader_image_format(format) > }; > return u; > } > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev