Hi Brian, looks plausible to me. Reviewed-by: Mathias Fröhlich <mathias.froehl...@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: > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev