diff options
author | Craig Topper <craig.topper@intel.com> | 2018-10-01 22:56:11 +0000 |
---|---|---|
committer | Craig Topper <craig.topper@intel.com> | 2018-10-01 22:56:11 +0000 |
commit | 90c0a0621ccc9ee19ccc6e283982a5ef8270fe58 (patch) | |
tree | abbda4c4fd00346625ee472428942f1da22b504e /llvm/lib/Transforms/Utils/SimplifyCFG.cpp | |
parent | a67765ac8dfe8848be61c34b4192ee378b8afc72 (diff) | |
download | bcm5719-llvm-90c0a0621ccc9ee19ccc6e283982a5ef8270fe58.tar.gz bcm5719-llvm-90c0a0621ccc9ee19ccc6e283982a5ef8270fe58.zip |
[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
Diffstat (limited to 'llvm/lib/Transforms/Utils/SimplifyCFG.cpp')
-rw-r--r-- | llvm/lib/Transforms/Utils/SimplifyCFG.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp index 1e93b873f37..c5bd173df18 100644 --- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp @@ -1940,11 +1940,11 @@ static bool SpeculativelyExecuteBB(BranchInst *BI, BasicBlock *ThenBB, } assert(EndBB == BI->getSuccessor(!Invert) && "No edge from to end block"); - // Keep a count of how many times instructions are used within CondBB when - // they are candidates for sinking into CondBB. Specifically: + // Keep a count of how many times instructions are used within ThenBB when + // they are candidates for sinking into ThenBB. Specifically: // - They are defined in BB, 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; 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 // and so iteration order isn't significant. for (SmallDenseMap<Instruction *, unsigned, 4>::iterator |