[Mesa-dev] [PATCH] st/mesa: add missing GLSL_TYPE_[U]INT8 cases in st_glsl_type_dword_size()

Mathias Fröhlich Mathias.Froehlich at gmx.net
Fri Mar 30 14:54:42 UTC 2018


Hi Brian,

looks plausible to me.
Reviewed-by: Mathias Fröhlich <mathias.froehlich at web.de>

best
Mathias

On Thursday, 29 March 2018 03:40:29 CEST Brian Paul wrote:
> Silences a compiler warning about unhandled enum switch cases.
> ---
>  src/mesa/state_tracker/st_glsl_types.cpp | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/mesa/state_tracker/st_glsl_types.cpp b/src/mesa/
state_tracker/st_glsl_types.cpp
> index ef7b7fa..9ad76c9 100644
> --- a/src/mesa/state_tracker/st_glsl_types.cpp
> +++ b/src/mesa/state_tracker/st_glsl_types.cpp
> @@ -124,6 +124,9 @@ st_glsl_type_dword_size(const struct glsl_type *type)
>     case GLSL_TYPE_INT16:
>     case GLSL_TYPE_FLOAT16:
>        return DIV_ROUND_UP(type->components(), 2);
> +   case GLSL_TYPE_UINT8:
> +   case GLSL_TYPE_INT8:
> +      return DIV_ROUND_UP(type->components(), 4);
>     case GLSL_TYPE_DOUBLE:
>     case GLSL_TYPE_UINT64:
>     case GLSL_TYPE_INT64:
> 






More information about the mesa-dev mailing list