nir: Don't forget if-uses in new nir_opt_dead_cf liveness check
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 20 Feb 2019 07:17:28 +0000 (23:17 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 20 Feb 2019 17:44:06 +0000 (09:44 -0800)
commitd6337b59f66eddedcba4853042007507c6d5171d
tree92cc4e96e5a25854fa4fe6a83a06079cb7350a24
parentb9eed05e7f9717ce36ce9e080f746f2d1684f17e
nir: Don't forget if-uses in new nir_opt_dead_cf liveness check

Commit 08bfd710a25c14df5f690cce9604617536d7c560. (nir/dead_cf: Stop
relying on liveness analysis) introduced a new check that iterated
through a SSA def's uses, to see if it's used.  But it only checked
normal uses, and not uses which are part of an 'if' condition.  This
led to it thinking more nodes were dead than possible.

Fixes Piglit's variable-indexing/tcs-output-array-float-index-wr test
(and related tests) with the out-of-tree Iris driver.

Fixes: 08bfd710a25 nir/dead_cf: Stop relying on liveness analysis
Reviewed-by: Connor Abbott <cwabbott0@gmail.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
src/compiler/nir/nir_opt_dead_cf.c