diff options
| author | Jason Ekstrand <[email protected]> | 2019-05-22 22:13:15 -0500 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2019-05-31 01:08:03 +0000 |
| commit | 8948048c6f01209bac0051e41cd84c38853bd251 (patch) | |
| tree | 90f7a208ec100c988b7360ded9f0191e96364e07 /src/compiler | |
| parent | 5441d562433a6315ca00c0c69160ff848e5ec34a (diff) | |
nir/dead_cf: Call instructions aren't dead
When we inlined cf_node_has_side_effects into node_is_dead, all the
conditions flipped and we forgot to flip one. Fortunately, it doesn't
matter right now because no one uses this pass on shaders with more than
one function.
Fixes: b50465d197 "nir/dead_cf: Inline cf_node_has_side_effects"
Reviewed-by: Dave Airlie <[email protected]>
Reviewed-by: Caio Marcelo de Oliveira Filho <[email protected]>
Diffstat (limited to 'src/compiler')
| -rw-r--r-- | src/compiler/nir/nir_opt_dead_cf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/compiler/nir/nir_opt_dead_cf.c b/src/compiler/nir/nir_opt_dead_cf.c index b0e9723d36c..33f3565c564 100644 --- a/src/compiler/nir/nir_opt_dead_cf.c +++ b/src/compiler/nir/nir_opt_dead_cf.c @@ -216,7 +216,7 @@ node_is_dead(nir_cf_node *node) nir_foreach_instr(instr, block) { if (instr->type == nir_instr_type_call) - return true; + return false; /* Return instructions can cause us to skip over other side-effecting * instructions after the loop, so consider them to have side effects |