intel/vec4: fix valgrind errors with vf_values array
authorTapani Pälli <tapani.palli@intel.com>
Tue, 4 Feb 2020 12:58:17 +0000 (14:58 +0200)
committerMarge Bot <eric+marge@anholt.net>
Fri, 7 Feb 2020 09:06:18 +0000 (09:06 +0000)
Fixes valgrind errors introduced since commit a8ec4082.

Closes: https://gitlab.freedesktop.org/mesa/mesa/issues/2346
Fixes: a8ec4082 ("nir+vtn: vec8+vec16 support")
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3691>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3691>

src/intel/compiler/brw_vec4_nir.cpp

index d32f65a9dbed3fd7b922f640b2493e9628d0f900..31219b4b584d2249d1667a0ee1c32bf5132cd379 100644 (file)
@@ -1030,7 +1030,8 @@ try_immediate_source(const nir_alu_instr *instr, src_reg *op,
       } else {
          uint8_t vf_values[4] = { 0, 0, 0, 0 };
 
-         for (unsigned i = 0; i < NIR_MAX_VEC_COMPONENTS; i++) {
+         for (unsigned i = 0; i < ARRAY_SIZE(vf_values); i++) {
+
             if (op[idx].abs)
                f[i] = fabs(f[i]);