diff options
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/CodeGen/MachineVerifier.cpp | 58 | 
1 files changed, 16 insertions, 42 deletions
diff --git a/llvm/lib/CodeGen/MachineVerifier.cpp b/llvm/lib/CodeGen/MachineVerifier.cpp index 02e48dd2feb..f851dcf8881 100644 --- a/llvm/lib/CodeGen/MachineVerifier.cpp +++ b/llvm/lib/CodeGen/MachineVerifier.cpp @@ -185,7 +185,7 @@ bool MachineVerifier::runOnMachineFunction(MachineFunction &MF) {        errs() << "Error opening '" << OutFileName << "': " << ErrorInfo << '\n';        exit(1);      } -     +      OS = OutFile;    } else {      OS = &errs(); @@ -238,9 +238,7 @@ void MachineVerifier::report(const char *msg, const MachineFunction *MF) {        << "- function:    " << MF->getFunction()->getNameStr() << "\n";  } -void -MachineVerifier::report(const char *msg, const MachineBasicBlock *MBB) -{ +void MachineVerifier::report(const char *msg, const MachineBasicBlock *MBB) {    assert(MBB);    report(msg, MBB->getParent());    *OS << "- basic block: " << MBB->getBasicBlock()->getNameStr() @@ -248,19 +246,15 @@ MachineVerifier::report(const char *msg, const MachineBasicBlock *MBB)        << " (#" << MBB->getNumber() << ")\n";  } -void -MachineVerifier::report(const char *msg, const MachineInstr *MI) -{ +void MachineVerifier::report(const char *msg, const MachineInstr *MI) {    assert(MI);    report(msg, MI->getParent());    *OS << "- instruction: ";    MI->print(*OS, TM);  } -void -MachineVerifier::report(const char *msg, -                        const MachineOperand *MO, unsigned MONum) -{ +void MachineVerifier::report(const char *msg, +                             const MachineOperand *MO, unsigned MONum) {    assert(MO);    report(msg, MO->getParent());    *OS << "- operand " << MONum << ":   "; @@ -268,9 +262,7 @@ MachineVerifier::report(const char *msg,    *OS << "\n";  } -void -MachineVerifier::markReachable(const MachineBasicBlock *MBB) -{ +void MachineVerifier::markReachable(const MachineBasicBlock *MBB) {    BBInfo &MInfo = MBBInfoMap[MBB];    if (!MInfo.reachable) {      MInfo.reachable = true; @@ -280,9 +272,7 @@ MachineVerifier::markReachable(const MachineBasicBlock *MBB)    }  } -void -MachineVerifier::visitMachineFunctionBefore() -{ +void MachineVerifier::visitMachineFunctionBefore() {    regsReserved = TRI->getReservedRegs(*MF);    // A sub-register of a reserved register is also reserved @@ -297,9 +287,7 @@ MachineVerifier::visitMachineFunctionBefore()    markReachable(&MF->front());  } -void -MachineVerifier::visitMachineBasicBlockBefore(const MachineBasicBlock *MBB) -{ +void MachineVerifier::visitMachineBasicBlockBefore(const MachineBasicBlock *MBB) {    const TargetInstrInfo *TII = MF->getTarget().getInstrInfo();    // Start with minimal CFG sanity checks. @@ -462,9 +450,7 @@ MachineVerifier::visitMachineBasicBlockBefore(const MachineBasicBlock *MBB)    regsDefined.clear();  } -void -MachineVerifier::visitMachineInstrBefore(const MachineInstr *MI) -{ +void MachineVerifier::visitMachineInstrBefore(const MachineInstr *MI) {    const TargetInstrDesc &TI = MI->getDesc();    if (MI->getNumOperands() < TI.getNumOperands()) {      report("Too few operands", MI); @@ -474,8 +460,7 @@ MachineVerifier::visitMachineInstrBefore(const MachineInstr *MI)  }  void -MachineVerifier::visitMachineOperand(const MachineOperand *MO, unsigned MONum) -{ +MachineVerifier::visitMachineOperand(const MachineOperand *MO, unsigned MONum) {    const MachineInstr *MI = MO->getParent();    const TargetInstrDesc &TI = MI->getDesc(); @@ -608,9 +593,7 @@ MachineVerifier::visitMachineOperand(const MachineOperand *MO, unsigned MONum)    }  } -void -MachineVerifier::visitMachineInstrAfter(const MachineInstr *MI) -{ +void MachineVerifier::visitMachineInstrAfter(const MachineInstr *MI) {    BBInfo &MInfo = MBBInfoMap[MI->getParent()];    set_union(MInfo.regsKilled, regsKilled);    set_subtract(regsLive, regsKilled); @@ -650,8 +633,7 @@ MachineVerifier::visitMachineInstrAfter(const MachineInstr *MI)  }  void -MachineVerifier::visitMachineBasicBlockAfter(const MachineBasicBlock *MBB) -{ +MachineVerifier::visitMachineBasicBlockAfter(const MachineBasicBlock *MBB) {    MBBInfoMap[MBB].regsLiveOut = regsLive;    regsLive.clear();  } @@ -659,9 +641,7 @@ MachineVerifier::visitMachineBasicBlockAfter(const MachineBasicBlock *MBB)  // Calculate the largest possible vregsPassed sets. These are the registers that  // can pass through an MBB live, but may not be live every time. It is assumed  // that all vregsPassed sets are empty before the call. -void -MachineVerifier::calcMaxRegsPassed() -{ +void MachineVerifier::calcMaxRegsPassed() {    // First push live-out regs to successors' vregsPassed. Remember the MBBs that    // have any vregsPassed.    DenseSet<const MachineBasicBlock*> todo; @@ -699,9 +679,7 @@ MachineVerifier::calcMaxRegsPassed()  // Calculate the minimum vregsPassed set. These are the registers that always  // pass live through an MBB. The calculation assumes that calcMaxRegsPassed has  // been called earlier. -void -MachineVerifier::calcMinRegsPassed() -{ +void MachineVerifier::calcMinRegsPassed() {    DenseSet<const MachineBasicBlock*> todo;    for (MachineFunction::const_iterator MFI = MF->begin(), MFE = MF->end();         MFI != MFE; ++MFI) @@ -736,9 +714,7 @@ MachineVerifier::calcMinRegsPassed()  // Check PHI instructions at the beginning of MBB. It is assumed that  // calcMinRegsPassed has been run so BBInfo::isLiveOut is valid. -void -MachineVerifier::checkPHIOps(const MachineBasicBlock *MBB) -{ +void MachineVerifier::checkPHIOps(const MachineBasicBlock *MBB) {    for (MachineBasicBlock::const_iterator BBI = MBB->begin(), BBE = MBB->end();         BBI != BBE && BBI->getOpcode() == TargetInstrInfo::PHI; ++BBI) {      DenseSet<const MachineBasicBlock*> seen; @@ -767,9 +743,7 @@ MachineVerifier::checkPHIOps(const MachineBasicBlock *MBB)    }  } -void -MachineVerifier::visitMachineFunctionAfter() -{ +void MachineVerifier::visitMachineFunctionAfter() {    calcMaxRegsPassed();    // With the maximal set of vregsPassed we can verify dead-in registers.  | 

