6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-06-17 20:23:59 +00:00
Code Issues Projects Releases Wiki Activity

Don't recompute MachineRegisterInfo in the Optimize* method.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@116750 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Bill Wendling
2010-10-18 21:22:31 +00:00
parent 9d28fdd722
commit b41ee96d76
4 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
include/llvm/Target/TargetInstrInfo.h
View File

@ -600,6 +600,7 @@ public:
/// *only* if a transformation took place. /// *only* if a transformation took place.
virtual bool OptimizeCompareInstr(MachineInstr *CmpInstr, virtual bool OptimizeCompareInstr(MachineInstr *CmpInstr,
unsigned SrcReg, int Mask, int Value, unsigned SrcReg, int Mask, int Value,
const MachineRegisterInfo *MRI,
MachineBasicBlock::iterator &) const { MachineBasicBlock::iterator &) const {
return false; return false;
} }

2
lib/CodeGen/PeepholeOptimizer.cpp
View File

@ -247,7 +247,7 @@ bool PeepholeOptimizer::OptimizeCmpInstr(MachineInstr *MI,
return false; return false;
// Attempt to optimize the comparison instruction. // Attempt to optimize the comparison instruction.
if (TII->OptimizeCompareInstr(MI, SrcReg, CmpMask, CmpValue, NextIter)) { if (TII->OptimizeCompareInstr(MI, SrcReg, CmpMask, CmpValue, MRI, NextIter)) {
++NumEliminated; ++NumEliminated;
return true; return true;
} }

12
lib/Target/ARM/ARMBaseInstrInfo.cpp
View File

@ -1490,13 +1490,13 @@ static bool isSuitableForMask(MachineInstr *&MI, unsigned SrcReg,
/// iterator *only* if a transformation took place. /// iterator *only* if a transformation took place.
bool ARMBaseInstrInfo:: bool ARMBaseInstrInfo::
OptimizeCompareInstr(MachineInstr *CmpInstr, unsigned SrcReg, int CmpMask, OptimizeCompareInstr(MachineInstr *CmpInstr, unsigned SrcReg, int CmpMask,
int CmpValue, MachineBasicBlock::iterator &MII) const { int CmpValue, const MachineRegisterInfo *MRI,
MachineBasicBlock::iterator &MII) const {
if (CmpValue != 0) if (CmpValue != 0)
return false; return false;
MachineRegisterInfo &MRI = CmpInstr->getParent()->getParent()->getRegInfo(); MachineRegisterInfo::def_iterator DI = MRI->def_begin(SrcReg);
MachineRegisterInfo::def_iterator DI = MRI.def_begin(SrcReg); if (llvm::next(DI) != MRI->def_end())
if (llvm::next(DI) != MRI.def_end())
// Only support one definition. // Only support one definition.
return false; return false;
@ -1506,8 +1506,8 @@ OptimizeCompareInstr(MachineInstr *CmpInstr, unsigned SrcReg, int CmpMask,
if (CmpMask != ~0) { if (CmpMask != ~0) {
if (!isSuitableForMask(MI, SrcReg, CmpMask, false)) { if (!isSuitableForMask(MI, SrcReg, CmpMask, false)) {
MI = 0; MI = 0;
for (MachineRegisterInfo::use_iterator UI = MRI.use_begin(SrcReg), for (MachineRegisterInfo::use_iterator UI = MRI->use_begin(SrcReg),
UE = MRI.use_end(); UI != UE; ++UI) { UE = MRI->use_end(); UI != UE; ++UI) {
if (UI->getParent() != CmpInstr->getParent()) continue; if (UI->getParent() != CmpInstr->getParent()) continue;
MachineInstr *PotentialAND = &*UI; MachineInstr *PotentialAND = &*UI;
if (!isSuitableForMask(PotentialAND, SrcReg, CmpMask, true)) if (!isSuitableForMask(PotentialAND, SrcReg, CmpMask, true))

1
lib/Target/ARM/ARMBaseInstrInfo.h
View File

@ -341,6 +341,7 @@ public:
/// that we can remove a "comparison with zero". /// that we can remove a "comparison with zero".
virtual bool OptimizeCompareInstr(MachineInstr *CmpInstr, unsigned SrcReg, virtual bool OptimizeCompareInstr(MachineInstr *CmpInstr, unsigned SrcReg,
int CmpMask, int CmpValue, int CmpMask, int CmpValue,
const MachineRegisterInfo *MRI,
MachineBasicBlock::iterator &MII) const; MachineBasicBlock::iterator &MII) const;
virtual unsigned getNumMicroOps(const MachineInstr *MI, virtual unsigned getNumMicroOps(const MachineInstr *MI,