diff --git a/.pick_status.json b/.pick_status.json index 02cbf2843d6..ca41a05f8ee 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -607,7 +607,7 @@ "description": "r600: Split tex CF only if written component is read", "nominated": true, "nomination_type": 0, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": null }, diff --git a/src/gallium/drivers/r600/r600_asm.c b/src/gallium/drivers/r600/r600_asm.c index d7898e9b809..dcbb672a6ab 100644 --- a/src/gallium/drivers/r600/r600_asm.c +++ b/src/gallium/drivers/r600/r600_asm.c @@ -1539,12 +1539,21 @@ int r600_bytecode_add_tex(struct r600_bytecode *bc, const struct r600_bytecode_t if (bc->cf_last != NULL && bc->cf_last->op == CF_OP_TEX) { struct r600_bytecode_tex *ttex; + uint8_t use_mask = ((1 << ntex->src_sel_x) | + (1 << ntex->src_sel_y) | + (1 << ntex->src_sel_z) | + (1 << ntex->src_sel_w)) & 0xf; + LIST_FOR_EACH_ENTRY(ttex, &bc->cf_last->tex, list) { - if (ttex->dst_gpr == ntex->src_gpr && - (ttex->dst_sel_x < 4 || ttex->dst_sel_y < 4 || - ttex->dst_sel_z < 4 || ttex->dst_sel_w < 4)) { - bc->force_add_cf = 1; - break; + if (ttex->dst_gpr == ntex->src_gpr) { + uint8_t write_mask = (ttex->dst_sel_x < 6 ? 1 : 0) | + (ttex->dst_sel_y < 6 ? 2 : 0) | + (ttex->dst_sel_z < 6 ? 4 : 0) | + (ttex->dst_sel_w < 6 ? 8 : 0); + if (use_mask & write_mask) { + bc->force_add_cf = 1; + break; + } } } /* vtx instrs get inserted after tex, so make sure we aren't moving the tex