diff options
author | Tom Stellard <thomas.stellard@amd.com> | 2013-11-22 00:41:08 +0000 |
---|---|---|
committer | Tom Stellard <thomas.stellard@amd.com> | 2013-11-22 00:41:08 +0000 |
commit | cd6b0a658a388e542d9eb1d2a7817a070dbd560e (patch) | |
tree | 1cb6709fc06e194d96c950c9dc9b2b7e4c15cf6a /llvm/lib | |
parent | 06c67bcbe4382fe315c2a524b20e4fff50dc8940 (diff) | |
download | bcm5719-llvm-cd6b0a658a388e542d9eb1d2a7817a070dbd560e.tar.gz bcm5719-llvm-cd6b0a658a388e542d9eb1d2a7817a070dbd560e.zip |
R600: Implement TargetInstrInfo::isLegalToSplitMBBAt()
Splitting a basic block will create a new ALU clause, so we need to make
sure we aren't moving uses of registers that are local to their
current clause into a new one.
I had a test case for this, but unfortunately unrelated schedule changes
invalidated it, and I wasn't been able to come up with another one.
NOTE: This is a candidate for the 3.4 branch.
llvm-svn: 195399
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Target/R600/R600InstrInfo.cpp | 12 | ||||
-rw-r--r-- | llvm/lib/Target/R600/R600InstrInfo.h | 2 |
2 files changed, 14 insertions, 0 deletions
diff --git a/llvm/lib/Target/R600/R600InstrInfo.cpp b/llvm/lib/Target/R600/R600InstrInfo.cpp index 1f4741634e1..c0827fc1ca4 100644 --- a/llvm/lib/Target/R600/R600InstrInfo.cpp +++ b/llvm/lib/Target/R600/R600InstrInfo.cpp @@ -77,6 +77,18 @@ R600InstrInfo::copyPhysReg(MachineBasicBlock &MBB, } } +/// \returns true if \p MBBI can be moved into a new basic. +bool R600InstrInfo::isLegalToSplitMBBAt(MachineBasicBlock &MBB, + MachineBasicBlock::iterator MBBI) const { + for (MachineInstr::const_mop_iterator I = MBBI->operands_begin(), + E = MBBI->operands_end(); I != E; ++I) { + if (I->isReg() && !TargetRegisterInfo::isVirtualRegister(I->getReg()) && + I->isUse() && RI.isPhysRegLiveAcrossClauses(I->getReg())) + return false; + } + return true; +} + unsigned R600InstrInfo::getIEQOpcode() const { return AMDGPU::SETE_INT; } diff --git a/llvm/lib/Target/R600/R600InstrInfo.h b/llvm/lib/Target/R600/R600InstrInfo.h index b29b91f8d7e..13d981094ed 100644 --- a/llvm/lib/Target/R600/R600InstrInfo.h +++ b/llvm/lib/Target/R600/R600InstrInfo.h @@ -55,6 +55,8 @@ namespace llvm { MachineBasicBlock::iterator MI, DebugLoc DL, unsigned DestReg, unsigned SrcReg, bool KillSrc) const; + bool isLegalToSplitMBBAt(MachineBasicBlock &MBB, + MachineBasicBlock::iterator MBBI) const; bool isTrig(const MachineInstr &MI) const; bool isPlaceHolderOpcode(unsigned opcode) const; |