From 0d81a684c125626ad0ed5db97fa75ed3336d5396 Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Tue, 24 May 2016 15:10:25 -0700 Subject: [PATCH] i965: Add missing types to type_sz(). Coverity warns in multiple places about the potential for division by zero, caused by this function's default case. Reviewed-by: Francisco Jerez Reviewed-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_reg.h | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_reg.h b/src/mesa/drivers/dri/i965/brw_reg.h index bd7ea1ba7bc..38cf8e31371 100644 --- a/src/mesa/drivers/dri/i965/brw_reg.h +++ b/src/mesa/drivers/dri/i965/brw_reg.h @@ -292,15 +292,19 @@ type_sz(unsigned type) case BRW_REGISTER_TYPE_UD: case BRW_REGISTER_TYPE_D: case BRW_REGISTER_TYPE_F: + case BRW_REGISTER_TYPE_VF: return 4; case BRW_REGISTER_TYPE_UW: case BRW_REGISTER_TYPE_W: + case BRW_REGISTER_TYPE_UV: + case BRW_REGISTER_TYPE_V: + case BRW_REGISTER_TYPE_HF: return 2; case BRW_REGISTER_TYPE_UB: case BRW_REGISTER_TYPE_B: return 1; default: - return 0; + unreachable("not reached"); } } -- 2.30.2