maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Simplify code a bit, don't go off the end of the block, now that the current 

block we are in might be empty

llvm-svn: 11744
This commit is contained in:
Chris Lattner 2004-02-23 07:42:19 +00:00
parent 4ffd4443ce
commit cb185a34bb
1 changed files with 9 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
llvm/lib/Target/X86/InstSelectSimple.cpp
View File

@ -682,7 +682,6 @@ void ISel::InsertFPRegKills() {
const TargetInstrInfo &TII = TM.getInstrInfo(); const TargetInstrInfo &TII = TM.getInstrInfo();
for (MachineFunction::iterator BB = F->begin(), E = F->end(); BB != E; ++BB) { for (MachineFunction::iterator BB = F->begin(), E = F->end(); BB != E; ++BB) {
bool UsesFPReg = false;
for (MachineBasicBlock::iterator I = BB->begin(), E = BB->end(); I!=E; ++I) for (MachineBasicBlock::iterator I = BB->begin(), E = BB->end(); I!=E; ++I)
for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i)
if (I->getOperand(i).isRegister()) { if (I->getOperand(i).isRegister()) {
@ -695,16 +694,15 @@ void ISel::InsertFPRegKills() {
// If we haven't found an FP register use or def in this basic block, check // If we haven't found an FP register use or def in this basic block, check
// to see if any of our successors has an FP PHI node, which will cause a // to see if any of our successors has an FP PHI node, which will cause a
// copy to be inserted into this block. // copy to be inserted into this block.
if (!UsesFPReg) for (succ_const_iterator SI = succ_begin(BB->getBasicBlock()),
for (succ_const_iterator SI = succ_begin(BB->getBasicBlock()), E = succ_end(BB->getBasicBlock()); SI != E; ++SI) {
E = succ_end(BB->getBasicBlock()); SI != E; ++SI) { MachineBasicBlock *SBB = MBBMap[*SI];
MachineBasicBlock *SBB = MBBMap[*SI]; for (MachineBasicBlock::iterator I = SBB->begin();
for (MachineBasicBlock::iterator I = SBB->begin(); I != SBB->end() && I->getOpcode() == X86::PHI; ++I) {
I != SBB->end() && I->getOpcode() == X86::PHI; ++I) { if (RegMap.getRegClass(I->getOperand(0).getReg())->getSize() == 10)
if (RegMap.getRegClass(I->getOperand(0).getReg())->getSize() == 10) goto UsesFPReg;
goto UsesFPReg;
}
} }
}
continue; continue;
UsesFPReg: UsesFPReg:
// Okay, this block uses an FP register. If the block has successors (ie, // Okay, this block uses an FP register. If the block has successors (ie,
@ -714,7 +712,7 @@ void ISel::InsertFPRegKills() {
// Rewind past any terminator instructions that might exist. // Rewind past any terminator instructions that might exist.
MachineBasicBlock::iterator I = BB->end(); MachineBasicBlock::iterator I = BB->end();
while (I != BB->begin() && TII.isTerminatorInstr((--I)->getOpcode())); while (I != BB->begin() && TII.isTerminatorInstr((--I)->getOpcode()));
++I; if (I != BB->end()) ++I;
BMI(BB, I, X86::FP_REG_KILL, 0); BMI(BB, I, X86::FP_REG_KILL, 0);
++NumFPKill; ++NumFPKill;
} }