nir: initialize value in copy_prop_vars_block
authorTapani Pälli <tapani.palli@intel.com>
Mon, 25 Feb 2019 09:14:11 +0000 (11:14 +0200)
committerTapani Pälli <tapani.palli@intel.com>
Tue, 26 Feb 2019 06:56:25 +0000 (08:56 +0200)
Fixes following valgrind warning:

   ==27561== Conditional jump or move depends on uninitialised value(s)
   ==27561==    at 0x667856B: value_set_ssa_components (nir_opt_copy_prop_vars.c:78)
   ==27561==    by 0x667A1C4: copy_prop_vars_block (nir_opt_copy_prop_vars.c:797)

Fixes: 62332d139c8 "nir: Add a local variable-based copy propagation pass"
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
src/compiler/nir/nir_opt_copy_prop_vars.c

index 61f3dd8a9df89820903c7246fa74ac6e0638b8c0..4a900096b6c9812a7cfc16a90529a4deb6627a04 100644 (file)
@@ -763,7 +763,7 @@ copy_prop_vars_block(struct copy_prop_var_state *state,
 
          struct copy_entry *src_entry =
             lookup_entry_for_deref(copies, src, nir_derefs_a_contains_b_bit);
-         struct value value;
+         struct value value = {0};
          if (try_load_from_entry(state, src_entry, b, intrin, src, &value)) {
             if (value.is_ssa) {
                /* lookup_load has already ensured that we get a single SSA