diff options
author | Bill Wendling <isanbard@gmail.com> | 2010-09-10 23:46:12 +0000 |
---|---|---|
committer | Bill Wendling <isanbard@gmail.com> | 2010-09-10 23:46:12 +0000 |
commit | 3665661a5708c8adc2727be38b56d1d87ddeb661 (patch) | |
tree | 950e2818cdb04156591a728b71d31ea9f79b8425 /lib | |
parent | 92ad57f066e9f256e4e3d72febf152e68caa80c7 (diff) | |
download | llvm-3665661a5708c8adc2727be38b56d1d87ddeb661.tar.gz llvm-3665661a5708c8adc2727be38b56d1d87ddeb661.tar.bz2 llvm-3665661a5708c8adc2727be38b56d1d87ddeb661.tar.xz |
No need to recompute the SrcReg and CmpValue.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@113666 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/CodeGen/PeepholeOptimizer.cpp | 2 | ||||
-rw-r--r-- | lib/Target/ARM/ARMBaseInstrInfo.cpp | 6 | ||||
-rw-r--r-- | lib/Target/ARM/ARMBaseInstrInfo.h | 3 |
3 files changed, 5 insertions, 6 deletions
diff --git a/lib/CodeGen/PeepholeOptimizer.cpp b/lib/CodeGen/PeepholeOptimizer.cpp index 7a1bf4050d..529f994f00 100644 --- a/lib/CodeGen/PeepholeOptimizer.cpp +++ b/lib/CodeGen/PeepholeOptimizer.cpp @@ -244,7 +244,7 @@ bool PeepholeOptimizer::OptimizeCmpInstr(MachineInstr *MI, return false; // Attempt to convert the defining instruction to set the "zero" flag. - if (TII->ConvertToSetZeroFlag(MI, NextIter)) { + if (TII->ConvertToSetZeroFlag(MI, SrcReg, CmpValue, NextIter)) { ++NumEliminated; return true; } diff --git a/lib/Target/ARM/ARMBaseInstrInfo.cpp b/lib/Target/ARM/ARMBaseInstrInfo.cpp index a73e7ad44d..d79905a80b 100644 --- a/lib/Target/ARM/ARMBaseInstrInfo.cpp +++ b/lib/Target/ARM/ARMBaseInstrInfo.cpp @@ -1381,11 +1381,9 @@ AnalyzeCompare(const MachineInstr *MI, unsigned &SrcReg, int &CmpValue) const { /// comparison into one that sets the zero bit in the flags register. Update the /// iterator *only* if a transformation took place. bool ARMBaseInstrInfo:: -ConvertToSetZeroFlag(MachineInstr *CmpInstr, +ConvertToSetZeroFlag(MachineInstr *CmpInstr, unsigned SrcReg, int CmpValue, MachineBasicBlock::iterator &MII) const { - unsigned SrcReg; - int CmpValue; - if (!AnalyzeCompare(CmpInstr, SrcReg, CmpValue) || CmpValue != 0) + if (CmpValue != 0) return false; MachineRegisterInfo &MRI = CmpInstr->getParent()->getParent()->getRegInfo(); diff --git a/lib/Target/ARM/ARMBaseInstrInfo.h b/lib/Target/ARM/ARMBaseInstrInfo.h index 2be0d3687d..e76e2fe402 100644 --- a/lib/Target/ARM/ARMBaseInstrInfo.h +++ b/lib/Target/ARM/ARMBaseInstrInfo.h @@ -346,7 +346,8 @@ public: /// ConvertToSetZeroFlag - Convert the instruction to set the zero flag so /// that we can remove a "comparison with zero". - virtual bool ConvertToSetZeroFlag(MachineInstr *CmpInstr, + virtual bool ConvertToSetZeroFlag(MachineInstr *CmpInstr, unsigned SrcReg, + int CmpValue, MachineBasicBlock::iterator &MII) const; virtual unsigned getNumMicroOps(const MachineInstr *MI, |