diff options
author | Rob Clark <[email protected]> | 2018-12-10 10:39:28 -0500 |
---|---|---|
committer | Rob Clark <[email protected]> | 2018-12-13 15:51:01 -0500 |
commit | c19c4bf48845be044605a97d0ea0c278bf2e1d6e (patch) | |
tree | 81ab38f0afb786cf4a106d5449f07269c492af87 /src/freedreno/ir3/ir3_depth.c | |
parent | 3e8e033f4c1f09d5c1b9373ba850fe4d46e5b048 (diff) |
freedreno/ir3: fix crash
Fixes a crash in dEQP-GLES3.functional.shaders.fragdepth.compare.fragcoord_z
Fixes: 0d240c22141 freedreno/ir3: don't fetch unused tex components
Signed-off-by: Rob Clark <[email protected]>
Diffstat (limited to 'src/freedreno/ir3/ir3_depth.c')
-rw-r--r-- | src/freedreno/ir3/ir3_depth.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/src/freedreno/ir3/ir3_depth.c b/src/freedreno/ir3/ir3_depth.c index 77f9af6173f..4e377fa4ee5 100644 --- a/src/freedreno/ir3/ir3_depth.c +++ b/src/freedreno/ir3/ir3_depth.c @@ -177,27 +177,21 @@ remove_unused_by_block(struct ir3_block *block) if (instr->opc == OPC_META_FO) { struct ir3_instruction *src = ssa(instr->regs[1]); if (src->regs[0]->wrmask > 1) { - unsigned newn, lastn; - - lastn = util_last_bit(src->regs[0]->wrmask); src->regs[0]->wrmask &= ~(1 << instr->fo.off); - newn = util_last_bit(src->regs[0]->wrmask); /* prune no-longer needed right-neighbors. We could * probably do the same for left-neighbors (ie. tex * fetch that only need .yw components), but that * makes RA a bit more confusing than it already is */ - if (newn < lastn) { - struct ir3_instruction *n = ir3_neighbor_first(instr); - for (unsigned i = 1; i < newn; i++) { - n = n->cp.right; - } - // XXX I don't think n should be null here! - if (n) { - debug_assert(n->cp.right->flags & IR3_INSTR_UNUSED); - n->cp.right = NULL; - } + struct ir3_instruction *n = instr; + while (n && n->cp.right) + n = n->cp.right; + while (n->flags & IR3_INSTR_UNUSED) { + n = n->cp.left; + if (!n) + break; + n->cp.right = NULL; } } } |