diff options
| author | Vincent Lejeune <vljn@ovi.com> | 2013-07-09 15:03:33 +0000 |
|---|---|---|
| committer | Vincent Lejeune <vljn@ovi.com> | 2013-07-09 15:03:33 +0000 |
| commit | f2cfef8172fd2eceb036b8caff50623a189ba2ff (patch) | |
| tree | 73a7c07a0a19f3084b05c7e2241302be420d59f0 /lib/Target/R600/R600ControlFlowFinalizer.cpp | |
| parent | f4bdec2ebeb1306a77e9377583c5799199775f88 (diff) | |
| download | llvm-f2cfef8172fd2eceb036b8caff50623a189ba2ff.tar.gz llvm-f2cfef8172fd2eceb036b8caff50623a189ba2ff.tar.bz2 llvm-f2cfef8172fd2eceb036b8caff50623a189ba2ff.tar.xz | |
R600: Do not predicated basic block with multiple alu clause
Test is not included as it is several 1000 lines long.
To test this functionnality, a test case must generate at least 2 ALU clauses,
where an ALU clause is ~110 instructions long.
NOTE: This is a candidate for the stable branch.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@185943 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/R600/R600ControlFlowFinalizer.cpp')
| -rw-r--r-- | lib/Target/R600/R600ControlFlowFinalizer.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/Target/R600/R600ControlFlowFinalizer.cpp b/lib/Target/R600/R600ControlFlowFinalizer.cpp index 887c808d5b..932a6a7243 100644 --- a/lib/Target/R600/R600ControlFlowFinalizer.cpp +++ b/lib/Target/R600/R600ControlFlowFinalizer.cpp @@ -256,6 +256,7 @@ private: ClauseContent.push_back(MILit); } } + assert(ClauseContent.size() < 128 && "ALU clause is too big"); ClauseHead->getOperand(7).setImm(ClauseContent.size() - 1); return ClauseFile(ClauseHead, ClauseContent); } @@ -276,6 +277,7 @@ private: void EmitALUClause(MachineBasicBlock::iterator InsertPos, ClauseFile &Clause, unsigned &CfCount) { + Clause.first->getOperand(0).setImm(0); CounterPropagateAddr(Clause.first, CfCount); MachineBasicBlock *BB = Clause.first->getParent(); BuildMI(BB, InsertPos->getDebugLoc(), TII->get(AMDGPU::ALU_CLAUSE)) |