summaryrefslogtreecommitdiffstats
path: root/src/compiler/nir/nir_control_flow.c
diff options
context:
space:
mode:
authorCaio Marcelo de Oliveira Filho <[email protected]>2018-06-15 13:59:45 -0700
committerCaio Marcelo de Oliveira Filho <[email protected]>2018-07-16 16:33:39 -0700
commitd7849fd1dafafa3489e0be389dabb09502dae26e (patch)
tree68fc9a1e525b4dd1dd3aeb0cf991771a9f87751a /src/compiler/nir/nir_control_flow.c
parent229836fb379eea6fb34c31410203082e822f3213 (diff)
glsl: don't let an 'if' then-branch kill const propagation for else-branch
When handling 'if' in constant propagation, if a certain variable was killed when processing the first branch of the 'if', then the second would get any propagation from previous nodes. This is similar to the change done for copy propagation code. x = 1; if (...) { z = x; // This would turn into z = 1. x = 22; // x gets killed. } else { w = x; // This would NOT turn into w = 1. } With the change, we let constant propagation happen independently in the two branches and only then apply the killed values for the subsequent code. The new code use a single hash table for keeping the kills of both branches (the branches only write to it), and it gets deleted after we use -- instead of waiting for mem_ctx to collect it. NIR deals well with constant propagation, so it already covered for the missing ones that this patch fixes. Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/compiler/nir/nir_control_flow.c')
0 files changed, 0 insertions, 0 deletions