r600g/sb: don't propagate dead values in GVN pass

In some cases we use value::gvn_source field to link values that
are known to be equal before gvn pass (e.g. results of DOT4 in different
slots of the same alu group), but then source value may become dead later
and this confuses further passes.

This patch resets value::gvn_source to NULL in the dce_cleanup pass
if it points to dead value.

Fixes segfault during shader optimization with ETQW.

Signed-off-by: Vadim Girlin <vadimgirlin@gmail.com>

diff --git a/src/gallium/drivers/r600/sb/sb_dce_cleanup.cpp b/src/gallium/drivers/r600/sb/sb_dce_cleanup.cpp
index acd6613166f645a13d9c8c32395fbd14f14574e0..f879395f67c8327ddeddcaadcc92500ca03449e4 100644 (file)
--- a/src/gallium/drivers/r600/sb/sb_dce_cleanup.cpp
+++ b/src/gallium/drivers/r600/sb/sb_dce_cleanup.cpp
@@ -125,6 +125,9 @@ void dce_cleanup::cleanup_dst_vec(vvec& vv) {
                if (!v)
                        continue;
 
+               if (v->gvn_source && v->gvn_source->is_dead())
+                       v->gvn_source = NULL;
+
                if (v->is_dead())
                        v = NULL;
        }