[llvm-commits] [llvm] r97870 - /llvm/trunk/lib/Target/Sparc/AsmPrinter/SparcAsmPrinter.cpp
Chris Lattner
sabre at nondot.org
Fri Mar 5 23:02:29 PST 2010
Author: lattner
Date: Sat Mar 6 01:02:28 2010
New Revision: 97870
URL: http://llvm.org/viewvc/llvm-project?rev=97870&view=rev
Log:
clean this up.
Modified:
llvm/trunk/lib/Target/Sparc/AsmPrinter/SparcAsmPrinter.cpp
Modified: llvm/trunk/lib/Target/Sparc/AsmPrinter/SparcAsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Sparc/AsmPrinter/SparcAsmPrinter.cpp?rev=97870&r1=97869&r2=97870&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Sparc/AsmPrinter/SparcAsmPrinter.cpp (original)
+++ llvm/trunk/lib/Target/Sparc/AsmPrinter/SparcAsmPrinter.cpp Sat Mar 6 01:02:28 2010
@@ -204,9 +204,10 @@
/// isBlockOnlyReachableByFallthough - Return true if the basic block has
/// exactly one predecessor and the control transfer mechanism between
/// the predecessor and this block is a fall-through.
-/// Override AsmPrinter implementation to handle delay slots
-bool SparcAsmPrinter::isBlockOnlyReachableByFallthrough(const MachineBasicBlock *MBB)
- const {
+///
+/// This overrides AsmPrinter's implementation to handle delay slots.
+bool SparcAsmPrinter::
+isBlockOnlyReachableByFallthrough(const MachineBasicBlock *MBB) const {
// If this is a landing pad, it isn't a fall through. If it has no preds,
// then nothing falls through to it.
if (MBB->isLandingPad() || MBB->pred_empty())
@@ -224,10 +225,10 @@
if (!Pred->isLayoutSuccessor(MBB))
return false;
- // Check if the last terminator is an unconditional branch
+ // Check if the last terminator is an unconditional branch.
MachineBasicBlock::const_iterator I = Pred->end();
- while( I != Pred->begin() && !(--I)->getDesc().isTerminator() )
- ; /* Noop */
+ while (I != Pred->begin() && !(--I)->getDesc().isTerminator())
+ ; // Noop
return I == Pred->end() || !I->getDesc().isBarrier();
}
More information about the llvm-commits
mailing list