nir: properly clear the entry sources in copy_prop_vars
When updating a copy entry source value from a "non-SSA" (the data
come from a copy instruction) to a "SSA" (the data or parts of it come
from SSA values), it was possible to hold invalid data in ssa[0]
depending on the writemask. Because the union, ssa[0] could contain a
pointer to a nir_deref_instr left-over from previous non-SSA usage.
Change code to clean up the array before use to avoid invalid data
around.
Fixes: 62332d139c "nir: Add a local variable-based copy propagation pass"
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
This commit is contained in:
@@ -348,6 +348,9 @@ store_to_entry(struct copy_prop_var_state *state, struct copy_entry *entry,
|
||||
const struct value *value, unsigned write_mask)
|
||||
{
|
||||
if (value->is_ssa) {
|
||||
/* Clear src if it was being used as non-SSA. */
|
||||
if (!entry->src.is_ssa)
|
||||
memset(entry->src.ssa, 0, sizeof(entry->src.ssa));
|
||||
entry->src.is_ssa = true;
|
||||
/* Only overwrite the written components */
|
||||
for (unsigned i = 0; i < 4; i++) {
|
||||
|
||||
Reference in New Issue
Block a user