diff options
| author | Eric Anholt <eric@anholt.net> | 2016-05-02 14:45:45 -0700 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2016-05-06 10:25:55 -0700 |
| commit | 1387e722cde7c33ad07766eb715bc74e84a43e80 (patch) | |
| tree | 49f1cac44530e4fd88aaf42af61600e67a7f772d | |
| parent | 44de03b0f8e14285d3a870934f805846b1d90f19 (diff) | |
vc4: Make sure that we don't overwrite the signal for PROG_END.
We should have already emitted a NOP due to the last instruction being a
TLB or VPM write. However, if you disable dead code elimination then you
might get dead code at the end, and that dead code might have the signal
bits set to something non-default, at which point you die in assertion
failure.
| -rw-r--r-- | src/gallium/drivers/vc4/vc4_qpu_emit.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/gallium/drivers/vc4/vc4_qpu_emit.c b/src/gallium/drivers/vc4/vc4_qpu_emit.c index 11640faa63..ecb70948ec 100644 --- a/src/gallium/drivers/vc4/vc4_qpu_emit.c +++ b/src/gallium/drivers/vc4/vc4_qpu_emit.c @@ -510,6 +510,14 @@ vc4_generate_code(struct vc4_context *vc4, struct vc4_compile *c) if (qpu_inst_is_tlb(c->qpu_insts[c->qpu_inst_count - 1])) qpu_serialize_one_inst(c, qpu_NOP()); + /* Make sure there's no existing signal set (like for a small + * immediate) + */ + if (QPU_GET_FIELD(c->qpu_insts[c->qpu_inst_count - 1], + QPU_SIG) != QPU_SIG_NONE) { + qpu_serialize_one_inst(c, qpu_NOP()); + } + c->qpu_insts[c->qpu_inst_count - 1] = qpu_set_sig(c->qpu_insts[c->qpu_inst_count - 1], QPU_SIG_PROG_END); |