diff options
author | Samuel Pitoiset <[email protected]> | 2016-06-28 00:59:46 +0200 |
---|---|---|
committer | Samuel Pitoiset <[email protected]> | 2016-06-28 18:38:56 +0200 |
commit | cc97b6a34a81e58c29bed6dae06613b388506283 (patch) | |
tree | 064701c47f504fbc200b57cc0fd0c18885dc1fe6 | |
parent | 234323558dd239a4648c6c25a001c061d663fd49 (diff) |
gm107/ir: make sure that flagsDef is set when emitting setcond
Rely on the existence of a second destination when emitting a setcond
flag is dangerous, because this doesn't mean that the flag has been
correctly set. Instead rely on flagsDef like what emitX() does
for flagsSrc.
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Ilia Mirkin <[email protected]>
Cc: <[email protected]>
-rw-r--r-- | src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp index 40afbce9c6f..2c5e8f69d13 100644 --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp @@ -420,7 +420,7 @@ CodeEmitterGM107::emitSAT(int pos) void CodeEmitterGM107::emitCC(int pos) { - emitField(pos, 1, insn->defExists(1)); + emitField(pos, 1, insn->flagsDef >= 0); } void |