diff options
| author | Ilia Mirkin <[email protected]> | 2018-11-11 02:19:36 -0500 |
|---|---|---|
| committer | Ilia Mirkin <[email protected]> | 2018-11-16 20:53:09 -0500 |
| commit | 955d943c33d55ebb469efaeb5710d42e267b121a (patch) | |
| tree | 2b6a7c9a9b29565cd0d16239ab177e88a1a7c489 /src | |
| parent | 7e9fc11ff8ee47ad25d96f618f5d1a1c26590083 (diff) | |
nv50/ir: delete MINMAX instruction that is no longer in the BB
We removed the op from the BB, but it was still listed in its sources'
uses. This could trip up some logic down the line which analyzes all the
uses of an l-value, e.g. spilling.
Signed-off-by: Ilia Mirkin <[email protected]>
Reviewed-by: Karol Herbst <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp index 8767e5efb99..04d26dcbf53 100644 --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp @@ -1898,7 +1898,7 @@ AlgebraicOpt::handleMINMAX(Instruction *minmax) if (minmax->src(0).mod == minmax->src(1).mod) { if (minmax->def(0).mayReplace(minmax->src(0))) { minmax->def(0).replace(minmax->src(0), false); - minmax->bb->remove(minmax); + delete_Instruction(prog, minmax); } else { minmax->op = OP_CVT; minmax->setSrc(1, NULL); |