glsl: remove redundant es_shader checks
authorTimothy Arceri <tarceri@itsqueeze.com>
Thu, 11 Oct 2018 00:25:08 +0000 (11:25 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Thu, 11 Oct 2018 03:45:43 +0000 (14:45 +1100)
The es check is already covered by the is_version() check.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/compiler/glsl/ast_to_hir.cpp
src/compiler/glsl_types.cpp

index 1082d6c91cf7f2732c4a9670aa70d7cd4dd0ae1c..77fe0afef86c9198e8ad240bf21479cdf11077b3 100644 (file)
@@ -314,10 +314,6 @@ apply_implicit_conversion(const glsl_type *to, ir_rvalue * &from,
    if (!state->is_version(120, 0))
       return false;
 
-   /* ESSL does not allow implicit conversions */
-   if (state->es_shader)
-      return false;
-
    /* From page 27 (page 33 of the PDF) of the GLSL 1.50 spec:
     *
     *    "There are no implicit array or structure conversions. For
index ca5368aa53fde15af97451cac9c9a848ae898cd1..70bce6ace8e9304308e95ed00635ef78f6fa061c 100644 (file)
@@ -1425,7 +1425,7 @@ glsl_type::can_implicitly_convert_to(const glsl_type *desired,
     * state, we're doing intra-stage function linking where these checks have
     * already been done.
     */
-   if (state && (state->es_shader || !state->is_version(120, 0)))
+   if (state && !state->is_version(120, 0))
       return false;
 
    /* There is no conversion among matrix types. */