diff options
author | Eric Anholt <[email protected]> | 2018-11-06 09:39:40 -0800 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2018-11-15 11:12:54 -0800 |
commit | 538bca78e267b1c8cce41d5b4623fafd1e253d82 (patch) | |
tree | 0b994d48c462d98e046cd04734f2ebf3039a5479 | |
parent | 03928dd682d0a86bfc5a780614f376163c72d72b (diff) |
v3d: Don't try to set PF flags on a LDTMU operation
We need an ALU op in order to set PF. Fixes a recent assertion failure in
dEQP-GLES3.functional.ubo.single_basic_type.shared.bool_vertex
-rw-r--r-- | src/broadcom/compiler/vir.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/broadcom/compiler/vir.c b/src/broadcom/compiler/vir.c index 3f5a28d1be0..9a671c6c97f 100644 --- a/src/broadcom/compiler/vir.c +++ b/src/broadcom/compiler/vir.c @@ -1006,6 +1006,12 @@ vir_can_set_flags(struct v3d_compile *c, struct qinst *inst) return false; } + if (inst->qpu.type != V3D_QPU_INSTR_TYPE_ALU || + (inst->qpu.alu.add.op == V3D_QPU_A_NOP && + inst->qpu.alu.mul.op == V3D_QPU_M_NOP)) { + return false; + } + return true; } |