From 4d143328dfe507f36cad821a1319919f8debf557 Mon Sep 17 00:00:00 2001 From: Vincent Lejeune Date: Fri, 7 Jun 2013 23:30:26 +0000 Subject: R600: Anti dep better handled in tex clause llvm-svn: 183592 --- llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp') diff --git a/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp b/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp index 6e21df8..ab29d60 100644 --- a/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp +++ b/llvm/lib/Target/R600/R600ControlFlowFinalizer.cpp @@ -110,7 +110,7 @@ private: } bool isCompatibleWithClause(const MachineInstr *MI, - std::set &DstRegs, std::set &SrcRegs) const { + std::set &DstRegs) const { unsigned DstMI, SrcMI; for (MachineInstr::const_mop_iterator I = MI->operands_begin(), E = MI->operands_end(); I != E; ++I) { @@ -136,9 +136,7 @@ private: &AMDGPU::R600_Reg128RegClass); } } - if ((DstRegs.find(SrcMI) == DstRegs.end()) && - (SrcRegs.find(DstMI) == SrcRegs.end())) { - SrcRegs.insert(SrcMI); + if ((DstRegs.find(SrcMI) == DstRegs.end())) { DstRegs.insert(DstMI); return true; } else @@ -152,7 +150,7 @@ private: std::vector ClauseContent; unsigned AluInstCount = 0; bool IsTex = TII->usesTextureCache(ClauseHead); - std::set DstRegs, SrcRegs; + std::set DstRegs; for (MachineBasicBlock::iterator E = MBB.end(); I != E; ++I) { if (IsTrivialInst(I)) continue; @@ -161,7 +159,7 @@ private: if ((IsTex && !TII->usesTextureCache(I)) || (!IsTex && !TII->usesVertexCache(I))) break; - if (!isCompatibleWithClause(I, DstRegs, SrcRegs)) + if (!isCompatibleWithClause(I, DstRegs)) break; AluInstCount ++; ClauseContent.push_back(I); -- cgit v1.1