diff options
author | Gert Wollny <[email protected]> | 2020-04-12 16:51:07 +0200 |
---|---|---|
committer | Marge Bot <[email protected]> | 2020-04-21 15:10:43 +0000 |
commit | 9c7ce4d76e7b772e9d51dda2532a94d69bd4bee1 (patch) | |
tree | 5ac0f460d8b82ed1b7f14b5e1577531c279ece0d | |
parent | 67495ff9aa6bed9bce37a064b33ef561809fc35c (diff) |
r600/sfn: Fix using the result of a fetch instruction in next fetch
The result of a fetch instruction can't be used as source in the same CF
block, so force a new CF block when the result would be used in the same
vertex fetch block.
Signed-off-by: Gert Wollny <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4609>
-rw-r--r-- | src/gallium/drivers/r600/sfn/sfn_ir_to_assembly.cpp | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/src/gallium/drivers/r600/sfn/sfn_ir_to_assembly.cpp b/src/gallium/drivers/r600/sfn/sfn_ir_to_assembly.cpp index cc20d9fbe30..cf76d475679 100644 --- a/src/gallium/drivers/r600/sfn/sfn_ir_to_assembly.cpp +++ b/src/gallium/drivers/r600/sfn/sfn_ir_to_assembly.cpp @@ -88,6 +88,7 @@ public: PValue m_last_addr; int m_loop_nesting; int m_nliterals_in_group; + std::set<int> vtx_fetch_results; }; @@ -147,6 +148,9 @@ bool AssemblyFromShaderLegacy::do_lower(const std::vector<InstructionBlock>& ir) bool AssemblyFromShaderLegacyImpl::emit(const Instruction::Pointer i) { + if (i->type() != Instruction::vtx) + vtx_fetch_results.clear(); + sfn_log << SfnLog::assembly << "Emit from '" << *i << "\n"; switch (i->type()) { case Instruction::alu: @@ -740,6 +744,13 @@ bool AssemblyFromShaderLegacyImpl::emit_vtx(const FetchInstruction& fetch_instr) } } + if (vtx_fetch_results.find(fetch_instr.src().sel()) != + vtx_fetch_results.end()) { + m_bc->force_add_cf = 1; + vtx_fetch_results.clear(); + } + vtx_fetch_results.insert(fetch_instr.dst().sel()); + struct r600_bytecode_vtx vtx; memset(&vtx, 0, sizeof(vtx)); vtx.op = fetch_instr.vc_opcode(); |