[llvm] r297529 - Strip trailing whitespace.
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 10 14:53:20 PST 2017
Author: rksimon
Date: Fri Mar 10 16:53:19 2017
New Revision: 297529
URL: http://llvm.org/viewvc/llvm-project?rev=297529&view=rev
Log:
Strip trailing whitespace.
Modified:
llvm/trunk/lib/CodeGen/BranchCoalescing.cpp
Modified: llvm/trunk/lib/CodeGen/BranchCoalescing.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/BranchCoalescing.cpp?rev=297529&r1=297528&r2=297529&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/BranchCoalescing.cpp (original)
+++ llvm/trunk/lib/CodeGen/BranchCoalescing.cpp Fri Mar 10 16:53:19 2017
@@ -178,9 +178,9 @@ public:
bool mergeCandidates(CoalescingCandidateInfo &SourceRegion,
CoalescingCandidateInfo &TargetRegion);
- bool canMoveToBeginning(const MachineInstr &MI,
+ bool canMoveToBeginning(const MachineInstr &MI,
const MachineBasicBlock &MBB) const;
- bool canMoveToEnd(const MachineInstr &MI,
+ bool canMoveToEnd(const MachineInstr &MI,
const MachineBasicBlock &MBB) const;
bool canMerge(CoalescingCandidateInfo &SourceRegion,
CoalescingCandidateInfo &TargetRegion) const;
@@ -276,10 +276,10 @@ bool BranchCoalescing::canCoalesceBranch
assert(Cand.BranchBlock->canFallThrough() &&
"Expecting the block to fall through!");
- // We have already ensured there are exactly two successors to
- // BranchBlock and that BranchTargetBlock is a successor to BranchBlock.
+ // We have already ensured there are exactly two successors to
+ // BranchBlock and that BranchTargetBlock is a successor to BranchBlock.
// Ensure the single fall though block is empty.
- MachineBasicBlock *Succ =
+ MachineBasicBlock *Succ =
(*Cand.BranchBlock->succ_begin() == Cand.BranchTargetBlock)
? *Cand.BranchBlock->succ_rbegin()
: *Cand.BranchBlock->succ_begin();
@@ -292,7 +292,7 @@ bool BranchCoalescing::canCoalesceBranch
}
if (!Succ->isSuccessor(Cand.BranchTargetBlock)) {
- DEBUG(dbgs()
+ DEBUG(dbgs()
<< "Successor of fall through block is not branch taken block\n");
return false;
}
@@ -353,8 +353,8 @@ bool BranchCoalescing::identicalOperands
}
///
-/// Moves ALL PHI instructions in SourceMBB to beginning of TargetMBB
-/// and update them to refer to the new block. PHI node ordering
+/// Moves ALL PHI instructions in SourceMBB to beginning of TargetMBB
+/// and update them to refer to the new block. PHI node ordering
/// cannot be assumed so it does not matter where the PHI instructions
/// are moved to in TargetMBB.
///
@@ -386,19 +386,19 @@ void BranchCoalescing::moveAndUpdatePHIs
///
/// This function checks if MI can be moved to the beginning of the TargetMBB
-/// following PHI instructions. A MI instruction can be moved to beginning of
+/// following PHI instructions. A MI instruction can be moved to beginning of
/// the TargetMBB if there are no uses of it within the TargetMBB PHI nodes.
///
/// \param[in] MI the machine instruction to move.
/// \param[in] TargetMBB the machine basic block to move to
-/// \return true if it is safe to move MI to beginning of TargetMBB,
+/// \return true if it is safe to move MI to beginning of TargetMBB,
/// false otherwise.
///
-bool BranchCoalescing::canMoveToBeginning(const MachineInstr &MI,
+bool BranchCoalescing::canMoveToBeginning(const MachineInstr &MI,
const MachineBasicBlock &TargetMBB
) const {
- DEBUG(dbgs() << "Checking if " << MI << " can move to beginning of "
+ DEBUG(dbgs() << "Checking if " << MI << " can move to beginning of "
<< TargetMBB.getNumber() << "\n");
for (auto &Def : MI.defs()) { // Looking at Def
@@ -417,19 +417,19 @@ bool BranchCoalescing::canMoveToBeginnin
///
/// This function checks if MI can be moved to the end of the TargetMBB,
/// immediately before the first terminator. A MI instruction can be moved
-/// to then end of the TargetMBB if no PHI node defines what MI uses within
+/// to then end of the TargetMBB if no PHI node defines what MI uses within
/// it's own MBB.
///
/// \param[in] MI the machine instruction to move.
/// \param[in] TargetMBB the machine basic block to move to
-/// \return true if it is safe to move MI to end of TargetMBB,
+/// \return true if it is safe to move MI to end of TargetMBB,
/// false otherwise.
///
-bool BranchCoalescing::canMoveToEnd(const MachineInstr &MI,
+bool BranchCoalescing::canMoveToEnd(const MachineInstr &MI,
const MachineBasicBlock &TargetMBB
) const {
- DEBUG(dbgs() << "Checking if " << MI << " can move to end of "
+ DEBUG(dbgs() << "Checking if " << MI << " can move to end of "
<< TargetMBB.getNumber() << "\n");
for (auto &Use : MI.uses()) {
@@ -437,9 +437,9 @@ bool BranchCoalescing::canMoveToEnd(cons
MachineInstr *DefInst = MRI->getVRegDef(Use.getReg());
if (DefInst->isPHI() && DefInst->getParent() == MI.getParent()) {
DEBUG(dbgs() << " *** Cannot move this instruction ***\n");
- return false;
- } else {
- DEBUG(dbgs() << " *** def is in another block -- safe to move!\n");
+ return false;
+ } else {
+ DEBUG(dbgs() << " *** def is in another block -- safe to move!\n");
}
}
}
@@ -482,8 +482,8 @@ bool BranchCoalescing::validateCandidate
/// Merging involves moving the instructions in the
/// TargetRegion.BranchTargetBlock (also SourceRegion.BranchBlock).
///
-/// This function first try to move instructions from the
-/// TargetRegion.BranchTargetBlock down, to the beginning of the
+/// This function first try to move instructions from the
+/// TargetRegion.BranchTargetBlock down, to the beginning of the
/// SourceRegion.BranchTargetBlock. This is not possible if any register defined
/// in TargetRegion.BranchTargetBlock is used in a PHI node in the
/// SourceRegion.BranchTargetBlock. In this case, check whether the statement
@@ -492,7 +492,7 @@ bool BranchCoalescing::validateCandidate
/// be merged.
///
/// Note that there is no analysis for moving instructions past the fall-through
-/// blocks because they are confirmed to be empty. An assert is thrown if they
+/// blocks because they are confirmed to be empty. An assert is thrown if they
/// are not.
///
/// \param[in] SourceRegion The candidate to move statements from
@@ -666,7 +666,7 @@ bool BranchCoalescing::mergeCandidates(C
assert(TargetRegion.FallThroughBlock->empty() &&
"FallThroughBlocks should be empty!");
- // Transfer successor information and move PHIs down to the
+ // Transfer successor information and move PHIs down to the
// branch-taken block.
TargetRegion.FallThroughBlock->transferSuccessorsAndUpdatePHIs(
SourceRegion.FallThroughBlock);
More information about the llvm-commits
mailing list