nv50/ir: fix false global CSE on instructions with multiple defs
authorIlia Mirkin <imirkin@alum.mit.edu>
Sat, 30 Jan 2016 22:13:33 +0000 (17:13 -0500)
committerIlia Mirkin <imirkin@alum.mit.edu>
Sat, 30 Jan 2016 22:18:41 +0000 (17:18 -0500)
commit3ca941d60ed38800038cd545842e0ed3a69946da
tree7b1dc66c12e9473c932712369eb042f12a4b4d86
parent3ca2001b537a2709e7ef60410e7dfad5d38663f4
nv50/ir: fix false global CSE on instructions with multiple defs

If an instruction has multiple defs, we have to do a lot more checks to
make sure that we can move it forward. Among other things, various code
likes to do

    a, b = tex()
    if () c = a
    else c = b

which means that a single phi node will have results pointing at the
same instruction. We obviously can't propagate the tex in this case, but
properly accounting for this situation is tricky. Just don't try for
instructions with multiple defs.

This fixes about 20 shaders in shader-db, including the dolphin efb2ram
shader.

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Cc: mesa-stable@lists.freedesktop.org
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp