[Mesa-dev] [PATCH 0/5] add support for EXT_shader_implicit_conversions
Tapani Pälli
tapani.palli at intel.com
Wed Oct 31 13:50:42 UTC 2018
LGTM, series is:
Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
On 10/30/18 7:11 PM, Erik Faye-Lund 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
>
> src/compiler/glsl/ast_function.cpp | 2 +-
> src/compiler/glsl/ast_to_hir.cpp | 8 ++++----
> src/compiler/glsl/glsl_parser_extras.cpp | 1 +
> src/compiler/glsl/glsl_parser_extras.h | 15 +++++++++++++++
> src/compiler/glsl/ir_function.cpp | 3 ++-
> src/compiler/glsl_types.cpp | 5 ++---
> src/mesa/main/extensions_table.h | 1 +
> 7 files changed, 26 insertions(+), 9 deletions(-)
>
More information about the mesa-dev
mailing list