[Mesa-dev] [PATCH 0/5] add support for EXT_shader_implicit_conversions

Erik Faye-Lund erik.faye-lund at collabora.com
Fri Nov 2 16:06:11 UTC 2018


On Fri, 2018-11-02 at 15:40 +0000, Emil Velikov wrote:
> On Tue, 30 Oct 2018 at 17:11, Erik Faye-Lund
> <erik.faye-lund at collabora.com> wrote:
> > EXT_shader_implicit_conversions is a useful extension that adds
> > implicit
> > conversions to OpenGL ES 3.1. Since it's tested excensively in
> > dEQP, and
> > Mesa already has support for implicit conversions, it seems
> > reasonable to
> > allow for the extension. This ended up mostly as code-cleanups
> > anyway.
> > 
> > While enabling this, one bug was discorvered due to a failing dEQP
> > test
> > (see the last patch).
> > 
> > This makes 2068 dEQP-GLE31 tests go from NotSupported to Pass on
> > i965.
> > 
> > No piglit regressions observed.
> > 
> > Erik Faye-Lund (5):
> >   glsl: add has_implicit_conversions()-helper
> >   glsl: add has_implicit_uint_to_int_conversion()-helper
> >   glsl: fall back to inexact function-match
> >   mesa/glsl: add support for EXT_shader_implicit_conversions
> >   glsl: do not allow implicit casts of unsized array initializers
> > 
> Erik, I think we'd want the hole series minus the ES extension?
> 
> The issues fixed with the final patch affect GLSL 1.30-4.60 at least.
> Even if there's no desktop gl tests for it.
> 

Do you mean for the 18.3 release? If so, yes that might be good. The
last patch alone should also fix it, it shouldn't depend on the other
patches...



More information about the mesa-dev mailing list