diff options
author | Tom Stellard <thomas.stellard@amd.com> | 2013-10-22 18:19:10 +0000 |
---|---|---|
committer | Tom Stellard <thomas.stellard@amd.com> | 2013-10-22 18:19:10 +0000 |
commit | 04c559569f87d755c3f2828a765f5eb7308e6753 (patch) | |
tree | e9d7c6b9c64b995fcca4ed4e6362d93f84854db9 /lib/Target/R600/SIInstrInfo.cpp | |
parent | 34adeaf8b9e82e68d1dc8de916a307143ddd290c (diff) | |
download | llvm-04c559569f87d755c3f2828a765f5eb7308e6753.tar.gz llvm-04c559569f87d755c3f2828a765f5eb7308e6753.tar.bz2 llvm-04c559569f87d755c3f2828a765f5eb7308e6753.tar.xz |
R600: Simplify handling of private address space
The AMDGPUIndirectAddressing pass was previously responsible for
lowering private loads and stores to indirect addressing instructions.
However, this pass was buggy and way too complicated. The only
advantage it had over the new simplified code was that it saved one
instruction per direct write to private memory. This optimization
likely has a minimal impact on performance, and we may be able
to duplicate it using some other transformation.
For the private address space, we now:
1. Lower private loads/store to Register(Load|Store) instructions
2. Reserve part of the register file as 'private memory'
3. After regalloc lower the Register(Load|Store) instructions to
MOV instructions that use indirect addressing.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@193179 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/R600/SIInstrInfo.cpp')
-rw-r--r-- | lib/Target/R600/SIInstrInfo.cpp | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/lib/Target/R600/SIInstrInfo.cpp b/lib/Target/R600/SIInstrInfo.cpp index d3ad06ba9b..8b35d89088 100644 --- a/lib/Target/R600/SIInstrInfo.cpp +++ b/lib/Target/R600/SIInstrInfo.cpp @@ -197,6 +197,13 @@ MachineInstr *SIInstrInfo::commuteInstruction(MachineInstr *MI, return MI; } +MachineInstr *SIInstrInfo::buildMovInstr(MachineBasicBlock *MBB, + MachineBasicBlock::iterator I, + unsigned DstReg, + unsigned SrcReg) const { + assert(!"Not Implemented"); +} + bool SIInstrInfo::isMov(unsigned Opcode) const { switch(Opcode) { default: return false; @@ -346,12 +353,7 @@ int SIInstrInfo::getIndirectIndexEnd(const MachineFunction &MF) const { llvm_unreachable("Unimplemented"); } -const TargetRegisterClass *SIInstrInfo::getIndirectAddrStoreRegClass( - unsigned SourceReg) const { - llvm_unreachable("Unimplemented"); -} - -const TargetRegisterClass *SIInstrInfo::getIndirectAddrLoadRegClass() const { +const TargetRegisterClass *SIInstrInfo::getIndirectAddrRegClass() const { llvm_unreachable("Unimplemented"); } @@ -370,7 +372,3 @@ MachineInstrBuilder SIInstrInfo::buildIndirectRead( unsigned Address, unsigned OffsetReg) const { llvm_unreachable("Unimplemented"); } - -const TargetRegisterClass *SIInstrInfo::getSuperIndirectRegClass() const { - llvm_unreachable("Unimplemented"); -} |