Shutting up this warning has been (fairly low) on my to-do list for some
time now... thanks for beating me to it. :)

Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>

On 02/10/2018 08:33 AM, Eric Anholt wrote:
> They should probably get unit tests implemented, but this cleans up a
> bunch of warnings in my build for now.
> 
> Fixes: 59f458cd8703 ("glsl: Add 16-bit types")
> Cc: Eduardo Lima Mitev <el...@igalia.com>
> ---
>  src/compiler/glsl/tests/uniform_initializer_utils.cpp | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/src/compiler/glsl/tests/uniform_initializer_utils.cpp 
> b/src/compiler/glsl/tests/uniform_initializer_utils.cpp
> index c5502b632697..701c8ef4d091 100644
> --- a/src/compiler/glsl/tests/uniform_initializer_utils.cpp
> +++ b/src/compiler/glsl/tests/uniform_initializer_utils.cpp
> @@ -110,6 +110,9 @@ generate_data_element(void *mem_ctx, const glsl_type 
> *type,
>        case GLSL_TYPE_INTERFACE:
>        case GLSL_TYPE_SUBROUTINE:
>        case GLSL_TYPE_FUNCTION:
> +      case GLSL_TYPE_FLOAT16:
> +      case GLSL_TYPE_UINT16:
> +      case GLSL_TYPE_INT16:
>        ASSERT_TRUE(false);
>        break;
>        }
> @@ -150,6 +153,9 @@ generate_data_element(void *mem_ctx, const glsl_type 
> *type,
>        case GLSL_TYPE_INTERFACE:
>        case GLSL_TYPE_SUBROUTINE:
>        case GLSL_TYPE_FUNCTION:
> +      case GLSL_TYPE_FLOAT16:
> +      case GLSL_TYPE_UINT16:
> +      case GLSL_TYPE_INT16:
>        ASSERT_TRUE(false);
>        break;
>        }
> @@ -262,6 +268,9 @@ verify_data(gl_constant_value *storage, unsigned 
> storage_array_size,
>        case GLSL_TYPE_INTERFACE:
>        case GLSL_TYPE_SUBROUTINE:
>           case GLSL_TYPE_FUNCTION:
> +         case GLSL_TYPE_FLOAT16:
> +         case GLSL_TYPE_UINT16:
> +         case GLSL_TYPE_INT16:
>           ASSERT_TRUE(false);
>           break;
>        }
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to