diff options
author | Evan Cheng <evan.cheng@apple.com> | 2008-06-24 07:10:51 +0000 |
---|---|---|
committer | Evan Cheng <evan.cheng@apple.com> | 2008-06-24 07:10:51 +0000 |
commit | 9ef4ca2e812fc62e345fa019c2358564bbe46245 (patch) | |
tree | 3a87cc23a714aceb10b06cf3fc05c9c8c03f775b /lib/Target/X86/X86InstrInfo.cpp | |
parent | 46e803b3e649c20ee3141341b2e58d5d64489e33 (diff) | |
download | llvm-9ef4ca2e812fc62e345fa019c2358564bbe46245.tar.gz llvm-9ef4ca2e812fc62e345fa019c2358564bbe46245.tar.bz2 llvm-9ef4ca2e812fc62e345fa019c2358564bbe46245.tar.xz |
If it's determined safe, remat MOV32r0 (i.e. xor r, r) and others as it is instead of using the longer MOV32ri instruction.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@52670 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86/X86InstrInfo.cpp')
-rw-r--r-- | lib/Target/X86/X86InstrInfo.cpp | 64 |
1 files changed, 53 insertions, 11 deletions
diff --git a/lib/Target/X86/X86InstrInfo.cpp b/lib/Target/X86/X86InstrInfo.cpp index bbe5bd363e..bfc8abbfa1 100644 --- a/lib/Target/X86/X86InstrInfo.cpp +++ b/lib/Target/X86/X86InstrInfo.cpp @@ -832,6 +832,40 @@ X86InstrInfo::isReallyTriviallyReMaterializable(const MachineInstr *MI) const { return true; } +/// isSafeToClobberEFLAGS - Return true if it's safe insert an instruction that +/// would clobber the EFLAGS condition register. Note the result may be +/// conservative. If it cannot definitely determine the safety after visiting +/// two instructions it assumes it's not safe. +static bool isSafeToClobberEFLAGS(MachineBasicBlock &MBB, + MachineBasicBlock::iterator I) { + // For compile time consideration, if we are not able to determine the + // safety after visiting 2 instructions, we will assume it's not safe. + for (unsigned i = 0; i < 2; ++i) { + if (I == MBB.end()) + // Reached end of block, it's safe. + return true; + bool SeenDef = false; + for (unsigned j = 0, e = I->getNumOperands(); j != e; ++j) { + MachineOperand &MO = I->getOperand(j); + if (!MO.isRegister()) + continue; + if (MO.getReg() == X86::EFLAGS) { + if (MO.isUse()) + return false; + SeenDef = true; + } + } + + if (SeenDef) + // This instruction defines EFLAGS, no need to look any further. + return true; + ++I; + } + + // Conservative answer. + return false; +} + void X86InstrInfo::reMaterialize(MachineBasicBlock &MBB, MachineBasicBlock::iterator I, unsigned DestReg, @@ -846,25 +880,33 @@ void X86InstrInfo::reMaterialize(MachineBasicBlock &MBB, // MOV32r0 etc. are implemented with xor which clobbers condition code. // Re-materialize them as movri instructions to avoid side effects. + bool Emitted = false; switch (Orig->getOpcode()) { + default: break; case X86::MOV8r0: - BuildMI(MBB, I, get(X86::MOV8ri), DestReg).addImm(0); - break; case X86::MOV16r0: - BuildMI(MBB, I, get(X86::MOV16ri), DestReg).addImm(0); - break; case X86::MOV32r0: - BuildMI(MBB, I, get(X86::MOV32ri), DestReg).addImm(0); - break; - case X86::MOV64r0: - BuildMI(MBB, I, get(X86::MOV64ri32), DestReg).addImm(0); + case X86::MOV64r0: { + if (!isSafeToClobberEFLAGS(MBB, I)) { + unsigned Opc = 0; + switch (Orig->getOpcode()) { + default: break; + case X86::MOV8r0: Opc = X86::MOV8ri; break; + case X86::MOV16r0: Opc = X86::MOV16ri; break; + case X86::MOV32r0: Opc = X86::MOV32ri; break; + case X86::MOV64r0: Opc = X86::MOV64ri32; break; + } + BuildMI(MBB, I, get(Opc), DestReg).addImm(0); + Emitted = true; + } break; - default: { + } + } + + if (!Emitted) { MachineInstr *MI = Orig->clone(); MI->getOperand(0).setReg(DestReg); MBB.insert(I, MI); - break; - } } if (ChangeSubIdx) { |