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

[SimplifyCFG] Update comments that refer to CondBB to say ThenBB instead. NFC 

There is no variable in this function named CondBB, but there is one named ThenBB and I believe the comments are all refering to it.

llvm-svn: 343548
This commit is contained in:
Craig Topper 2018-10-01 22:56:11 +00:00
parent a67765ac8d
commit 90c0a0621c
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/Transforms/Utils/SimplifyCFG.cpp
View File

@ -1940,11 +1940,11 @@ static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *ThenBB,
} }
assert(EndBB == BI->getSuccessor(!Invert) && "No edge from to end block"); assert(EndBB == BI->getSuccessor(!Invert) && "No edge from to end block");
// Keep a count of how many times instructions are used within CondBB when // Keep a count of how many times instructions are used within ThenBB when
// they are candidates for sinking into CondBB. Specifically: // they are candidates for sinking into ThenBB. Specifically:
// - They are defined in BB, and // - They are defined in BB, and
// - They have no side effects, and // - They have no side effects, and
// - All of their uses are in CondBB. // - All of their uses are in ThenBB.
SmallDenseMap<Instruction *, unsigned, 4> SinkCandidateUseCounts; SmallDenseMap<Instruction *, unsigned, 4> SinkCandidateUseCounts;
SmallVector<Instruction *, 4> SpeculatedDbgIntrinsics; SmallVector<Instruction *, 4> SpeculatedDbgIntrinsics;
@ -1994,7 +1994,7 @@ static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *ThenBB,
} }
} }
// Consider any sink candidates which are only used in CondBB as costs for // Consider any sink candidates which are only used in ThenBB as costs for
// speculation. Note, while we iterate over a DenseMap here, we are summing // speculation. Note, while we iterate over a DenseMap here, we are summing
// and so iteration order isn't significant. // and so iteration order isn't significant.
for (SmallDenseMap<Instruction *, unsigned, 4>::iterator for (SmallDenseMap<Instruction *, unsigned, 4>::iterator